Re: [PATCH] misc: Remove redundant new line in perror()

[Laurent Vivier <laurent@vivier.eu>]

Le 06/07/2021 à 11:55, Philippe Mathieu-Daudé a écrit :
> Cc'ing qemu-trivial@
> 
> On 7/6/21 11:44 AM, Li Zhijian wrote:
>> Signed-off-by: Li Zhijian <lizhijian@cn.fujitsu.com>
>> ---
>>  migration/rdma.c | 2 +-
>>  softmmu/cpus.c   | 2 +-
>>  2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/migration/rdma.c b/migration/rdma.c
>> index 7fb9646f6ef..e99e2e16a73 100644
>> --- a/migration/rdma.c
>> +++ b/migration/rdma.c
>> @@ -1133,7 +1133,7 @@ static int qemu_rdma_reg_whole_ram_blocks(RDMAContext *rdma)
>>                      IBV_ACCESS_REMOTE_WRITE
>>                      );
>>          if (!local->block[i].mr) {
>> -            perror("Failed to register local dest ram block!\n");
>> +            perror("Failed to register local dest ram block!");
>>              break;
>>          }
>>          rdma->total_registrations++;
>> diff --git a/softmmu/cpus.c b/softmmu/cpus.c
>> index c3caaeb26ec..071085f840b 100644
>> --- a/softmmu/cpus.c
>> +++ b/softmmu/cpus.c
>> @@ -325,7 +325,7 @@ static void sigbus_reraise(void)
>>          sigaddset(&set, SIGBUS);
>>          pthread_sigmask(SIG_UNBLOCK, &set, NULL);
>>      }
>> -    perror("Failed to re-raise SIGBUS!\n");
>> +    perror("Failed to re-raise SIGBUS!");
>>      abort();
>>  }
> 
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> 
>


Applied to my trivial-patches branch.

Thanks,
Laurent