Same logging messages were used for both IOTLB cache
insertion failure and IOTLB pending insertion failure.

This patch differentiate them to ease logs analysis.

Suggested-by: David Marchand <david.march...@redhat.com>
Signed-off-by: Maxime Coquelin <maxime.coque...@redhat.com>
---
 lib/vhost/iotlb.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/lib/vhost/iotlb.c b/lib/vhost/iotlb.c
index afa86d7c2b..adc426612a 100644
--- a/lib/vhost/iotlb.c
+++ b/lib/vhost/iotlb.c
@@ -70,7 +70,8 @@ vhost_user_iotlb_pending_insert(struct virtio_net *dev, 
struct vhost_virtqueue *
 
        ret = rte_mempool_get(vq->iotlb_pool, (void **)&node);
        if (ret) {
-               VHOST_LOG_CONFIG(DEBUG, "(%s) IOTLB pool %s empty, clear 
entries\n",
+               VHOST_LOG_CONFIG(DEBUG,
+                               "(%s) IOTLB pool %s empty, clear entries for 
pending insertion\n",
                                dev->ifname, vq->iotlb_pool->name);
                if (!TAILQ_EMPTY(&vq->iotlb_pending_list))
                        vhost_user_iotlb_pending_remove_all(vq);
@@ -78,7 +79,8 @@ vhost_user_iotlb_pending_insert(struct virtio_net *dev, 
struct vhost_virtqueue *
                        vhost_user_iotlb_cache_random_evict(vq);
                ret = rte_mempool_get(vq->iotlb_pool, (void **)&node);
                if (ret) {
-                       VHOST_LOG_CONFIG(ERR, "(%s) IOTLB pool %s still empty, 
failure\n",
+                       VHOST_LOG_CONFIG(ERR,
+                                       "(%s) IOTLB pool %s still empty, 
prending insertion failure\n",
                                        dev->ifname, vq->iotlb_pool->name);
                        return;
                }
@@ -167,7 +169,8 @@ vhost_user_iotlb_cache_insert(struct virtio_net *dev, 
struct vhost_virtqueue *vq
 
        ret = rte_mempool_get(vq->iotlb_pool, (void **)&new_node);
        if (ret) {
-               VHOST_LOG_CONFIG(DEBUG, "(%s) IOTLB pool %s empty, clear 
entries\n",
+               VHOST_LOG_CONFIG(DEBUG,
+                               "(%s) IOTLB pool %s empty, clear entries for 
cache insertion\n",
                                dev->ifname, vq->iotlb_pool->name);
                if (!TAILQ_EMPTY(&vq->iotlb_list))
                        vhost_user_iotlb_cache_random_evict(vq);
@@ -175,7 +178,8 @@ vhost_user_iotlb_cache_insert(struct virtio_net *dev, 
struct vhost_virtqueue *vq
                        vhost_user_iotlb_pending_remove_all(vq);
                ret = rte_mempool_get(vq->iotlb_pool, (void **)&new_node);
                if (ret) {
-                       VHOST_LOG_CONFIG(ERR, "(%s) IOTLB pool %s still empty, 
failure\n",
+                       VHOST_LOG_CONFIG(ERR,
+                                       "(%s) IOTLB pool %s still empty, cache 
insertion failed\n",
                                        dev->ifname, vq->iotlb_pool->name);
                        return;
                }
-- 
2.34.1

Reply via email to