[PATCH] vhost: fix log for async dequeue
Maxime Coquelin
maxime.coquelin at redhat.com
Fri Jun 17 16:00:12 CEST 2022
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 at 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 at redhat.com>
Thanks,
Maxime
More information about the dev
mailing list