From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rasesh Mody Subject: [PATCH 2/2] net/qede: fix default config option Date: Wed, 8 Nov 2017 22:52:20 -0800 Message-ID: <1510210340-5433-2-git-send-email-rasesh.mody@cavium.com> References: <1510210340-5433-1-git-send-email-rasesh.mody@cavium.com> Mime-Version: 1.0 Content-Type: text/plain Cc: Harish Patil , Dept-EngDPDKDev@cavium.com, Rasesh Mody To: dev@dpdk.org, thomas.monjalon@6wind.com, ferruh.yigit@intel.com Return-path: Received: from NAM01-SN1-obe.outbound.protection.outlook.com (mail-sn1nam01on0067.outbound.protection.outlook.com [104.47.32.67]) by dpdk.org (Postfix) with ESMTP id 6EC611B33E for ; Thu, 9 Nov 2017 07:52:35 +0100 (CET) In-Reply-To: <1510210340-5433-1-git-send-email-rasesh.mody@cavium.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" From: Harish Patil Restore the default configuration as in previous releases and add a debug msg. Fixes: f07aa795c92a ("net/qede: disable per-VF Tx switching feature") Signed-off-by: Harish Patil Signed-off-by: Rasesh Mody --- config/common_base | 2 +- drivers/net/qede/qede_ethdev.c | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/config/common_base b/config/common_base index 34f04a9..e74febe 100644 --- a/config/common_base +++ b/config/common_base @@ -415,7 +415,7 @@ CONFIG_RTE_LIBRTE_QEDE_DEBUG_INFO=n CONFIG_RTE_LIBRTE_QEDE_DEBUG_DRIVER=n CONFIG_RTE_LIBRTE_QEDE_DEBUG_TX=n CONFIG_RTE_LIBRTE_QEDE_DEBUG_RX=n -CONFIG_RTE_LIBRTE_QEDE_VF_TX_SWITCH=n +CONFIG_RTE_LIBRTE_QEDE_VF_TX_SWITCH=y #Provides abs path/name of the firmware file. #Empty string denotes driver will use default firmware CONFIG_RTE_LIBRTE_QEDE_FW="" diff --git a/drivers/net/qede/qede_ethdev.c b/drivers/net/qede/qede_ethdev.c index 8832145..6f5ba2a 100644 --- a/drivers/net/qede/qede_ethdev.c +++ b/drivers/net/qede/qede_ethdev.c @@ -457,6 +457,7 @@ int qede_activate_vport(struct rte_eth_dev *eth_dev, bool flg) if (IS_VF(edev)) { params.update_tx_switching_flg = 1; params.tx_switching_flg = !flg; + DP_INFO(edev, "VF tx-switching is disabled\n"); } #endif for_each_hwfn(edev, i) { @@ -469,8 +470,8 @@ int qede_activate_vport(struct rte_eth_dev *eth_dev, bool flg) break; } } - DP_INFO(edev, "vport %s VF tx-switch %s\n", flg ? "activated" : "deactivated", - params.tx_switching_flg ? "enabled" : "disabled"); + DP_INFO(edev, "vport is %s\n", flg ? "activated" : "deactivated"); + return rc; } -- 1.7.10.3