From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH 00/28] bnxt: new features, bug fixes and more Date: Wed, 29 Mar 2017 16:45:22 +0100 Message-ID: References: <20170328034903.41482-1-ajit.khaparde@broadcom.com> <6a0b1cac-97af-1ea5-a74b-9f930d3676ff@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit To: Ajit Khaparde , dev@dpdk.org Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 20556FA3A for ; Wed, 29 Mar 2017 17:45:24 +0200 (CEST) In-Reply-To: <6a0b1cac-97af-1ea5-a74b-9f930d3676ff@intel.com> 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 3/29/2017 1:14 PM, Ferruh Yigit wrote: > On 3/28/2017 4:48 AM, Ajit Khaparde wrote: >> Apart from adding support for VF-PF/PF-VF communication, new dev_ops, >> few bug fixes and improve debug capabilities, this patchset updates >> the driver to the newer HWRM version 1.7.4 >> >> New dev_ops supported: >> set_mc_addr_list, xstats_get/reset, mac_addr_set, vlan_filter_set, >> vlan_strip_queue_set, vlan_offload_set, udp_tunnel_port_add/del >> >> bug fixes: >> use only long BDs in Tx path to avoid a potential bug in hardware, >> fix interrupt handler, set the VMDQ pool size correctly >> >> Derived from an autogenerated file, the last patch updates the >> HWRM version to 1.7.4 > > Hi Ajit, > > It looks like bnxt also started supporting PMD specific APIs in this > patchset. Perhaps we should investigate them if are there common ones > with other PMDs and if so move them to ethdev layer instead of PMD > specific API. > > > BTW, There are new PMD features in this patchset, and patchset sent two > days before integration deadline. > > The process is, new feature patches sent before proposal deadline, which > was March 3 for this release, and let patches reviewed and new versions > sent until integration deadline. If patches acked before integration > deadline, they are integrated. > > So this patchset is too late for this release, will be considered for > next release. For further usage, patchset gives build error for 32bit and shared library builds. > > If you have urgent fixes, please separate them into another patchset, > and we can work on them to get into this release, but still it is so > close to integration deadline, no promises. > > Thanks, > ferruh