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

[Philippe Mathieu-Daudé]

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>