Re: [PATCH] migration/rdma: Use error_report to suppress errno message

[Dr. David Alan Gilbert]

* Li Zhijian (lizhijian@cn.fujitsu.com) wrote:
> Since the prior calls are successful, in this case a errno doesn't
> indicate a real error which would just make us confused.
> 
> before:
> (qemu) migrate -d rdma:192.168.22.23:8888
> source_resolve_host RDMA Device opened: kernel name rxe_eth0 uverbs device 
> name uverbs2, infiniband_verbs class device path 
> /sys/class/infiniband_verbs/uverbs2, infiniband class device path 
> /sys/class/infiniband/rxe_eth0, transport: (2) Ethernet
> rdma_get_cm_event != EVENT_ESTABLISHED after rdma_connect: No space left on 
> device
> 
> Signed-off-by: Li Zhijian <lizhijian@cn.fujitsu.com>


Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>

and queued

> ---
>  migration/rdma.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/migration/rdma.c b/migration/rdma.c
> index d90b29a4b51..b6cc4bef4a8 100644
> --- a/migration/rdma.c
> +++ b/migration/rdma.c
> @@ -1006,7 +1006,7 @@ route:
>      if (cm_event->event != RDMA_CM_EVENT_ADDR_RESOLVED) {
>          ERROR(errp, "result not equal to event_addr_resolved %s",
>                  rdma_event_str(cm_event->event));
> -        perror("rdma_resolve_addr");
> +        error_report("rdma_resolve_addr");
>          rdma_ack_cm_event(cm_event);
>          ret = -EINVAL;
>          goto err_resolve_get_addr;
> @@ -2544,7 +2544,7 @@ static int qemu_rdma_connect(RDMAContext *rdma, Error 
> **errp, bool return_path)
>      }
>  
>      if (cm_event->event != RDMA_CM_EVENT_ESTABLISHED) {
> -        perror("rdma_get_cm_event != EVENT_ESTABLISHED after rdma_connect");
> +        error_report("rdma_get_cm_event != EVENT_ESTABLISHED after 
> rdma_connect");
>          ERROR(errp, "connecting to destination!");
>          rdma_ack_cm_event(cm_event);
>          goto err_rdma_source_connect;
> -- 
> 2.31.1
> 
> 
> 
> 
-- 
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK