[dpdk-dev] [PATCH 1/7] net/qede: reduce noise in debug logs

Ferruh Yigit ferruh.yigit at intel.com
Thu Dec 8 17:48:32 CET 2016


On 12/3/2016 2:43 AM, Harish Patil wrote:
> From: Rasesh Mody <Rasesh.Mody at cavium.com>
> 
> Replace CONFIG_RTE_LIBRTE_QEDE_DEBUG_DRIVER with
> CONFIG_RTE_LIBRTE_QEDE_DEBUG_VAL which is a 32-bit bitmapped value
> where each bit represent a particular submodule to debug. Also move
> notice messages under CONFIG_RTE_LIBRTE_QEDE_DEBUG_INFO.
> 
> Signed-off-by: Harish Patil <harish.patil at qlogic.com>
> Signed-off-by: Rasesh Mody <Rasesh.Mody at cavium.com>
> ---

Is 32bit supported by driver?

If so it is throwing a compilation error for it [1], if not can you
please document it?

[1]
In file included from .../drivers/net/qede/base/ecore.h:35:0,
                 from .../drivers/net/qede/qede_ethdev.h:22,
                 from .../drivers/net/qede/qede_ethdev.c:9:
.../drivers/net/qede/qede_ethdev.c: In function ‘qede_rss_hash_update’:
.../drivers/net/qede/base/../qede_logs.h:33:3: error: format ‘%lx’
expects argument of type ‘long unsigned int’, but argument 7 has type
‘uint64_t {aka long long unsigned int}’ [-Werror=format=]
   "[%s:%d(%s)]" fmt, \
   ^
.../drivers/net/qede/qede_ethdev.c:1472:2: note: in expansion of macro
‘DP_INFO’
  DP_INFO(edev, "RSS hf = 0x%lx len = %u key = %p\n", hf, len, key);
  ^~~~~~~
cc1: all warnings being treated as errors


<...>


More information about the dev mailing list