[ovs-dev] [PATCH] dpdk: Stop dumping memzones to stdout.

Ilya Maximets i.maximets at samsung.com
Thu Mar 14 14:43:48 UTC 2019


Information about memzones reserved on init is not much useful.
Anyway, we need to log it in more civilized manner, i.e. through
the OVS logging subsystem.

Signed-off-by: Ilya Maximets <i.maximets at samsung.com>
---
 lib/dpdk.c | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/lib/dpdk.c b/lib/dpdk.c
index d7901e175..dc6171546 100644
--- a/lib/dpdk.c
+++ b/lib/dpdk.c
@@ -403,7 +403,23 @@ dpdk_init__(const struct smap *ovs_other_config)
         return false;
     }
 
-    rte_memzone_dump(stdout);
+    if (VLOG_IS_DBG_ENABLED()) {
+        size_t size;
+        char *response = NULL;
+        FILE *stream = open_memstream(&response, &size);
+
+        if (stream) {
+            rte_memzone_dump(stream);
+            fclose(stream);
+            if (size) {
+                VLOG_DBG("rte_memzone_dump:\n%s", response);
+            }
+            free(response);
+        } else {
+            VLOG_DBG("Could not dump memzone. Unable to open memstream: %s.",
+                     ovs_strerror(errno));
+        }
+    }
 
     /* We are called from the main thread here */
     RTE_PER_LCORE(_lcore_id) = NON_PMD_CORE_ID;
-- 
2.17.1



More information about the dev mailing list