Re: [PATCH] vhost: fix log for async dequeue

[Maxime Coquelin <maxime.coquelin@redhat.com>]

On 6/17/22 07:40, David Marchand wrote:
> Since the commit 02798b073520 ("vhost: improve virtio-net layer logs"),
> vhost logs contain the socket path as a prefix.
> Async dequeue path was copied from the sync dequeue path but a log
> was incorrect.
> 
> Fixes: 84d5204310d7 ("vhost: support async dequeue for split ring")
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
>   lib/vhost/virtio_net.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/vhost/virtio_net.c b/lib/vhost/virtio_net.c
> index 68a26eb17d..bedd406b37 100644
> --- a/lib/vhost/virtio_net.c
> +++ b/lib/vhost/virtio_net.c
> @@ -3529,7 +3529,7 @@ rte_vhost_async_try_dequeue_burst(int vid, uint16_t queue_id,
>   
>   		rarp_mbuf = rte_net_make_rarp_packet(mbuf_pool, &dev->mac);
>   		if (rarp_mbuf == NULL) {
> -			VHOST_LOG_DATA(ERR, "Failed to make RARP packet.\n");
> +			VHOST_LOG_DATA(ERR, "(%s) failed to make RARP packet.\n", dev->ifname);
>   			count = 0;
>   			goto out;
>   		}

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime