| Message ID | 1421811523-18737-1-git-send-email-changchun.ouyang@intel.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 3B0CA5A31; Wed, 21 Jan 2015 04:38:54 +0100 (CET) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 69D535A15 for <dev@dpdk.org>; Wed, 21 Jan 2015 04:38:51 +0100 (CET) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP; 20 Jan 2015 19:38:50 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.09,439,1418112000"; d="scan'208";a="665007039" Received: from shvmail01.sh.intel.com ([10.239.29.42]) by fmsmga002.fm.intel.com with ESMTP; 20 Jan 2015 19:38:49 -0800 Received: from shecgisg004.sh.intel.com (shecgisg004.sh.intel.com [10.239.29.89]) by shvmail01.sh.intel.com with ESMTP id t0L3cl9e011860; Wed, 21 Jan 2015 11:38:47 +0800 Received: from shecgisg004.sh.intel.com (localhost [127.0.0.1]) by shecgisg004.sh.intel.com (8.13.6/8.13.6/SuSE Linux 0.8) with ESMTP id t0L3cjq9018918; Wed, 21 Jan 2015 11:38:47 +0800 Received: (from couyang@localhost) by shecgisg004.sh.intel.com (8.13.6/8.13.6/Submit) id t0L3cj3O018914; Wed, 21 Jan 2015 11:38:45 +0800 From: Ouyang Changchun <changchun.ouyang@intel.com> To: dev@dpdk.org Date: Wed, 21 Jan 2015 11:38:43 +0800 Message-Id: <1421811523-18737-1-git-send-email-changchun.ouyang@intel.com> X-Mailer: git-send-email 1.7.12.2 Subject: [dpdk-dev] [PATCH] ixgbe: Fix an unnecessary check in vf rss X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Commit Message
Ouyang Changchun
Jan. 21, 2015, 3:38 a.m. UTC
To follow up the comments from Wodkowski, PawelX, remove this unnecessary check,
as check_mq_mode has already check the queue number in device configure stage,
if the queue number of vf is not correct, it will return error code and exit,
so it doesn't need check again here in device start stage(note: pf_host_configure
is called in device start stage).
This fixes commit 42d2f78abcb77ecb769be4149df550308169ef0f
Signed-off-by: Changchun Ouyang <changchun.ouyang@intel.com>
---
lib/librte_pmd_ixgbe/ixgbe_pf.c | 15 ---------------
1 file changed, 15 deletions(-)
Comments
> To follow up the comments from Wodkowski, PawelX, remove this unnecessary check, > as check_mq_mode has already check the queue number in device configure stage, > if the queue number of vf is not correct, it will return error code and exit, > so it doesn't need check again here in device start stage(note: pf_host_configure > is called in device start stage). > > This fixes commit 42d2f78abcb77ecb769be4149df550308169ef0f > > Signed-off-by: Changchun Ouyang <changchun.ouyang@intel.com> Suggested-by: Pawel Wodkowski <pawelx.wodkowski@intel.com> Fixes: 42d2f78abcb77 ("configure VF RSS") Applied Changchun, as you are working on ixgbe, maybe you would like to review some ixgbe patches from others? Thanks
Hi Thomas, > -----Original Message----- > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > Sent: Tuesday, January 27, 2015 8:13 PM > To: Ouyang, Changchun > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] ixgbe: Fix an unnecessary check in vf rss > > > To follow up the comments from Wodkowski, PawelX, remove this > > unnecessary check, as check_mq_mode has already check the queue > number > > in device configure stage, if the queue number of vf is not correct, > > it will return error code and exit, so it doesn't need check again > > here in device start stage(note: pf_host_configure is called in device start > stage). > > > > This fixes commit 42d2f78abcb77ecb769be4149df550308169ef0f > > > > Signed-off-by: Changchun Ouyang <changchun.ouyang@intel.com> > > Suggested-by: Pawel Wodkowski <pawelx.wodkowski@intel.com> > Fixes: 42d2f78abcb77 ("configure VF RSS") > > Applied > Thanks very much for the applying! > Changchun, as you are working on ixgbe, maybe you would like to review > some ixgbe patches from others? > No problem, I will try to do it when my bandwidth allows me to do it, :-) Thanks Changchun
> -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ouyang, Changchun > Sent: Wednesday, January 28, 2015 2:35 AM > To: Thomas Monjalon > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] ixgbe: Fix an unnecessary check in vf rss > > Hi Thomas, > > > -----Original Message----- > > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > Sent: Tuesday, January 27, 2015 8:13 PM > > To: Ouyang, Changchun > > Cc: dev@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH] ixgbe: Fix an unnecessary check in vf rss > > > > > To follow up the comments from Wodkowski, PawelX, remove this > > > unnecessary check, as check_mq_mode has already check the queue > > number > > > in device configure stage, if the queue number of vf is not correct, > > > it will return error code and exit, so it doesn't need check again > > > here in device start stage(note: pf_host_configure is called in device start > > stage). > > > > > > This fixes commit 42d2f78abcb77ecb769be4149df550308169ef0f > > > > > > Signed-off-by: Changchun Ouyang <changchun.ouyang@intel.com> > > > > Suggested-by: Pawel Wodkowski <pawelx.wodkowski@intel.com> > > Fixes: 42d2f78abcb77 ("configure VF RSS") > > > > Applied > > > > Thanks very much for the applying! > > > Changchun, as you are working on ixgbe, maybe you would like to review > > some ixgbe patches from others? > > > > No problem, I will try to do it when my bandwidth allows me to do it, :-) > Thanks > Changchun Actually I was suggesting exactly opposite direction. Main issue is that the sriov field in rte_eth_dev_data is only used by igb and ixgbe drivers. In addition rte_eth_dev_check_mq_mode() is specialized for ixgbe driver. I am thinking about moving sriov from rte_eth_dev_data to driver's private structure or at least move rte_eth_dev_check_mq_mode() to struct eth_dev_ops as optional driver configuration step. What do you think about both steps? Pawel
Hi PawelX > -----Original Message----- > From: Wodkowski, PawelX > Sent: Friday, January 30, 2015 12:14 AM > To: Ouyang, Changchun; Thomas Monjalon; Richardson, Bruce > Cc: dev@dpdk.org > Subject: RE: [dpdk-dev] [PATCH] ixgbe: Fix an unnecessary check in vf rss > > > -----Original Message----- > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ouyang, > Changchun > > Sent: Wednesday, January 28, 2015 2:35 AM > > To: Thomas Monjalon > > Cc: dev@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH] ixgbe: Fix an unnecessary check in vf > > rss > > > > Hi Thomas, > > > > > -----Original Message----- > > > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > > Sent: Tuesday, January 27, 2015 8:13 PM > > > To: Ouyang, Changchun > > > Cc: dev@dpdk.org > > > Subject: Re: [dpdk-dev] [PATCH] ixgbe: Fix an unnecessary check in > > > vf rss > > > > > > > To follow up the comments from Wodkowski, PawelX, remove this > > > > unnecessary check, as check_mq_mode has already check the queue > > > number > > > > in device configure stage, if the queue number of vf is not > > > > correct, it will return error code and exit, so it doesn't need > > > > check again here in device start stage(note: pf_host_configure is > > > > called in device start > > > stage). > > > > > > > > This fixes commit 42d2f78abcb77ecb769be4149df550308169ef0f > > > > > > > > Signed-off-by: Changchun Ouyang <changchun.ouyang@intel.com> > > > > > > Suggested-by: Pawel Wodkowski <pawelx.wodkowski@intel.com> > > > Fixes: 42d2f78abcb77 ("configure VF RSS") > > > > > > Applied > > > > > > > Thanks very much for the applying! > > > > > Changchun, as you are working on ixgbe, maybe you would like to > > > review some ixgbe patches from others? > > > > > > > No problem, I will try to do it when my bandwidth allows me to do it, > > :-) Thanks Changchun > > Actually I was suggesting exactly opposite direction. Main issue is that the > sriov field in rte_eth_dev_data is only used by igb and ixgbe drivers. In > addition > rte_eth_dev_check_mq_mode() is specialized for ixgbe driver. > > I am thinking about moving sriov from rte_eth_dev_data to driver's private > structure or at least move rte_eth_dev_check_mq_mode() to struct > eth_dev_ops as optional driver configuration step. > > What do you think about both steps? Good opinion! I prefer to move rte_eth_dev_check_mq_mode to eth_dev_ops as optional driver configure, The reason is that in future other eth type may also need such kind of check or even refine some queue number values by their own way, I can help review your patch after you send out. Thanks for your enhancing that. Changchun
> -----Original Message----- > From: Ouyang, Changchun > Sent: Friday, January 30, 2015 2:20 AM > To: Wodkowski, PawelX; Thomas Monjalon; Richardson, Bruce > Cc: dev@dpdk.org; Ouyang, Changchun > Subject: RE: [dpdk-dev] [PATCH] ixgbe: Fix an unnecessary check in vf rss > > Hi PawelX > > > -----Original Message----- > > From: Wodkowski, PawelX > > Sent: Friday, January 30, 2015 12:14 AM > > To: Ouyang, Changchun; Thomas Monjalon; Richardson, Bruce > > Cc: dev@dpdk.org > > Subject: RE: [dpdk-dev] [PATCH] ixgbe: Fix an unnecessary check in vf rss > > > > > -----Original Message----- > > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ouyang, > > Changchun > > > Sent: Wednesday, January 28, 2015 2:35 AM > > > To: Thomas Monjalon > > > Cc: dev@dpdk.org > > > Subject: Re: [dpdk-dev] [PATCH] ixgbe: Fix an unnecessary check in vf > > > rss > > > > > > Hi Thomas, > > > > > > > -----Original Message----- > > > > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > > > Sent: Tuesday, January 27, 2015 8:13 PM > > > > To: Ouyang, Changchun > > > > Cc: dev@dpdk.org > > > > Subject: Re: [dpdk-dev] [PATCH] ixgbe: Fix an unnecessary check in > > > > vf rss > > > > > > > > > To follow up the comments from Wodkowski, PawelX, remove this > > > > > unnecessary check, as check_mq_mode has already check the queue > > > > number > > > > > in device configure stage, if the queue number of vf is not > > > > > correct, it will return error code and exit, so it doesn't need > > > > > check again here in device start stage(note: pf_host_configure is > > > > > called in device start > > > > stage). > > > > > > > > > > This fixes commit 42d2f78abcb77ecb769be4149df550308169ef0f > > > > > > > > > > Signed-off-by: Changchun Ouyang <changchun.ouyang@intel.com> > > > > > > > > Suggested-by: Pawel Wodkowski <pawelx.wodkowski@intel.com> > > > > Fixes: 42d2f78abcb77 ("configure VF RSS") > > > > > > > > Applied > > > > > > > > > > Thanks very much for the applying! > > > > > > > Changchun, as you are working on ixgbe, maybe you would like to > > > > review some ixgbe patches from others? > > > > > > > > > > No problem, I will try to do it when my bandwidth allows me to do it, > > > :-) Thanks Changchun > > > > Actually I was suggesting exactly opposite direction. Main issue is that the > > sriov field in rte_eth_dev_data is only used by igb and ixgbe drivers. In > > addition > > rte_eth_dev_check_mq_mode() is specialized for ixgbe driver. > > > > I am thinking about moving sriov from rte_eth_dev_data to driver's private > > structure or at least move rte_eth_dev_check_mq_mode() to struct > > eth_dev_ops as optional driver configuration step. > > > > What do you think about both steps? > > Good opinion! > I prefer to move rte_eth_dev_check_mq_mode to eth_dev_ops as optional > driver configure, > The reason is that in future other eth type may also need such kind of check or > even refine some queue number values by their own way, > I can help review your patch after you send out. > Thanks for your enhancing that. > What about about moving sriov from rte_eth_dev_data to driver's private structure? Pawel
diff --git a/lib/librte_pmd_ixgbe/ixgbe_pf.c b/lib/librte_pmd_ixgbe/ixgbe_pf.c index 93f6e43..dbda9b5 100644 --- a/lib/librte_pmd_ixgbe/ixgbe_pf.c +++ b/lib/librte_pmd_ixgbe/ixgbe_pf.c @@ -187,21 +187,6 @@ int ixgbe_pf_host_configure(struct rte_eth_dev *eth_dev) IXGBE_WRITE_REG(hw, IXGBE_MPSAR_LO(hw->mac.num_rar_entries), 0); IXGBE_WRITE_REG(hw, IXGBE_MPSAR_HI(hw->mac.num_rar_entries), 0); - /* - * VF RSS can support at most 4 queues for each VF, even if - * 8 queues are available for each VF, it need refine to 4 - * queues here due to this limitation, otherwise no queue - * will receive any packet even RSS is enabled. - */ - if (eth_dev->data->dev_conf.rxmode.mq_mode == ETH_MQ_RX_VMDQ_RSS) { - if (RTE_ETH_DEV_SRIOV(eth_dev).nb_q_per_pool == 8) { - RTE_ETH_DEV_SRIOV(eth_dev).active = ETH_32_POOLS; - RTE_ETH_DEV_SRIOV(eth_dev).nb_q_per_pool = 4; - RTE_ETH_DEV_SRIOV(eth_dev).def_pool_q_idx = - dev_num_vf(eth_dev) * 4; - } - } - /* set VMDq map to default PF pool */ hw->mac.ops.set_vmdq(hw, 0, RTE_ETH_DEV_SRIOV(eth_dev).def_vmdq_idx);