Hi David,
> -----Original Message-----
> From: David Marchand <david.march...@redhat.com>
> Sent: Monday, June 27, 2022 5:27 PM
> To: dev@dpdk.org
> Cc: maxime.coque...@redhat.com; Xia, Chenbo <chenbo....@intel.com>; Wang,
> YuanX <yuanx.w...@intel.com>; Ding, Xuan <xuan.d...@intel.com>
> Subject: [PATCH 2/4] vhost: restore device information in log messages
>
> device information in the log messages was dropped.
>
> Fixes: 52ade97e3641 ("vhost: fix physical address mapping")
Should we add cc-stable tag in this case as it's 22.03 commit?
Last time when I asked Ferruh, he said although now we don't maintain
non-LTS release, we should still add this tag. Not sure this changed
or not.
For the patch:
Reviewed-by: Chenbo Xia <chenbo....@intel.com>
>
> Signed-off-by: David Marchand <david.march...@redhat.com>
> ---
> lib/vhost/vhost_user.c | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/lib/vhost/vhost_user.c b/lib/vhost/vhost_user.c
> index 2b9a3b69fa..f324f822d6 100644
> --- a/lib/vhost/vhost_user.c
> +++ b/lib/vhost/vhost_user.c
> @@ -144,7 +144,8 @@ async_dma_map(struct virtio_net *dev, bool do_map)
> return;
>
> /* DMA mapping errors won't stop
> VHOST_USER_SET_MEM_TABLE. */
> - VHOST_LOG_CONFIG(ERR, "DMA engine map
> failed\n");
> + VHOST_LOG_CONFIG(ERR, "(%s) DMA engine map
> failed\n",
> + dev->ifname);
> }
> }
>
> @@ -160,7 +161,8 @@ async_dma_map(struct virtio_net *dev, bool do_map)
> if (rte_errno == EINVAL)
> return;
>
> - VHOST_LOG_CONFIG(ERR, "DMA engine unmap
> failed\n");
> + VHOST_LOG_CONFIG(ERR, "(%s) DMA engine unmap
> failed\n",
> + dev->ifname);
> }
> }
> }
> @@ -945,7 +947,8 @@ add_one_guest_page(struct virtio_net *dev, uint64_t
> guest_phys_addr,
> dev->max_guest_pages * sizeof(*page),
> RTE_CACHE_LINE_SIZE);
> if (dev->guest_pages == NULL) {
> - VHOST_LOG_CONFIG(ERR, "cannot realloc guest_pages\n");
> + VHOST_LOG_CONFIG(ERR, "(%s) cannot realloc
> guest_pages\n",
> + dev->ifname);
> rte_free(old_pages);
> return -1;
> }
> --
> 2.36.1
