From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH 00/14] net/qede/base: update PMD version to 2.8.0.1 Date: Fri, 6 Apr 2018 10:03:20 +0100 Message-ID: <0a8a14aa-b9f5-7e60-8d8a-86a6ab20c888@intel.com> References: <1522561624-15817-1-git-send-email-rasesh.mody@cavium.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: Dept-EngDPDKDev@cavium.com To: Rasesh Mody , dev@dpdk.org Return-path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id C2E061CDEB for ; Fri, 6 Apr 2018 11:03:24 +0200 (CEST) In-Reply-To: <1522561624-15817-1-git-send-email-rasesh.mody@cavium.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 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