From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id D092BA0567; Fri, 13 Mar 2020 06:24:56 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 18F371BFA5; Fri, 13 Mar 2020 06:24:56 +0100 (CET) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id E9EF92BE3; Fri, 13 Mar 2020 06:24:53 +0100 (CET) X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 12 Mar 2020 22:24:52 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.70,547,1574150400"; d="scan'208";a="278093417" Received: from yexl-server.sh.intel.com (HELO localhost) ([10.67.117.17]) by fmsmga002.fm.intel.com with ESMTP; 12 Mar 2020 22:24:50 -0700 Date: Fri, 13 Mar 2020 13:22:13 +0800 From: Ye Xiaolong To: "Li, Xiaoyun" Cc: Zhike Wang , "dev@dpdk.org" , "Xing, Beilei" , "Zhang, Qi Z" , "stable@dpdk.org" Message-ID: <20200313052213.GB44839@intel.com> References: <1579137597-14076-1-git-send-email-wangzhike@jd.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.4 (2018-02-28) Subject: Re: [dpdk-dev] [dpdk-stable] [PATCH] net/i40e: fix X722 judgement when disable adminq operation 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi, Xiaoyun Thanks for the review, just want to double check, are you suggesting that this fix should just be applied to LTS? Latest code doesn't need it, right? Thanks, Xiaolong On 03/13, Li, Xiaoyun wrote: >Hi > >This issue is already fixed in 20.02 with base code update of the following commit. > > > >commit 37b091c75b13d2f26359be9b77adbc33c55a7581 > >Author: Xiaolong Ye > >Date: Mon Jan 13 10:39:31 2020 +0800 > > > > net/i40e/base: extend PHY access AQ command > > > > Currently FW use MDIO I/F number corresponded with current PF for PHY > > access. This code allow to specify used MDIO I/F number. > > > > Signed-off-by: Piotr Azarewicz > > Signed-off-by: Xiaolong Ye > > Acked-by: Qi Zhang > >Acked-by: Beilei Xing beilei.xing@intel.com > > > >But it makes sense to back port this to stable release so it can work with dpdk version <=19.11. > >Still some comments inline. > > > >> -----Original Message----- > >> From: stable [mailto:stable-bounces@dpdk.org] On Behalf Of Zhike Wang > >> Sent: Thursday, January 16, 2020 09:20 > >> To: dev@dpdk.org > >> Cc: Xing, Beilei ; Zhang, Qi Z ; > >> stable@dpdk.org; Zhike Wang > >> Subject: [dpdk-stable] [PATCH] net/i40e: fix X722 judgement when disable > >> adminq operation > >> > >> X722 SFPs have different flavors, eg I40E_DEV_ID_SFP_X722 > >> /I40E_DEV_ID_SFP_I_X722. So instead we use mac.type to judge whether it is > >> X722 or not. > >> > >Please be careful about your commit log. If you use mac.type == X722, then that's not just X722 SFP but all X722 devices. > >You can just say something like X722 doesn't support adminq operation so disable it. > > > >> Fixes: 9efa8d28b ("net/i40e: fix SFP X722 with FW4.16") > >Please follow the DPDK community code style. Fix line needs commit id of 12 bit. > >In this case, should be: Fixes: 9efa8d28b4da ("net/i40e: fix SFP X722 with FW4.16") > >And since this fix makes sense for <=19.11, please add this line: Cc: stable@dpdk.org > > > >> Signed-off-by: Zhike Wang > > >> --- > >> drivers/net/i40e/i40e_ethdev.c | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c > >> index 5999c96..85ccb76 100644 > >> --- a/drivers/net/i40e/i40e_ethdev.c > >> +++ b/drivers/net/i40e/i40e_ethdev.c > >> @@ -1443,7 +1443,7 @@ static inline void i40e_config_automask(struct > >> i40e_pf *pf) > >> return -EIO; > >> } > >> /* Firmware of SFP x722 does not support adminq option */ > >> - if (hw->device_id == I40E_DEV_ID_SFP_X722) > >> + if (hw->mac.type == I40E_MAC_X722) > >> hw->flags &= ~I40E_HW_FLAG_802_1AD_CAPABLE; > >> > >> PMD_INIT_LOG(INFO, "FW %d.%d API %d.%d NVM %02d.%02d.%02d > >> eetrack %04x", > >> -- > >> 1.8.3.1 > >>