From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39402) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a7JaG-0000O7-Tu for qemu-devel@nongnu.org; Fri, 11 Dec 2015 04:01:49 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a7JaA-0003Pc-T1 for qemu-devel@nongnu.org; Fri, 11 Dec 2015 04:01:48 -0500 Received: from szxga01-in.huawei.com ([58.251.152.64]:36921) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a7JaA-0003PU-1J for qemu-devel@nongnu.org; Fri, 11 Dec 2015 04:01:42 -0500 References: <1449768232-22924-1-git-send-email-armbru@redhat.com> <1449768232-22924-2-git-send-email-armbru@redhat.com> <20151210191616.7abafe7f.cornelia.huck@de.ibm.com> From: Hailiang Zhang Message-ID: <566A90E0.5070000@huawei.com> Date: Fri, 11 Dec 2015 17:01:20 +0800 MIME-Version: 1.0 In-Reply-To: <20151210191616.7abafe7f.cornelia.huck@de.ibm.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 1/4] error: Strip trailing '\n' from error string arguments (again) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Cornelia Huck , Markus Armbruster Cc: Fam Zheng , Stefan Berger , qemu-devel@nongnu.org, Pavel Fedin , peter.huangpeng@huawei.com, Dominik Dingel , "Dr. David Alan Gilbert" , David Hildenbrand , Peter Crosthwaite , "Jason J. Herne" , Bharata B Rao , Changchun Ouyang On 2015/12/11 2:16, Cornelia Huck wrote: > On Thu, 10 Dec 2015 18:23:49 +0100 > Markus Armbruster wrote: > >> Commit 6daf194d, be62a2eb and 312fd5f got rid of a bunch, but they >> keep coming back. Tracked down with the Coccinelle semantic patch >> from commit 312fd5f. >> >> Cc: Fam Zheng >> Cc: Peter Crosthwaite >> Cc: Bharata B Rao >> Cc: Dominik Dingel >> Cc: David Hildenbrand >> Cc: Jason J. Herne >> Cc: Stefan Berger >> Cc: Dr. David Alan Gilbert >> Cc: Changchun Ouyang >> Cc: zhanghailiang >> Cc: Pavel Fedin >> Signed-off-by: Markus Armbruster >> --- >> block/vmdk.c | 4 ++-- >> hw/arm/xlnx-zynqmp.c | 2 +- >> hw/ppc/spapr.c | 3 ++- >> hw/s390x/ipl.c | 8 ++++---- >> hw/s390x/s390-skeys-kvm.c | 2 +- >> hw/s390x/s390-skeys.c | 16 ++++++++-------- >> hw/tpm/tpm_tis.c | 2 +- >> migration/ram.c | 2 +- >> migration/savevm.c | 4 ++-- >> net/vhost-user.c | 6 +++--- >> qga/commands-posix.c | 2 +- >> target-arm/cpu.c | 2 +- >> target-arm/machine.c | 4 ++-- >> 13 files changed, 29 insertions(+), 28 deletions(-) >> > > s390x parts: > > Acked-by: Cornelia Huck > > Can somebody with Perl-fu extend checkpatch.pl if these keep coming > back? > > Ha, that's really a good idea ... > . >