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. [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