From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id 77661A0096 for ; Fri, 15 Mar 2019 13:59:50 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 02E62324D; Fri, 15 Mar 2019 13:59:49 +0100 (CET) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id E09E52C28 for ; Fri, 15 Mar 2019 13:59:46 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 15 Mar 2019 05:59:45 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.58,482,1544515200"; d="scan'208";a="152631249" Received: from irsmsx107.ger.corp.intel.com ([163.33.3.99]) by fmsmga004.fm.intel.com with ESMTP; 15 Mar 2019 05:59:44 -0700 Received: from irsmsx105.ger.corp.intel.com ([169.254.7.210]) by IRSMSX107.ger.corp.intel.com ([169.254.10.15]) with mapi id 14.03.0415.000; Fri, 15 Mar 2019 12:59:43 +0000 From: "Ananyev, Konstantin" To: "Zhao1, Wei" , "Zhang, Qi Z" , "David Harton (dharton)" CC: "Lu, Wenzhuo" , "dev@dpdk.org" Thread-Topic: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf Thread-Index: AQHU1e5mMTKeI4TkL0ONdEJSd+GPAaYLKkiAgAAbE4CAAApZgIAAntoAgABFZgCAAEDuYIAABAIAgAA4mJA= Date: Fri, 15 Mar 2019 12:59:43 +0000 Message-ID: <2601191342CEEE43887BDE71AB977258013655BFC4@irsmsx105.ger.corp.intel.com> References: <20190306152355.1697-1-dharton@cisco.com> <20190308203430.31742-1-dharton@cisco.com> <039ED4275CED7440929022BC67E706115334CA1F@SHSMSX103.ccr.corp.intel.com> <039ED4275CED7440929022BC67E706115334CD63@SHSMSX103.ccr.corp.intel.com> <2601191342CEEE43887BDE71AB977258013655BDCD@irsmsx105.ger.corp.intel.com> In-Reply-To: Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiYzdlOGI0NzQtZjc4Yi00Y2U5LTk4MGEtM2Y1YjRiMDY4OWUzIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiMXJRdmJDR3hySlZiU0pseHhkVmFleFNUT094VlhGdnM5dFwvU0dOc3ZjRjFcL0k5dldqcjdrdDhtK0k5dlwvWHdPWSJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.400.15 dlp-reaction: no-action x-originating-ip: [163.33.239.182] Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf 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" Message-ID: <20190315125943.NVTscCbOBEXXIEGQbASauQJ-P-7RDn8WeRWCTh9t7G8@z> Hi Wei, > > > > > > > > > > > > > > Hi Qi, > > > > > > > > > > > > > > This patch was rejected with no feedback. > > > > > > > > > > > > > > Can you share why? > > > > > > > > > > > > Because as Zhao Wei explained in v1 thread, in > > > > > > ixgbe_vlan_offload_config , ETH_VLAN_FILTER_MASK and > > > > > > DEV_RX_OFFLOAD_VLAN_EXTEND are ignored This looks like > > > > VLAN_FILTER > > > > > AND > > > > > > VLAN_EXTEND offload is not supported and I didn't see you reply > > > > > > for that, so I > > > > > assume you agree with that. > > > > > > > > > > > > > > > > > > > > > > > > > > Without this patch users are not able to call the > > > > > > > rte_eth_dev_vlan_filter() API for ixgbevf based ports because > > > > > > > VLAN_FILTER cannot be enabled when calling > > rte_eth_dev_configure(). > > > > > > > > > > > > OK, seems vlan filter is able to be configured for ixgbevf, but > > > > > > I don't know how this will work because vlan filter table needs > > > > > > to be enabled first (ixgbe_vlan_hw_filter_enable) So at least > > > > > > when it work with a DPDK PF, vlan filter will not work, maybe > > > > > > kernel PF driver will enable it by default? Could you share mor= e > > insight? > > > > > > > > > > I can't answer that but ixgbevf_vlan_filter_set() has been > > > > > implemented for quite some time and it can no longer be called > > > > > since the new offload flags are being enforced now. > > > > > > > > > > I can share that > > > > > rte_eth_dev_vlan_filter()/ixgbevf_vlan_filter_set() > > > > > works properly with the propose change (VLAN traffic flows) where > > > > > it doesn't without it. > > > > > > > > Ok, if it is proved to work well with kernel PF, I think we should > > > > have this offload > > > > > > > > > > > > > > > > > And same question to VLAN_EXTEND > > > > > > > > > > I actually don't care about extend but simply changed it because > > > > > it seems to be in the same situation. I'm happy to leave it out > > > > > of the patch if that grants acceptance. > > > > > > > > Seems VLAN Extend is only supported by DPDK PF driver, I didn't see > > > > any related configuration in kernel driver. > > > > Properly we need to remove it. > > > > > > > > @Wei, would you help to confirm on that? > > > > > > > > > Yes, you are right, kernel pf do not has any code to set bit of > > IXGBE_DMATXCTL_GDV which enable double valn. > > > So, that means (kernel pf host + dpdk vf) can not enable double valn > > feature. It seems only dpdk pf host + vf can do that. > > > > Wonder can vf request pf what is supported? > > Konstantin >=20 > No such type of message defined in mailbox by now. My thought was try to execute command (set vlan) with some dummy value, and analyze response to decide do we provide such capability or not. Or is too much hassle? >=20 > > > > > > > > > > Maybe we can change code to > > > " > > > offloads |=3D DEV_RX_OFFLOAD_VLAN_FILTER; > > > > > > if (ixgbe_is_vf(dev) =3D=3D 0) > > > offloads |=3D DEV_RX_OFFLOAD_VLAN_EXTEND; " > > > But that means when in ENV (kernel pf host + dpdk vf), this is much > > depended on kernel pf default configuration. > > > > > > > > > > > > > > > > > Thanks > > > > Qi > > > > > > > > > > > > > > Thanks, > > > > > Dave > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Or, am I missing something? > > > > > > > > > > > > > > Thanks, > > > > > > > Dave > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > From: David Harton > > > > > > > > Sent: Friday, March 08, 2019 3:35 PM > > > > > > > > To: dev@dpdk.org > > > > > > > > Cc: wenzhuo.lu@intel.com; konstantin.ananyev@intel.com; > > > > > > > > qi.z.zhang@intel.com; David Harton (dharton) > > > > > > > > > > > > > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend > > > > > > > > for ixgbevf > > > > > > > > > > > > > > > > ixgbevf vlan strip and extend capabilities were removed whe= n > > > > > > > > migrating to the bit flags implementation. > > > > > > > > > > > > > > > > Restoring the capbility to enable these offloads at > > > > > > > > configuration > > > > > > time. > > > > > > > > > > > > > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offload= s > > > > > > > > API\") > > > > > > > > Signed-off-by: David Harton > > > > > > > > --- > > > > > > > > > > > > > > > > v2: removed unused function ixgbe_is_vf() > > > > > > > > > > > > > > > > drivers/net/ixgbe/ixgbe_rxtx.c | 23 ++--------------------= - > > > > > > > > 1 file changed, 2 insertions(+), 21 deletions(-) > > > > > > > > > > > > > > > > diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c > > > > > > > > b/drivers/net/ixgbe/ixgbe_rxtx.c index e92a70fb3..b1b83613e > > > > > > > > 100644 > > > > > > > > --- a/drivers/net/ixgbe/ixgbe_rxtx.c > > > > > > > > +++ b/drivers/net/ixgbe/ixgbe_rxtx.c > > > > > > > > @@ -2813,23 +2813,6 @@ ixgbe_reset_rx_queue(struct > > > > ixgbe_adapter > > > > > > > > *adapter, struct ixgbe_rx_queue *rxq) #endif } > > > > > > > > > > > > > > > > -static int > > > > > > > > -ixgbe_is_vf(struct rte_eth_dev *dev) -{ > > > > > > > > - struct ixgbe_hw *hw =3D IXGBE_DEV_PRIVATE_TO_HW(dev- > > > > >data- > > > > > > > > >dev_private); > > > > > > > > - > > > > > > > > - switch (hw->mac.type) { > > > > > > > > - case ixgbe_mac_82599_vf: > > > > > > > > - case ixgbe_mac_X540_vf: > > > > > > > > - case ixgbe_mac_X550_vf: > > > > > > > > - case ixgbe_mac_X550EM_x_vf: > > > > > > > > - case ixgbe_mac_X550EM_a_vf: > > > > > > > > - return 1; > > > > > > > > - default: > > > > > > > > - return 0; > > > > > > > > - } > > > > > > > > -} > > > > > > > > - > > > > > > > > uint64_t > > > > > > > > ixgbe_get_rx_queue_offloads(struct rte_eth_dev *dev) { @@ > > > > > > > > -2853,15 > > > > > > > > +2836,13 @@ ixgbe_get_rx_port_offloads(struct rte_eth_dev > > > > > > > > +*dev) > > > > > > > > DEV_RX_OFFLOAD_TCP_CKSUM | > > > > > > > > DEV_RX_OFFLOAD_KEEP_CRC | > > > > > > > > DEV_RX_OFFLOAD_JUMBO_FRAME | > > > > > > > > + DEV_RX_OFFLOAD_VLAN_FILTER | > > > > > > > > + DEV_RX_OFFLOAD_VLAN_EXTEND | > > > > > > > > DEV_RX_OFFLOAD_SCATTER; > > > > > > > > > > > > > > > > if (hw->mac.type =3D=3D ixgbe_mac_82598EB) > > > > > > > > offloads |=3D DEV_RX_OFFLOAD_VLAN_STRIP; > > > > > > > > > > > > > > > > - if (ixgbe_is_vf(dev) =3D=3D 0) > > > > > > > > - offloads |=3D (DEV_RX_OFFLOAD_VLAN_FILTER | > > > > > > > > - DEV_RX_OFFLOAD_VLAN_EXTEND); > > > > > > > > - > > > > > > > > /* > > > > > > > > * RSC is only supported by 82599 and x540 PF devices in = a > > > > > > > > non-SR- IOV > > > > > > > > * mode. > > > > > > > > -- > > > > > > > > 2.19.1