From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id D7D9A28EE for ; Mon, 22 May 2017 12:55:29 +0200 (CEST) Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 May 2017 03:55:28 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.38,377,1491289200"; d="scan'208";a="90232862" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.81]) ([10.237.220.81]) by orsmga004.jf.intel.com with ESMTP; 22 May 2017 03:55:27 -0700 To: Ajit Khaparde , dev@dpdk.org References: <20170518015813.7862-1-ajit.khaparde@broadcom.com> Cc: Thomas Monjalon From: Ferruh Yigit Message-ID: Date: Mon, 22 May 2017 11:55:26 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <20170518015813.7862-1-ajit.khaparde@broadcom.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH 00/23] bnxt patchset 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: Mon, 22 May 2017 10:55:30 -0000 On 5/18/2017 2:57 AM, Ajit Khaparde wrote: > This patchset amongst other changes adds support few more dev_ops, > updates HWRM to version 1.7.5, switches to polling stats from the > hardware, support for LRO etc.. > > bnxt: add various hwrm input/output structures > bnxt: code reorg to properly allocate resources in PF/VF modes > bnxt: add tunneling support > bnxt: support lack of huge pages > bnxt: add functions for tx_loopback, set_vf_mac and queues_drop_en > bnxt: add support for set VF QOS and MAC anti spoof > bnxt: add support for VLAN stripq, VLAN anti spoof and VLAN filtering for VFs > bnxt: add support to get and clear VF specific stats > bnxt: add code to determine the Rx status of VF > bnxt: add support to add a VF MAC address > bnxt: add support for xstats get/reset > bnxt: Add support for VLAN filter and strip dev_ops > bnxt: add code to configure a default VF VLAN > bnxt: add support for set_mc_addr_list and mac_addr_set > bnxt: add support for fw_version_get dev_op > bnxt: add support to set MTU > bnxt: add support for LRO > bnxt: add rxq_info_get and txq_info_get dev_ops > bnxt: add additonal HWRM debug info to error messages > bnxt: reorg the query stats code > bnxt: update to HWRM version 1.7.5 > bnxt: Add support to set VF rxmode > bnxt: add code to support vlan_pvid_set dev_op > 25 files changed, 10407 insertions(+), 3036 deletions(-) Hi Ajit, This is a big patch, and as far as I can see it does mainly three things: 1) Update base driver kind of updates (although bnxt doesn't have a base folder, there are files we can put into this category). 2) Add some new driver features. 3) Add PMD specific APIs mainly to control VF from PF. - Is it possible to group patches in the patchset as above, currently patches are all mixed and harder to understand? - for 3), Some of the PMD specific APIs are common with exiting ones in i40e and ixgbe, should we extract them into eth_dev_ops now? @Thomas, any comment? - For 2), can you please update features/bnxt.ini file in each patch that adds a feature, to enable that feature in the document. And finally a release doc update in separate patch can be good. - Some commits has function names updated in the patch subject, can you please update them to describe the feature added, function name detail can be in commit log. You can use ./devtools/check-git-log.sh to verify commit logs, there are some set of errors now. Thanks, ferruh