Re: [Qemu-devel] [PATCH 1/4] error: Strip trailing '\n' from error string arguments (again)

[Hailiang Zhang <zhang.zhanghailiang@huawei.com>]

On 2015/12/11 2:16, Cornelia Huck wrote:
> On Thu, 10 Dec 2015 18:23:49 +0100
> Markus Armbruster <armbru@redhat.com> 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 <famz@redhat.com>
>> Cc: Peter Crosthwaite <crosthwaitepeter@gmail.com>
>> Cc: Bharata B Rao <bharata@linux.vnet.ibm.com>
>> Cc: Dominik Dingel <dingel@linux.vnet.ibm.com>
>> Cc: David Hildenbrand <dahi@linux.vnet.ibm.com>
>> Cc: Jason J. Herne <jjherne@linux.vnet.ibm.com>
>> Cc: Stefan Berger <stefanb@linux.vnet.ibm.com>
>> Cc: Dr. David Alan Gilbert <dgilbert@redhat.com>
>> Cc: Changchun Ouyang <changchun.ouyang@intel.com>
>> Cc: zhanghailiang <zhang.zhanghailiang@huawei.com>
>> Cc: Pavel Fedin <p.fedin@samsung.com>
>> Signed-off-by: Markus Armbruster <armbru@redhat.com>
>> ---
>>   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 <cornelia.huck@de.ibm.com>
>
> Can somebody with Perl-fu extend checkpatch.pl if these keep coming
> back?
>
>

Ha, that's really a good idea ...

> .
>