Currently nbl information is getting dumped whenever a nbl is copied or
allocated, since OVS_DBG_DEFAULT is set to OVS_DBG_INFO for debug builds,
which affects the ovs performance. Instead dump nbl information only when
OVS_DBG_DEFAULT is set to OVS_LOG_LOUD

Signed-off-by: Anand Kumar <[email protected]>
---
 datapath-windows/ovsext/BufferMgmt.c | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/datapath-windows/ovsext/BufferMgmt.c 
b/datapath-windows/ovsext/BufferMgmt.c
index 03470d7..448cd76 100644
--- a/datapath-windows/ovsext/BufferMgmt.c
+++ b/datapath-windows/ovsext/BufferMgmt.c
@@ -273,6 +273,7 @@ OvsInitNBLContext(POVS_BUFFER_CONTEXT ctx,
 static VOID
 OvsDumpForwardingDetails(PNET_BUFFER_LIST nbl)
 {
+#if OVS_DBG_DEFAULT >= OVS_DBG_LOUD
     PNDIS_SWITCH_FORWARDING_DETAIL_NET_BUFFER_LIST_INFO info;
     info = NET_BUFFER_LIST_SWITCH_FORWARDING_DETAIL(nbl);
     if (info == NULL) {
@@ -284,12 +285,15 @@ OvsDumpForwardingDetails(PNET_BUFFER_LIST nbl)
                  info->SourceNicIndex,
                  info->IsPacketDataSafe ? "TRUE" : "FALSE",
                  info->IsPacketDataSafe ? 0 : info->SafePacketDataSize);
-
+#else
+    UNREFERENCED_PARAMETER(nbl);
+#endif
 }
 
 static VOID
 OvsDumpNBLContext(PNET_BUFFER_LIST nbl)
 {
+#if OVS_DBG_DEFAULT >= OVS_DBG_LOUD
     PNET_BUFFER_LIST_CONTEXT ctx = nbl->Context;
     if (ctx == NULL) {
         OVS_LOG_INFO("No Net Buffer List context");
@@ -300,6 +304,9 @@ OvsDumpNBLContext(PNET_BUFFER_LIST nbl)
                      nbl, ctx, ctx->Size, ctx->Offset);
         ctx = ctx->Next;
     }
+#else
+    UNREFERENCED_PARAMETER(nbl);
+#endif
 }
 
 
@@ -337,6 +344,7 @@ OvsDumpNetBuffer(PNET_BUFFER nb)
 static VOID
 OvsDumpNetBufferList(PNET_BUFFER_LIST nbl)
 {
+#if OVS_DBG_DEFAULT >= OVS_DBG_LOUD
     PNET_BUFFER nb;
     OVS_LOG_INFO("NBL: %p, parent: %p, SrcHandle: %p, ChildCount:%d "
                  "poolHandle: %p",
@@ -349,6 +357,9 @@ OvsDumpNetBufferList(PNET_BUFFER_LIST nbl)
         OvsDumpNetBuffer(nb);
         nb = NET_BUFFER_NEXT_NB(nb);
     }
+#else
+    UNREFERENCED_PARAMETER(nbl);
+#endif
 }
 
 /*
-- 
2.9.3.windows.1

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to