From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id D8591994F for ; Thu, 25 May 2017 11:51:19 +0200 (CEST) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga104.jf.intel.com with ESMTP; 25 May 2017 02:51:17 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.38,391,1491289200"; d="scan'208";a="91624245" Received: from yliu-dev.sh.intel.com ([10.239.67.162]) by orsmga002.jf.intel.com with ESMTP; 25 May 2017 02:50:56 -0700 From: Yuanhan Liu To: Harish Patil Cc: Yuanhan Liu , dpdk stable Date: Thu, 25 May 2017 17:48:20 +0800 Message-Id: <1495705809-21416-48-git-send-email-yuanhan.liu@linux.intel.com> X-Mailer: git-send-email 1.9.0 In-Reply-To: <1495705809-21416-1-git-send-email-yuanhan.liu@linux.intel.com> References: <1495705809-21416-1-git-send-email-yuanhan.liu@linux.intel.com> Subject: [dpdk-stable] patch 'net/qede: fix VF RSS configuration' has been queued to stable release 17.02.1 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 25 May 2017 09:51:20 -0000 Hi, FYI, your patch has been queued to stable release 17.02.1 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 05/28/17. So please shout if anyone has objections. Thanks. --yliu --- >>From 650039cd41fa57a8628a8e0088fbe70ef3879c22 Mon Sep 17 00:00:00 2001 From: Harish Patil Date: Fri, 24 Mar 2017 00:40:58 -0700 Subject: [PATCH] net/qede: fix VF RSS configuration [ upstream commit 8130abb3ec692be11b7456a3563b32f543a6955c ] The newer SR-IOV PF drivers expects RX/TX queues to be created before applying RSS configuration. This patch addresses this requirement by deferring RSS configuration till the queues are created. Even though this issue is only seen in SR-IOV context, the changes will be made applicable to PF also to keep the behavior consistent between VF/PF. Fixes: 7ab35bf6b97b ("net/qede: fix RSS") Signed-off-by: Harish Patil --- drivers/net/qede/qede_ethdev.c | 18 +++--------------- drivers/net/qede/qede_ethdev.h | 2 +- drivers/net/qede/qede_rxtx.c | 9 +++++++++ 3 files changed, 13 insertions(+), 16 deletions(-) diff --git a/drivers/net/qede/qede_ethdev.c b/drivers/net/qede/qede_ethdev.c index 987c451..49ebd9a 100644 --- a/drivers/net/qede/qede_ethdev.c +++ b/drivers/net/qede/qede_ethdev.c @@ -797,7 +797,7 @@ static void qede_prandom_bytes(uint32_t *buff) buff[i] = rand(); } -static int qede_config_rss(struct rte_eth_dev *eth_dev) +int qede_config_rss(struct rte_eth_dev *eth_dev) { struct qede_dev *qdev = QEDE_INIT_QDEV(eth_dev); struct ecore_dev *edev = QEDE_INIT_EDEV(qdev); @@ -903,20 +903,8 @@ static int qede_dev_configure(struct rte_eth_dev *eth_dev) if (rc != 0) return rc; - /* Do RSS configuration after vport-start */ - switch (rxmode->mq_mode) { - case ETH_MQ_RX_RSS: - rc = qede_config_rss(eth_dev); - if (rc != 0) { - qdev->ops->vport_stop(edev, 0); - qede_dealloc_fp_resc(eth_dev); - return -EINVAL; - } - break; - case ETH_MQ_RX_NONE: - DP_INFO(edev, "RSS is disabled\n"); - break; - default: + if (!(rxmode->mq_mode == ETH_MQ_RX_RSS || + rxmode->mq_mode == ETH_MQ_RX_NONE)) { DP_ERR(edev, "Unsupported RSS mode\n"); qdev->ops->vport_stop(edev, 0); qede_dealloc_fp_resc(eth_dev); diff --git a/drivers/net/qede/qede_ethdev.h b/drivers/net/qede/qede_ethdev.h index be54f31..90ce05a 100644 --- a/drivers/net/qede/qede_ethdev.h +++ b/drivers/net/qede/qede_ethdev.h @@ -209,7 +209,7 @@ static void qede_init_rss_caps(uint8_t *rss_caps, uint64_t hf); static inline uint32_t qede_rx_cqe_to_pkt_type(uint16_t flags); /* Non-static functions */ -void qede_init_rss_caps(uint8_t *rss_caps, uint64_t hf); +int qede_config_rss(struct rte_eth_dev *eth_dev); int qed_fill_eth_dev_info(struct ecore_dev *edev, struct qed_dev_eth_info *info); diff --git a/drivers/net/qede/qede_rxtx.c b/drivers/net/qede/qede_rxtx.c index 01ea9b4..329c8e4 100644 --- a/drivers/net/qede/qede_rxtx.c +++ b/drivers/net/qede/qede_rxtx.c @@ -1340,6 +1340,15 @@ int qede_dev_start(struct rte_eth_dev *eth_dev) return rc; } + /* Newer SR-IOV PF driver expects RX/TX queues to be started before + * enabling RSS. Hence RSS configuration is deferred upto this point. + * Also, we would like to retain similar behavior in PF case, so we + * don't do PF/VF specific check here. + */ + if (eth_dev->data->dev_conf.rxmode.mq_mode == ETH_MQ_RX_RSS) + if (qede_config_rss(eth_dev)) + return -1; + /* Bring-up the link */ qede_dev_set_link_state(eth_dev, true); -- 1.9.0