From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id 820D42661 for ; Thu, 12 Oct 2017 03:51:20 +0200 (CEST) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga105.fm.intel.com with ESMTP; 11 Oct 2017 18:51:19 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.43,363,1503385200"; d="scan'208";a="909113327" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.241.225.21]) ([10.241.225.21]) by FMSMGA003.fm.intel.com with ESMTP; 11 Oct 2017 18:51:19 -0700 From: Ferruh Yigit To: Tomasz Duszynski , dev@dpdk.org Cc: mw@semihalf.com, dima@marvell.com, nsamsono@marvell.com, Jianbo.liu@linaro.org References: <1507031500-11473-1-git-send-email-tdu@semihalf.com> <1507561244-20115-1-git-send-email-tdu@semihalf.com> <1186b189-bb4f-0fb2-602d-6003b9371ea6@intel.com> Message-ID: <60720e51-6dd5-b7dc-5de1-a1e6b0a681c6@intel.com> Date: Thu, 12 Oct 2017 02:51:19 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <1186b189-bb4f-0fb2-602d-6003b9371ea6@intel.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v4 00/16] add net mrvl pmd driver X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 12 Oct 2017 01:51:20 -0000 On 10/9/2017 9:59 PM, Ferruh Yigit wrote: > On 10/9/2017 4:00 PM, Tomasz Duszynski wrote: >> Hello, >> >> This patch series introduces the net driver for Marvell Armada 7k/8k >> SoCs along with documentation. >> >> Below you can find the list of features which net pmd supports: >> * Speed capabilities >> * Link status >> * MTU update >> * Jumbo frame >> * Promiscuous mode >> * Allmulticast mode >> * Unicast MAC filter >> * Multicast MAC filter >> * RSS hash >> * VLAN filter >> * CRC offload >> * L3 checksum offload >> * L4 checksum offload >> * Packet type parsing >> * Basic stats >> * QoS >> >> Changes since v3: >> * Split driver into skeleton, rx/tx, features, documentation parts >> * Added speed capabilities flags. >> * Added missing rx offload flags: VLAN/JUMBOFRAME >> * Updated release notes. >> * Updated documentation. >> >> Changes since v2: >> * Removed LINE_SPACING, MULTILINE_DEREFERENCE and SPLIT_STRING >> checkpatch warnings. >> * Removed unnecessary forward declarations. >> * Fixed whitespace warnings. >> >> Changes since v1: >> * Changed commit message to explain problem better. >> * Removed bunch of checkpatch warnings about unnecessary parentheses. >> >> Tomasz Duszynski (4): >> app: link the whole rte_cfgfile library >> net/mrvl: add mrvl net pmd driver skeleton >> net/mrvl: add rx/tx support >> net/mrvl: add link update >> net/mrvl: add link speed capabilities >> net/mrvl: add support for updating mtu >> net/mrvl: add jumbo frame support >> net/mrvl: add support for promiscuous and allmulticast modes >> net/mrvl: add support for mac filtering >> net/mrvl: add rss hashing support >> net/mrvl: add support for vlan filtering >> net/mrvl: add crc, l3 and l4 offloads support >> net/mrvl: add packet type parsing support. >> net/mrvl: add basic stats support >> maintainers: add maintainers for the mrvl net pmd >> doc: add mrvl net pmd documentation > > Series applied to dpdk-next-net/master, thanks. > > (I can't compile because of missing musdk library, but since PMD is > disabled by default this is not so bad, lets get this for rc1 and for > crypto dependency, later I can have my environment set and test) Did able to compile but I have questions :) 1- Used the "arm64-armv8a-linuxapp-gcc" config, can you please confirm. Does it make sense to document this? 2- I used different toolchain than documented in musdk, which uses a marvel one. Can you please confirm any aarch64-linux-gnu-gcc is OK? 3- Used following command: CROSS=/aarch64-linux-gnu- make EXTRA_CFLAGS="-I.../musdk-marvell/src/include -L.../musdk-marvell/src/.libs/" Do you also need to use EXTRA_CFLAGS? Otherwise I can't compile. If you also use it please document it, if not please share with us how to do? 4- musk generated a static library, can you please confirm there is a way to generate a shared musdk library as well? 5- Still not tested building kernel modules, and building musdk with ./configure options, I will do later. Thanks, ferruh