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 4E256A00C2; Wed, 22 Apr 2020 09:22:58 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 1DBD71C2A7; Wed, 22 Apr 2020 09:22:58 +0200 (CEST) Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by dpdk.org (Postfix) with ESMTP id D59BB1C29A; Wed, 22 Apr 2020 09:22:55 +0200 (CEST) IronPort-SDR: zkHuf2QjUb0vHJs+6gJgucrPEz5T8Mt85mLVfyijkqMCBJe3cibGu02lMcdjwqjoNk7vKD3Ca/ gf5LffB7eNwg== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 22 Apr 2020 00:22:55 -0700 IronPort-SDR: NSLi6jBN+ai+h2MCJA+6eRQYkBa/i+1M8SMjJBHdqoc5kGswFVVwG28cPlVp48J3+qzoQGREVb kpCKU5cl07+Q== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.72,412,1580803200"; d="scan'208";a="245904391" Received: from yexl-server.sh.intel.com (HELO localhost) ([10.67.117.17]) by fmsmga007.fm.intel.com with ESMTP; 22 Apr 2020 00:22:53 -0700 Date: Wed, 22 Apr 2020 15:18:31 +0800 From: Ye Xiaolong To: Shougang Wang Cc: dev@dpdk.org, Wenzhuo Lu , Qiming Yang , stable@dpdk.org Message-ID: <20200422071831.GA105307@intel.com> References: <20200413013839.71374-1-shougangx.wang@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200413013839.71374-1-shougangx.wang@intel.com> User-Agent: Mutt/1.9.4 (2018-02-28) Subject: Re: [dpdk-dev] [PATCH] net/ixgbe: fix link status after port reset 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" On 04/13, Shougang Wang wrote: >It's a normal behavior to change the link status to up after >resetting the port. So it is unnecessary to set link down before >starting port, and changing the link state(link up/down) frequently >will cause link speed unstable. > >Fixes: c3f2fbff78cf ("net/ixgbe: fix link status") >Cc: stable@dpdk.org > >Signed-off-by: Shougang Wang >--- > drivers/net/ixgbe/ixgbe_ethdev.c | 3 --- > 1 file changed, 3 deletions(-) > >diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c >index 23b3f5b0c..206358b85 100644 >--- a/drivers/net/ixgbe/ixgbe_ethdev.c >+++ b/drivers/net/ixgbe/ixgbe_ethdev.c >@@ -1196,7 +1196,6 @@ eth_ixgbe_dev_init(struct rte_eth_dev *eth_dev, void *init_params __rte_unused) > diag = ixgbe_bypass_init_hw(hw); > #else > diag = ixgbe_init_hw(hw); >- hw->mac.autotry_restart = false; > #endif /* RTE_LIBRTE_IXGBE_BYPASS */ > > /* >@@ -1307,8 +1306,6 @@ eth_ixgbe_dev_init(struct rte_eth_dev *eth_dev, void *init_params __rte_unused) > /* enable support intr */ > ixgbe_enable_intr(eth_dev); > >- ixgbe_dev_set_link_down(eth_dev); >- > /* initialize filter info */ > memset(filter_info, 0, > sizeof(struct ixgbe_filter_info)); >-- >2.17.1 > Applied to dpdk-next-net-intel, Thanks.