Hi Ferruh,
> From: Ferruh Yigit [mailto:ferruh.yi...@intel.com]
> Sent: Friday, April 06, 2018 2:03 AM
> To: Mody, Rasesh <rasesh.m...@cavium.com>; dev@dpdk.org
> Cc: Dept-Eng DPDK Dev <dept-engdpdk...@cavium.com>
> Subject: Re: [PATCH 00/14] net/qede/base: update PMD version to 2.8.0.1
>
> On 4/1/2018 6:46 AM, Rasesh Mody wrote:
> > Hi,
> >
> > This patch set updates QEDE base driver to use FW version 8.33.12.0.
> > It contains some base driver enhancements and fixes. The PMD version
> > is updated to 2.8.0.1.
> >
> > Please apply.
> >
> > Thanks!
> > -Rasesh
> >
> > Rasesh Mody (14):
> > net/qede/base: use path ID for HW init
> > net/qede/base: protect DMAE transactions
> > net/qede/base: add DMAE sanity check
> > net/qede/base: upgrade FW to 8.33.12.0
> > net/qede/base: symantic changes
> > net/qede/base: add new chain API
> > net/qede/base: allow changing VF MAC address
> > net/qede/base: add MFW support for driver load timeout
> > net/qede/base: refine error handling
> > net/qede/base: add stats counter for link state
> > net/qede/base: add APIs for xcvr
> > net/qede/base: fix to support OVLAN mode
> > net/qede/base: add packet pacing support
> > net/qede: update PMD version to 2.8.0.1
>
> Hi Rasesh,
>
> Getting build errors for 32bit [1], I didn't dig which commit cause them.
These 32bit build errors are due to couple of log messages from patch 2 and 3.
Have sent v2 series addressing the same. Please apply.
Thanks!
-Rasesh
>
>
> [1]
> In file included from .../dpdk/drivers/net/qede/base/ecore.h:50:0,
> from .../dpdk/drivers/net/qede/base/ecore_hw.c:12:
> .../dpdk/drivers/net/qede/base/ecore_hw.c: In function
> ‘ecore_dmae_execute_command’:
> .../dpdk/drivers/net/qede/base/../qede_logs.h:48:5: error: format ‘%lx’
> expects argument of type ‘long unsigned int’, but argument 7 has type ‘u64
> {aka long long unsigned int}’ [-Werror=format=]
> "[%s:%d(%s)]" fmt, \
> ^
> .../dpdk/drivers/net/qede/base/ecore_hw.c:789:3: note: in expansion of
> macro ‘DP_VERBOSE’
> DP_VERBOSE(p_hwfn, ECORE_MSG_HW,
> ^~~~~~~~~~
> .../dpdk/drivers/net/qede/base/ecore_hw.c:790:69: note: format string is
> defined here
> "No buffers allocated. Avoid DMAE transaction [{src: addr 0x%lx, type
> %d}, {dst: addr 0x%lx, type %d}, size %d].\n",
> ~~^
> %llx In
> file included from
> .../dpdk/drivers/net/qede/base/ecore.h:50:0,
> from .../dpdk/drivers/net/qede/base/ecore_hw.c:12:
> .../dpdk/drivers/net/qede/base/../qede_logs.h:48:5: error: format ‘%lx’
> expects argument of type ‘long unsigned int’, but argument 9 has type ‘u64
> {aka long long unsigned int}’ [-Werror=format=]
> "[%s:%d(%s)]" fmt, \
> ^
> .../dpdk/drivers/net/qede/base/ecore_hw.c:789:3: note: in expansion of
> macro ‘DP_VERBOSE’
> DP_VERBOSE(p_hwfn, ECORE_MSG_HW,
> ^~~~~~~~~~
> .../dpdk/drivers/net/qede/base/ecore_hw.c:790:97: note: format string is
> defined here
> "No buffers allocated. Avoid DMAE transaction [{src: addr 0x%lx, type
> %d}, {dst: addr 0x%lx, type %d}, size %d].\n",
>
> ~~^
>
> %llx
> In file included from .../dpdk/drivers/net/qede/base/ecore.h:50:0,
> from .../dpdk/drivers/net/qede/base/ecore_hw.c:12:
> .../dpdk/drivers/net/qede/base/ecore_hw.c: In function
> ‘ecore_dmae_sanity’:
> .../dpdk/drivers/net/qede/base/../qede_logs.h:48:5: error: format ‘%lx’
> expects argument of type ‘long unsigned int’, but argument 8 has type ‘long
> long unsigned int’ [-Werror=format=]
> "[%s:%d(%s)]" fmt, \
> ^
> .../dpdk/drivers/net/qede/base/ecore_hw.c:995:2: note: in expansion of
> macro ‘DP_VERBOSE’
> DP_VERBOSE(p_hwfn, ECORE_MSG_SP,
> ^~~~~~~~~~
> .../dpdk/drivers/net/qede/base/ecore_hw.c:996:44: note: format string is
> defined here
> "DMAE sanity [%s]: src_addr={phys 0x%lx, virt %p}, dst_addr={phys
> 0x%lx, virt %p}, size 0x%x\n",
> ~~^
> %llx In file included from
> .../dpdk/drivers/net/qede/base/ecore.h:50:0,
> from .../dpdk/drivers/net/qede/base/ecore_hw.c:12:
> .../dpdk/drivers/net/qede/base/../qede_logs.h:48:5: error: format ‘%lx’
> expects argument of type ‘long unsigned int’, but argument 10 has type ‘long
> long unsigned int’ [-Werror=format=]
> "[%s:%d(%s)]" fmt, \
> ^
> .../dpdk/drivers/net/qede/base/ecore_hw.c:995:2: note: in expansion of
> macro ‘DP_VERBOSE’
> DP_VERBOSE(p_hwfn, ECORE_MSG_SP,
> ^~~~~~~~~~
> .../dpdk/drivers/net/qede/base/ecore_hw.c:996:76: note: format string is
> defined here
> "DMAE sanity [%s]: src_addr={phys 0x%lx, virt %p}, dst_addr={phys
> 0x%lx, virt %p}, size 0x%x\n",
> ~~^
>
> %llx In file included from
> .../dpdk/drivers/net/qede/base/ecore.h:50:0,
> from .../dpdk/drivers/net/qede/base/ecore_hw.c:12:
> .../dpdk/drivers/net/qede/base/../qede_logs.h:25:4: error: format ‘%lx’
> expects argument of type ‘long unsigned int’, but argument 7 has type ‘long
> long unsigned int’ [-Werror=format=]
> "[QEDE PMD: (%s)]%s:" fmt, \
> ^
> .../dpdk/drivers/net/qede/base/ecore_hw.c:1018:4: note: in expansion of
> macro ‘DP_NOTICE’
> DP_NOTICE(p_hwfn, false,
> ^~~~~~~~~
> .../dpdk/drivers/net/qede/base/ecore_hw.c:1019:41: note: format string is
> defined here
> "DMAE sanity [%s]: addr={phys 0x%lx, virt %p}, read_val 0x%08x,
> expected_val 0x%08x\n",
> ~~^
> %llx In file included from
> .../dpdk/drivers/net/qede/base/ecore.h:50:0,
> from .../dpdk/drivers/net/qede/base/ecore_hw.c:12:
> .../dpdk/drivers/net/qede/base/../qede_logs.h:31:4: error: format ‘%lx’
> expects argument of type ‘long unsigned int’, but argument 7 has type ‘long
> long unsigned int’ [-Werror=format=]
> "[QEDE PMD: (%s)]%s:" fmt, \
> ^
> .../dpdk/drivers/net/qede/base/ecore_hw.c:1018:4: note: in expansion of
> macro ‘DP_NOTICE’
> DP_NOTICE(p_hwfn, false,
> ^~~~~~~~~
> .../dpdk/drivers/net/qede/base/ecore_hw.c:1019:41: note: format string is
> defined here
> "DMAE sanity [%s]: addr={phys 0x%lx, virt %p}, read_val 0x%08x,
> expected_val 0x%08x\n",
> ~~^
> %llx
> cc1: all warnings being treated as errors
