[dpdk-dev] [PATCH 00/14] net/qede/base: update PMD version to 2.8.0.1

Mody, Rasesh Rasesh.Mody at cavium.com
Mon Apr 9 06:49:34 CEST 2018


Hi Ferruh,
> From: Ferruh Yigit [mailto:ferruh.yigit at intel.com]
> Sent: Friday, April 06, 2018 2:03 AM
> To: Mody, Rasesh <Rasesh.Mody at cavium.com>; dev at dpdk.org
> Cc: Dept-Eng DPDK Dev <Dept-EngDPDKDev at 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



More information about the dev mailing list