diff options
author | Anand Kumar <kumaranand@vmware.com> | 2018-02-27 20:59:40 -0800 |
---|---|---|
committer | Alin Gabriel Serdean <aserdean@ovn.org> | 2018-03-01 23:11:12 +0200 |
commit | 34825d5e83a2941d652ec96da6ceec8382eead24 (patch) | |
tree | f5426ee423dbf7394a83dd331aaf5a4a7c5036ec | |
parent | d9cea8f5f341a60f60cd2bf61d0122f3c6715ca1 (diff) | |
download | openvswitch-34825d5e83a2941d652ec96da6ceec8382eead24.tar.gz |
datapath-windows: On Debug builds, dump NBL info based on OVS_DBG_DEFAULT macro
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 <kumaranand@vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean@ovn.org>
Signed-off-by: Alin Gabriel Serdean <aserdean@ovn.org>
-rw-r--r-- | datapath-windows/ovsext/BufferMgmt.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/datapath-windows/ovsext/BufferMgmt.c b/datapath-windows/ovsext/BufferMgmt.c index 03470d787..448cd7652 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 } /* |