[dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for ixgbvef

[dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for ixgbvef

[David Harton]

ixgevf vlan strip and extend capabilities were removed when
migrating to the bit flags implementation.

Restoring the capbility to enable these offloads at
configuration time.

Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
Signed-off-by: David Harton <dharton@cisco.com>
---
 drivers/net/ixgbe/ixgbe_rxtx.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
index e92a70fb3..95c32257c 100644
--- a/drivers/net/ixgbe/ixgbe_rxtx.c
+++ b/drivers/net/ixgbe/ixgbe_rxtx.c
@@ -2853,15 +2853,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 == ixgbe_mac_82598EB)
 		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
 
-	if (ixgbe_is_vf(dev) == 0)
-		offloads |= (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

Re: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for ixgbvef

[Zhang, Qi Z]

Hi

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Harton
> Sent: Wednesday, March 6, 2019 11:24 PM
> To: dev@dpdk.org; Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>
> Cc: David Harton <dharton@cisco.com>
> Subject: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for ixgbvef
> 
> ixgevf vlan strip and extend capabilities were removed when migrating to the bit
> flags implementation.
> 
> Restoring the capbility to enable these offloads at configuration time.
> 
> Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> Signed-off-by: David Harton <dharton@cisco.com>
> ---
>  drivers/net/ixgbe/ixgbe_rxtx.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
> index e92a70fb3..95c32257c 100644
> --- a/drivers/net/ixgbe/ixgbe_rxtx.c
> +++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> @@ -2853,15 +2853,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 == ixgbe_mac_82598EB)
>  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> 
> -	if (ixgbe_is_vf(dev) == 0)

Patchwork shows "build patch error" for ci/Performance-Testing.
Seems this is the only place ixgbe_is_vf is used, so it can also be removed

> -		offloads |= (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

Re: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for ixgbvef

[David Harton (dharton)]

Hi,

> -----Original Message-----
> From: Zhang, Qi Z <qi.z.zhang@intel.com>
> Sent: Thursday, March 07, 2019 8:11 AM
> To: David Harton (dharton) <dharton@cisco.com>; dev@dpdk.org; Lu, Wenzhuo
> <wenzhuo.lu@intel.com>; Ananyev, Konstantin <konstantin.ananyev@intel.com>
> Subject: RE: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for
> ixgbvef
> 
> Hi
> 
> > -----Original Message-----
> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Harton
> > Sent: Wednesday, March 6, 2019 11:24 PM
> > To: dev@dpdk.org; Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev,
> > Konstantin <konstantin.ananyev@intel.com>
> > Cc: David Harton <dharton@cisco.com>
> > Subject: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for
> > ixgbvef
> >
> > ixgevf vlan strip and extend capabilities were removed when migrating
> > to the bit flags implementation.
> >
> > Restoring the capbility to enable these offloads at configuration time.
> >
> > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> > Signed-off-by: David Harton <dharton@cisco.com>
> > ---
> >  drivers/net/ixgbe/ixgbe_rxtx.c | 6 ++----
> >  1 file changed, 2 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c
> > b/drivers/net/ixgbe/ixgbe_rxtx.c index e92a70fb3..95c32257c 100644
> > --- a/drivers/net/ixgbe/ixgbe_rxtx.c
> > +++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> > @@ -2853,15 +2853,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 == ixgbe_mac_82598EB)
> >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> >
> > -	if (ixgbe_is_vf(dev) == 0)
> 
> Patchwork shows "build patch error" for ci/Performance-Testing.
> Seems this is the only place ixgbe_is_vf is used, so it can also be
> removed

Yes, I didn't notice that.  Assuming people are good with the fundamental change I'll post a new patch with ixgbe_is_vf removed.

Thanks,
Dave

> 
> > -		offloads |= (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

Re: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for ixgbvef

[Zhao1, Wei]

Hi,  David Harton

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Harton
> Sent: Wednesday, March 6, 2019 11:24 PM
> To: dev@dpdk.org; Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev,
> Konstantin <konstantin.ananyev@intel.com>
> Cc: David Harton <dharton@cisco.com>
> Subject: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for
> ixgbvef
> 
> ixgevf vlan strip and extend capabilities were removed when migrating to the
> bit flags implementation.
> 
> Restoring the capbility to enable these offloads at configuration time.
> 
> Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> Signed-off-by: David Harton <dharton@cisco.com>
> ---
>  drivers/net/ixgbe/ixgbe_rxtx.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
> index e92a70fb3..95c32257c 100644
> --- a/drivers/net/ixgbe/ixgbe_rxtx.c
> +++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> @@ -2853,15 +2853,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;
> 

Why do we  set   DEV_RX_OFFLOAD_VLAN_EXTEND for ixgbevf ?  
It seems Ixgbevf do not support this configuration. Only pf does.
But this ixgbe_get_rx_port_offloads () is used by both vf and pf. 


>  	if (hw->mac.type == ixgbe_mac_82598EB)
>  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> 
> -	if (ixgbe_is_vf(dev) == 0)
> -		offloads |= (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

Re: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for ixgbvef

[David Harton (dharton)]

Hi

> -----Original Message-----
> From: Zhao1, Wei <wei.zhao1@intel.com>
> Sent: Friday, March 08, 2019 1:35 AM
> To: David Harton (dharton) <dharton@cisco.com>; dev@dpdk.org; Lu, Wenzhuo
> <wenzhuo.lu@intel.com>; Ananyev, Konstantin <konstantin.ananyev@intel.com>
> Subject: RE: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for
> ixgbvef
> 
> Hi,  David Harton
> 
> > -----Original Message-----
> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Harton
> > Sent: Wednesday, March 6, 2019 11:24 PM
> > To: dev@dpdk.org; Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev,
> > Konstantin <konstantin.ananyev@intel.com>
> > Cc: David Harton <dharton@cisco.com>
> > Subject: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for
> > ixgbvef
> >
> > ixgevf vlan strip and extend capabilities were removed when migrating
> > to the bit flags implementation.
> >
> > Restoring the capbility to enable these offloads at configuration time.
> >
> > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> > Signed-off-by: David Harton <dharton@cisco.com>
> > ---
> >  drivers/net/ixgbe/ixgbe_rxtx.c | 6 ++----
> >  1 file changed, 2 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c
> > b/drivers/net/ixgbe/ixgbe_rxtx.c index e92a70fb3..95c32257c 100644
> > --- a/drivers/net/ixgbe/ixgbe_rxtx.c
> > +++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> > @@ -2853,15 +2853,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;
> >
> 
> Why do we  set   DEV_RX_OFFLOAD_VLAN_EXTEND for ixgbevf ?
> It seems Ixgbevf do not support this configuration. Only pf does.
> But this ixgbe_get_rx_port_offloads () is used by both vf and pf.

I'm personally happy only to move FILTER but I noticed that EXTEND used to be 'supported' and I see references to it elsewhere in ixgbevf code:

ixgbevf_dev_start()
    /* Set HW strip */
    mask = ETH_VLAN_STRIP_MASK | ETH_VLAN_FILTER_MASK |
        ETH_VLAN_EXTEND_MASK;

ixgbe_vlan_offload_config()
    if (mask & ETH_VLAN_EXTEND_MASK) {
        if (rxmode->offloads & DEV_RX_OFFLOAD_VLAN_EXTEND)
            ixgbe_vlan_hw_extend_enable(dev);
        else
            ixgbe_vlan_hw_extend_disable(dev);
    }

These uses imply that EXTEND should be supported, correct?

Thanks,
Dave

> 
> 
> >  	if (hw->mac.type == ixgbe_mac_82598EB)
> >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> >
> > -	if (ixgbe_is_vf(dev) == 0)
> > -		offloads |= (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

Re: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for ixgbvef

[Zhao1, Wei]

Hi,



> -----Original Message-----

> From: David Harton (dharton) [mailto:dharton@cisco.com]

> Sent: Friday, March 8, 2019 8:39 PM

> To: Zhao1, Wei <wei.zhao1@intel.com>; dev@dpdk.org; Lu, Wenzhuo

> <wenzhuo.lu@intel.com>; Ananyev, Konstantin

> <konstantin.ananyev@intel.com>

> Subject: RE: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend for

> ixgbvef

>

> Hi

>

> > -----Original Message-----

> > From: Zhao1, Wei <wei.zhao1@intel.com<mailto:wei.zhao1@intel.com>>

> > Sent: Friday, March 08, 2019 1:35 AM

> > To: David Harton (dharton) <dharton@cisco.com<mailto:dharton@cisco.com>>; dev@dpdk.org<mailto:dev@dpdk.org>; Lu,

> > Wenzhuo <wenzhuo.lu@intel.com<mailto:wenzhuo.lu@intel.com>>; Ananyev, Konstantin

> > <konstantin.ananyev@intel.com<mailto:konstantin.ananyev@intel.com>>

> > Subject: RE: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend

> > for ixgbvef

> >

> > Hi,  David Harton

> >

> > > -----Original Message-----

> > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Harton

> > > Sent: Wednesday, March 6, 2019 11:24 PM

> > > To: dev@dpdk.org<mailto:dev@dpdk.org>; Lu, Wenzhuo <wenzhuo.lu@intel.com<mailto:wenzhuo.lu@intel.com>>; Ananyev,

> > > Konstantin <konstantin.ananyev@intel.com<mailto:konstantin.ananyev@intel.com>>

> > > Cc: David Harton <dharton@cisco.com<mailto:dharton@cisco.com>>

> > > Subject: [dpdk-dev] [PATCH] net/ixgbe: Restore vlan filter/extend

> > > for ixgbvef

> > >

> > > ixgevf vlan strip and extend capabilities were removed when

> > > migrating to the bit flags implementation.

> > >

> > > Restoring the capbility to enable these offloads at configuration time.

> > >

> > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")

> > > Signed-off-by: David Harton <dharton@cisco.com<mailto:dharton@cisco.com>>

> > > ---

> > >  drivers/net/ixgbe/ixgbe_rxtx.c | 6 ++----

> > >  1 file changed, 2 insertions(+), 4 deletions(-)

> > >

> > > diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c

> > > b/drivers/net/ixgbe/ixgbe_rxtx.c index e92a70fb3..95c32257c 100644

> > > --- a/drivers/net/ixgbe/ixgbe_rxtx.c

> > > +++ b/drivers/net/ixgbe/ixgbe_rxtx.c

> > > @@ -2853,15 +2853,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;

> > >

> >

> > Why do we  set   DEV_RX_OFFLOAD_VLAN_EXTEND for ixgbevf ?

> > It seems Ixgbevf do not support this configuration. Only pf does.

> > But this ixgbe_get_rx_port_offloads () is used by both vf and pf.

>

> I'm personally happy only to move FILTER but I noticed that EXTEND used to

> be 'supported' and I see references to it elsewhere in ixgbevf code:

>

> ixgbevf_dev_start()

>     /* Set HW strip */

>     mask = ETH_VLAN_STRIP_MASK | ETH_VLAN_FILTER_MASK |

>         ETH_VLAN_EXTEND_MASK;

>

> ixgbe_vlan_offload_config()

>     if (mask & ETH_VLAN_EXTEND_MASK) {

>         if (rxmode->offloads & DEV_RX_OFFLOAD_VLAN_EXTEND)

>             ixgbe_vlan_hw_extend_enable(dev);

>         else

>             ixgbe_vlan_hw_extend_disable(dev);

>     }

>

> These uses imply that EXTEND should be supported, correct?



No, not ixgbe_vlan_offload_config, but ixgbevf_vlan_offload_config,

ixgbevf_dev_start() is below:



                /* Set HW strip */

                mask = ETH_VLAN_STRIP_MASK | ETH_VLAN_FILTER_MASK |

                                ETH_VLAN_EXTEND_MASK;

                err = ixgbevf_vlan_offload_config(dev, mask);







static int

ixgbevf_vlan_offload_config(struct rte_eth_dev *dev, int mask)

{

                struct ixgbe_rx_queue *rxq;

                uint16_t i;

                int on = 0;



                /* VF function only support hw strip feature, others are not support */

                if (mask & ETH_VLAN_STRIP_MASK) {

                                for (i = 0; i < dev->data->nb_rx_queues; i++) {

                                                rxq = dev->data->rx_queues[i];

                                                on = !!(rxq->offloads & DEV_RX_OFFLOAD_VLAN_STRIP);

                                                ixgbevf_vlan_strip_queue_set(dev, i, on);

                                }

                }



                return 0;

}







>

> Thanks,

> Dave

>

> >

> >

> > >      if (hw->mac.type == ixgbe_mac_82598EB)

> > >                      offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;

> > >

> > > -   if (ixgbe_is_vf(dev) == 0)

> > > -                   offloads |= (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

[dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[David Harton]

ixgbevf vlan strip and extend capabilities were removed when
migrating to the bit flags implementation.

Restoring the capbility to enable these offloads at
configuration time.

Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
Signed-off-by: David Harton <dharton@cisco.com>
---

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 = 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 == ixgbe_mac_82598EB)
 		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
 
-	if (ixgbe_is_vf(dev) == 0)
-		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[David Harton (dharton)]

Hi Qi,

This patch was rejected with no feedback.

Can you share why?

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().

Or, am I missing something?

Thanks,
Dave

> -----Original Message-----
> From: David Harton <dharton@cisco.com>
> 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) <dharton@cisco.com>
> Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> ixgbevf vlan strip and extend capabilities were removed when migrating to
> the bit flags implementation.
> 
> Restoring the capbility to enable these offloads at configuration time.
> 
> Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> Signed-off-by: David Harton <dharton@cisco.com>
> ---
> 
> 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 = 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 == ixgbe_mac_82598EB)
>  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> 
> -	if (ixgbe_is_vf(dev) == 0)
> -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[David Harton (dharton)]

Hi Qi,

This patch was rejected with no feedback.

Can you share why?

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().

Or, am I missing something?

Thanks,
Dave

> -----Original Message-----
> From: David Harton <dharton@cisco.com>
> 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) <dharton@cisco.com>
> Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> ixgbevf vlan strip and extend capabilities were removed when migrating to
> the bit flags implementation.
> 
> Restoring the capbility to enable these offloads at configuration time.
> 
> Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> Signed-off-by: David Harton <dharton@cisco.com>
> ---
> 
> 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 = 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 == ixgbe_mac_82598EB)
>  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> 
> -	if (ixgbe_is_vf(dev) == 0)
> -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhang, Qi Z]

> -----Original Message-----
> From: David Harton (dharton) [mailto:dharton@cisco.com]
> Sent: Thursday, March 14, 2019 9:38 PM
> To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> <qi.z.zhang@intel.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 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 more insight? 
  
And same question to VLAN_EXTEND


> 
> Or, am I missing something?
> 
> Thanks,
> Dave
> 
> > -----Original Message-----
> > From: David Harton <dharton@cisco.com>
> > 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) <dharton@cisco.com>
> > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> >
> > ixgbevf vlan strip and extend capabilities were removed when migrating
> > to the bit flags implementation.
> >
> > Restoring the capbility to enable these offloads at configuration time.
> >
> > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> > Signed-off-by: David Harton <dharton@cisco.com>
> > ---
> >
> > 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 = 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 == ixgbe_mac_82598EB)
> >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> >
> > -	if (ixgbe_is_vf(dev) == 0)
> > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhang, Qi Z]

> -----Original Message-----
> From: David Harton (dharton) [mailto:dharton@cisco.com]
> Sent: Thursday, March 14, 2019 9:38 PM
> To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> <qi.z.zhang@intel.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 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 more insight? 
  
And same question to VLAN_EXTEND


> 
> Or, am I missing something?
> 
> Thanks,
> Dave
> 
> > -----Original Message-----
> > From: David Harton <dharton@cisco.com>
> > 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) <dharton@cisco.com>
> > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> >
> > ixgbevf vlan strip and extend capabilities were removed when migrating
> > to the bit flags implementation.
> >
> > Restoring the capbility to enable these offloads at configuration time.
> >
> > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> > Signed-off-by: David Harton <dharton@cisco.com>
> > ---
> >
> > 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 = 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 == ixgbe_mac_82598EB)
> >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> >
> > -	if (ixgbe_is_vf(dev) == 0)
> > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[David Harton (dharton)]

Hi Qi,

> -----Original Message-----
> From: Zhang, Qi Z <qi.z.zhang@intel.com>
> Sent: Thursday, March 14, 2019 11:14 AM
> To: David Harton (dharton) <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 
> 
> > -----Original Message-----
> > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > Sent: Thursday, March 14, 2019 9:38 PM
> > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > <qi.z.zhang@intel.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > ixgbevf
> >
> > 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 more 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.

> 
> 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.

Thanks,
Dave

> 
> 
> >
> > Or, am I missing something?
> >
> > Thanks,
> > Dave
> >
> > > -----Original Message-----
> > > From: David Harton <dharton@cisco.com>
> > > 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) <dharton@cisco.com>
> > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > > ixgbevf vlan strip and extend capabilities were removed when
> > > migrating to the bit flags implementation.
> > >
> > > Restoring the capbility to enable these offloads at configuration
> time.
> > >
> > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> > > Signed-off-by: David Harton <dharton@cisco.com>
> > > ---
> > >
> > > 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 = 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 == ixgbe_mac_82598EB)
> > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > >
> > > -	if (ixgbe_is_vf(dev) == 0)
> > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[David Harton (dharton)]

Hi Qi,

> -----Original Message-----
> From: Zhang, Qi Z <qi.z.zhang@intel.com>
> Sent: Thursday, March 14, 2019 11:14 AM
> To: David Harton (dharton) <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 
> 
> > -----Original Message-----
> > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > Sent: Thursday, March 14, 2019 9:38 PM
> > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > <qi.z.zhang@intel.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > ixgbevf
> >
> > 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 more 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.

> 
> 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.

Thanks,
Dave

> 
> 
> >
> > Or, am I missing something?
> >
> > Thanks,
> > Dave
> >
> > > -----Original Message-----
> > > From: David Harton <dharton@cisco.com>
> > > 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) <dharton@cisco.com>
> > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > > ixgbevf vlan strip and extend capabilities were removed when
> > > migrating to the bit flags implementation.
> > >
> > > Restoring the capbility to enable these offloads at configuration
> time.
> > >
> > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> > > Signed-off-by: David Harton <dharton@cisco.com>
> > > ---
> > >
> > > 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 = 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 == ixgbe_mac_82598EB)
> > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > >
> > > -	if (ixgbe_is_vf(dev) == 0)
> > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhang, Qi Z]

> -----Original Message-----
> From: David Harton (dharton) [mailto:dharton@cisco.com]
> Sent: Thursday, March 14, 2019 11:51 PM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> Hi Qi,
> 
> > -----Original Message-----
> > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > Sent: Thursday, March 14, 2019 11:14 AM
> > To: David Harton (dharton) <dharton@cisco.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > ixgbevf
> >
> >
> >
> > > -----Original Message-----
> > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > Sent: Thursday, March 14, 2019 9:38 PM
> > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > <qi.z.zhang@intel.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > > 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 more 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?

Thanks
Qi

> 
> Thanks,
> Dave
> 
> >
> >
> > >
> > > Or, am I missing something?
> > >
> > > Thanks,
> > > Dave
> > >
> > > > -----Original Message-----
> > > > From: David Harton <dharton@cisco.com>
> > > > 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) <dharton@cisco.com>
> > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > migrating to the bit flags implementation.
> > > >
> > > > Restoring the capbility to enable these offloads at configuration
> > time.
> > > >
> > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > API\")
> > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > ---
> > > >
> > > > 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 = 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 == ixgbe_mac_82598EB)
> > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > >
> > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhang, Qi Z]

> -----Original Message-----
> From: David Harton (dharton) [mailto:dharton@cisco.com]
> Sent: Thursday, March 14, 2019 11:51 PM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> Hi Qi,
> 
> > -----Original Message-----
> > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > Sent: Thursday, March 14, 2019 11:14 AM
> > To: David Harton (dharton) <dharton@cisco.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > ixgbevf
> >
> >
> >
> > > -----Original Message-----
> > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > Sent: Thursday, March 14, 2019 9:38 PM
> > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > <qi.z.zhang@intel.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > > 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 more 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?

Thanks
Qi

> 
> Thanks,
> Dave
> 
> >
> >
> > >
> > > Or, am I missing something?
> > >
> > > Thanks,
> > > Dave
> > >
> > > > -----Original Message-----
> > > > From: David Harton <dharton@cisco.com>
> > > > 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) <dharton@cisco.com>
> > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > migrating to the bit flags implementation.
> > > >
> > > > Restoring the capbility to enable these offloads at configuration
> > time.
> > > >
> > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > API\")
> > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > ---
> > > >
> > > > 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 = 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 == ixgbe_mac_82598EB)
> > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > >
> > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhao1, Wei]

> -----Original Message-----
> From: Zhang, Qi Z
> Sent: Friday, March 15, 2019 9:20 AM
> To: David Harton (dharton) <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org; Zhao1, Wei
> <wei.zhao1@intel.com>
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 
> 
> > -----Original Message-----
> > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > Sent: Thursday, March 14, 2019 11:51 PM
> > To: Zhang, Qi Z <qi.z.zhang@intel.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > ixgbevf
> >
> > Hi Qi,
> >
> > > -----Original Message-----
> > > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > Sent: Thursday, March 14, 2019 11:14 AM
> > > To: David Harton (dharton) <dharton@cisco.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > Sent: Thursday, March 14, 2019 9:38 PM
> > > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > > <qi.z.zhang@intel.com>
> > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > > 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 more 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

It seems kernel pf enable vlan filter by default?
This vf fuction is depend on kernel pf.

> >
> > >
> > > 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?

I will check that later.

> 
> Thanks
> Qi
> 
> >
> > Thanks,
> > Dave
> >
> > >
> > >
> > > >
> > > > Or, am I missing something?
> > > >
> > > > Thanks,
> > > > Dave
> > > >
> > > > > -----Original Message-----
> > > > > From: David Harton <dharton@cisco.com>
> > > > > 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) <dharton@cisco.com>
> > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > > ixgbevf
> > > > >
> > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > migrating to the bit flags implementation.
> > > > >
> > > > > Restoring the capbility to enable these offloads at
> > > > > configuration
> > > time.
> > > > >
> > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > API\")
> > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > ---
> > > > >
> > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > >
> > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhao1, Wei]

> -----Original Message-----
> From: Zhang, Qi Z
> Sent: Friday, March 15, 2019 9:20 AM
> To: David Harton (dharton) <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org; Zhao1, Wei
> <wei.zhao1@intel.com>
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 
> 
> > -----Original Message-----
> > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > Sent: Thursday, March 14, 2019 11:51 PM
> > To: Zhang, Qi Z <qi.z.zhang@intel.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > ixgbevf
> >
> > Hi Qi,
> >
> > > -----Original Message-----
> > > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > Sent: Thursday, March 14, 2019 11:14 AM
> > > To: David Harton (dharton) <dharton@cisco.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > Sent: Thursday, March 14, 2019 9:38 PM
> > > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > > <qi.z.zhang@intel.com>
> > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > > 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 more 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

It seems kernel pf enable vlan filter by default?
This vf fuction is depend on kernel pf.

> >
> > >
> > > 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?

I will check that later.

> 
> Thanks
> Qi
> 
> >
> > Thanks,
> > Dave
> >
> > >
> > >
> > > >
> > > > Or, am I missing something?
> > > >
> > > > Thanks,
> > > > Dave
> > > >
> > > > > -----Original Message-----
> > > > > From: David Harton <dharton@cisco.com>
> > > > > 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) <dharton@cisco.com>
> > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > > ixgbevf
> > > > >
> > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > migrating to the bit flags implementation.
> > > > >
> > > > > Restoring the capbility to enable these offloads at
> > > > > configuration
> > > time.
> > > > >
> > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > API\")
> > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > ---
> > > > >
> > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > >
> > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhao1, Wei]

Hi，qi

> -----Original Message-----
> From: Zhang, Qi Z
> Sent: Friday, March 15, 2019 9:20 AM
> To: David Harton (dharton) <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org; Zhao1, Wei
> <wei.zhao1@intel.com>
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 
> 
> > -----Original Message-----
> > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > Sent: Thursday, March 14, 2019 11:51 PM
> > To: Zhang, Qi Z <qi.z.zhang@intel.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > ixgbevf
> >
> > Hi Qi,
> >
> > > -----Original Message-----
> > > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > Sent: Thursday, March 14, 2019 11:14 AM
> > > To: David Harton (dharton) <dharton@cisco.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > Sent: Thursday, March 14, 2019 9:38 PM
> > > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > > <qi.z.zhang@intel.com>
> > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > > 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 more 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.

Maybe we can change code to
"
	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;

	if (ixgbe_is_vf(dev) == 0)
		offloads |= 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 <dharton@cisco.com>
> > > > > 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) <dharton@cisco.com>
> > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > > ixgbevf
> > > > >
> > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > migrating to the bit flags implementation.
> > > > >
> > > > > Restoring the capbility to enable these offloads at
> > > > > configuration
> > > time.
> > > > >
> > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > API\")
> > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > ---
> > > > >
> > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > >
> > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhao1, Wei]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="UTF-8", Size: 7185 bytes --]

Hi£¬qi

> -----Original Message-----
> From: Zhang, Qi Z
> Sent: Friday, March 15, 2019 9:20 AM
> To: David Harton (dharton) <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org; Zhao1, Wei
> <wei.zhao1@intel.com>
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 
> 
> > -----Original Message-----
> > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > Sent: Thursday, March 14, 2019 11:51 PM
> > To: Zhang, Qi Z <qi.z.zhang@intel.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > ixgbevf
> >
> > Hi Qi,
> >
> > > -----Original Message-----
> > > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > Sent: Thursday, March 14, 2019 11:14 AM
> > > To: David Harton (dharton) <dharton@cisco.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > Sent: Thursday, March 14, 2019 9:38 PM
> > > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > > <qi.z.zhang@intel.com>
> > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > > 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 more 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.

Maybe we can change code to
"
	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;

	if (ixgbe_is_vf(dev) == 0)
		offloads |= 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 <dharton@cisco.com>
> > > > > 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) <dharton@cisco.com>
> > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > > ixgbevf
> > > > >
> > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > migrating to the bit flags implementation.
> > > > >
> > > > > Restoring the capbility to enable these offloads at
> > > > > configuration
> > > time.
> > > > >
> > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > API\")
> > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > ---
> > > > >
> > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > >
> > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhang, Qi Z]

> -----Original Message-----
> From: Zhao1, Wei
> Sent: Friday, March 15, 2019 1:28 PM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>; David Harton (dharton)
> <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> Hi，qi
> 
> > -----Original Message-----
> > From: Zhang, Qi Z
> > Sent: Friday, March 15, 2019 9:20 AM
> > To: David Harton (dharton) <dharton@cisco.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org; Zhao1, Wei
> > <wei.zhao1@intel.com>
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > ixgbevf
> >
> >
> >
> > > -----Original Message-----
> > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > Sent: Thursday, March 14, 2019 11:51 PM
> > > To: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > > Hi Qi,
> > >
> > > > -----Original Message-----
> > > > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > > Sent: Thursday, March 14, 2019 11:14 AM
> > > > To: David Harton (dharton) <dharton@cisco.com>
> > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > > Sent: Thursday, March 14, 2019 9:38 PM
> > > > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > > > <qi.z.zhang@intel.com>
> > > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend
> > > > > for ixgbevf
> > > > >
> > > > > 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 more 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.
> 
> Maybe we can change code to
> "
> 	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;
> 
> 	if (ixgbe_is_vf(dev) == 0)
> 		offloads |= DEV_RX_OFFLOAD_VLAN_EXTEND; "

+1

> 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 <dharton@cisco.com>
> > > > > > 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)
> > > > > > <dharton@cisco.com>
> > > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > > > ixgbevf
> > > > > >
> > > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > > migrating to the bit flags implementation.
> > > > > >
> > > > > > Restoring the capbility to enable these offloads at
> > > > > > configuration
> > > > time.
> > > > > >
> > > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > > API\")
> > > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > > ---
> > > > > >
> > > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > > >
> > > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhang, Qi Z]

> -----Original Message-----
> From: Zhao1, Wei
> Sent: Friday, March 15, 2019 1:28 PM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>; David Harton (dharton)
> <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> Hi^[$B!$^[(Bqi
> 
> > -----Original Message-----
> > From: Zhang, Qi Z
> > Sent: Friday, March 15, 2019 9:20 AM
> > To: David Harton (dharton) <dharton@cisco.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org; Zhao1, Wei
> > <wei.zhao1@intel.com>
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > ixgbevf
> >
> >
> >
> > > -----Original Message-----
> > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > Sent: Thursday, March 14, 2019 11:51 PM
> > > To: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > > Hi Qi,
> > >
> > > > -----Original Message-----
> > > > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > > Sent: Thursday, March 14, 2019 11:14 AM
> > > > To: David Harton (dharton) <dharton@cisco.com>
> > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > > Sent: Thursday, March 14, 2019 9:38 PM
> > > > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > > > <qi.z.zhang@intel.com>
> > > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend
> > > > > for ixgbevf
> > > > >
> > > > > 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 more 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.
> 
> Maybe we can change code to
> "
> 	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;
> 
> 	if (ixgbe_is_vf(dev) == 0)
> 		offloads |= DEV_RX_OFFLOAD_VLAN_EXTEND; "

+1

> 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 <dharton@cisco.com>
> > > > > > 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)
> > > > > > <dharton@cisco.com>
> > > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > > > ixgbevf
> > > > > >
> > > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > > migrating to the bit flags implementation.
> > > > > >
> > > > > > Restoring the capbility to enable these offloads at
> > > > > > configuration
> > > > time.
> > > > > >
> > > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > > API\")
> > > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > > ---
> > > > > >
> > > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > > >
> > > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Ananyev, Konstantin]

> 
> Hi，qi
> 
> > -----Original Message-----
> > From: Zhang, Qi Z
> > Sent: Friday, March 15, 2019 9:20 AM
> > To: David Harton (dharton) <dharton@cisco.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org; Zhao1, Wei
> > <wei.zhao1@intel.com>
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> >
> >
> >
> > > -----Original Message-----
> > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > Sent: Thursday, March 14, 2019 11:51 PM
> > > To: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > > Hi Qi,
> > >
> > > > -----Original Message-----
> > > > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > > Sent: Thursday, March 14, 2019 11:14 AM
> > > > To: David Harton (dharton) <dharton@cisco.com>
> > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > > Sent: Thursday, March 14, 2019 9:38 PM
> > > > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > > > <qi.z.zhang@intel.com>
> > > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > > ixgbevf
> > > > >
> > > > > 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 more 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


> 
> Maybe we can change code to
> "
> 	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;
> 
> 	if (ixgbe_is_vf(dev) == 0)
> 		offloads |= 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 <dharton@cisco.com>
> > > > > > 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) <dharton@cisco.com>
> > > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > > > ixgbevf
> > > > > >
> > > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > > migrating to the bit flags implementation.
> > > > > >
> > > > > > Restoring the capbility to enable these offloads at
> > > > > > configuration
> > > > time.
> > > > > >
> > > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > > API\")
> > > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > > ---
> > > > > >
> > > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > > >
> > > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Ananyev, Konstantin]

> 
> Hi^[$B!$^[(Bqi
> 
> > -----Original Message-----
> > From: Zhang, Qi Z
> > Sent: Friday, March 15, 2019 9:20 AM
> > To: David Harton (dharton) <dharton@cisco.com>
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; dev@dpdk.org; Zhao1, Wei
> > <wei.zhao1@intel.com>
> > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> >
> >
> >
> > > -----Original Message-----
> > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > Sent: Thursday, March 14, 2019 11:51 PM
> > > To: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > > Hi Qi,
> > >
> > > > -----Original Message-----
> > > > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > > Sent: Thursday, March 14, 2019 11:14 AM
> > > > To: David Harton (dharton) <dharton@cisco.com>
> > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > > Sent: Thursday, March 14, 2019 9:38 PM
> > > > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > > > <qi.z.zhang@intel.com>
> > > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > > ixgbevf
> > > > >
> > > > > 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 more 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


> 
> Maybe we can change code to
> "
> 	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;
> 
> 	if (ixgbe_is_vf(dev) == 0)
> 		offloads |= 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 <dharton@cisco.com>
> > > > > > 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) <dharton@cisco.com>
> > > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > > > ixgbevf
> > > > > >
> > > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > > migrating to the bit flags implementation.
> > > > > >
> > > > > > Restoring the capbility to enable these offloads at
> > > > > > configuration
> > > > time.
> > > > > >
> > > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > > API\")
> > > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > > ---
> > > > > >
> > > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > > >
> > > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhao1, Wei]

Hi , Konstantin

> -----Original Message-----
> From: Ananyev, Konstantin
> Sent: Friday, March 15, 2019 5:24 PM
> To: Zhao1, Wei <wei.zhao1@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>;
> David Harton (dharton) <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 
> >
> > Hi，qi
> >
> > > -----Original Message-----
> > > From: Zhang, Qi Z
> > > Sent: Friday, March 15, 2019 9:20 AM
> > > To: David Harton (dharton) <dharton@cisco.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org; Zhao1, Wei
> > > <wei.zhao1@intel.com>
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > Sent: Thursday, March 14, 2019 11:51 PM
> > > > To: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > > Hi Qi,
> > > >
> > > > > -----Original Message-----
> > > > > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > > > Sent: Thursday, March 14, 2019 11:14 AM
> > > > > To: David Harton (dharton) <dharton@cisco.com>
> > > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend
> > > > > for ixgbevf
> > > > >
> > > > >
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > > > Sent: Thursday, March 14, 2019 9:38 PM
> > > > > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > > > > <qi.z.zhang@intel.com>
> > > > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend
> > > > > > for ixgbevf
> > > > > >
> > > > > > 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 more
> 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

No such type of message defined in mailbox by now.

> 
> 
> >
> > Maybe we can change code to
> > "
> > 	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;
> >
> > 	if (ixgbe_is_vf(dev) == 0)
> > 		offloads |= 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 <dharton@cisco.com>
> > > > > > > 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)
> > > > > > > <dharton@cisco.com>
> > > > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend
> > > > > > > for ixgbevf
> > > > > > >
> > > > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > > > migrating to the bit flags implementation.
> > > > > > >
> > > > > > > Restoring the capbility to enable these offloads at
> > > > > > > configuration
> > > > > time.
> > > > > > >
> > > > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > > > API\")
> > > > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > > > ---
> > > > > > >
> > > > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > > > >
> > > > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhao1, Wei]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="UTF-8", Size: 8558 bytes --]

Hi , Konstantin

> -----Original Message-----
> From: Ananyev, Konstantin
> Sent: Friday, March 15, 2019 5:24 PM
> To: Zhao1, Wei <wei.zhao1@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>;
> David Harton (dharton) <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 
> >
> > Hi£¬qi
> >
> > > -----Original Message-----
> > > From: Zhang, Qi Z
> > > Sent: Friday, March 15, 2019 9:20 AM
> > > To: David Harton (dharton) <dharton@cisco.com>
> > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>; dev@dpdk.org; Zhao1, Wei
> > > <wei.zhao1@intel.com>
> > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > ixgbevf
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > Sent: Thursday, March 14, 2019 11:51 PM
> > > > To: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for
> > > > ixgbevf
> > > >
> > > > Hi Qi,
> > > >
> > > > > -----Original Message-----
> > > > > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > > > Sent: Thursday, March 14, 2019 11:14 AM
> > > > > To: David Harton (dharton) <dharton@cisco.com>
> > > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend
> > > > > for ixgbevf
> > > > >
> > > > >
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: David Harton (dharton) [mailto:dharton@cisco.com]
> > > > > > Sent: Thursday, March 14, 2019 9:38 PM
> > > > > > To: David Harton (dharton) <dharton@cisco.com>; Zhang, Qi Z
> > > > > > <qi.z.zhang@intel.com>
> > > > > > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > > > > > <konstantin.ananyev@intel.com>; dev@dpdk.org
> > > > > > Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend
> > > > > > for ixgbevf
> > > > > >
> > > > > > 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 more
> 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

No such type of message defined in mailbox by now.

> 
> 
> >
> > Maybe we can change code to
> > "
> > 	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;
> >
> > 	if (ixgbe_is_vf(dev) == 0)
> > 		offloads |= 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 <dharton@cisco.com>
> > > > > > > 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)
> > > > > > > <dharton@cisco.com>
> > > > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend
> > > > > > > for ixgbevf
> > > > > > >
> > > > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > > > migrating to the bit flags implementation.
> > > > > > >
> > > > > > > Restoring the capbility to enable these offloads at
> > > > > > > configuration
> > > > > time.
> > > > > > >
> > > > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > > > API\")
> > > > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > > > ---
> > > > > > >
> > > > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > > > >
> > > > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Ananyev, Konstantin]

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 more
> > 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
> 
> 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?

> 
> >
> >
> > >
> > > Maybe we can change code to
> > > "
> > > 	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;
> > >
> > > 	if (ixgbe_is_vf(dev) == 0)
> > > 		offloads |= 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 <dharton@cisco.com>
> > > > > > > > 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)
> > > > > > > > <dharton@cisco.com>
> > > > > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend
> > > > > > > > for ixgbevf
> > > > > > > >
> > > > > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > > > > migrating to the bit flags implementation.
> > > > > > > >
> > > > > > > > Restoring the capbility to enable these offloads at
> > > > > > > > configuration
> > > > > > time.
> > > > > > > >
> > > > > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > > > > API\")
> > > > > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > > > > ---
> > > > > > > >
> > > > > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > > > > >
> > > > > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Ananyev, Konstantin]

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 more
> > 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
> 
> 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?

> 
> >
> >
> > >
> > > Maybe we can change code to
> > > "
> > > 	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;
> > >
> > > 	if (ixgbe_is_vf(dev) == 0)
> > > 		offloads |= 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 <dharton@cisco.com>
> > > > > > > > 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)
> > > > > > > > <dharton@cisco.com>
> > > > > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan filter/extend
> > > > > > > > for ixgbevf
> > > > > > > >
> > > > > > > > ixgbevf vlan strip and extend capabilities were removed when
> > > > > > > > migrating to the bit flags implementation.
> > > > > > > >
> > > > > > > > Restoring the capbility to enable these offloads at
> > > > > > > > configuration
> > > > > > time.
> > > > > > > >
> > > > > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads
> > > > > > > > API\")
> > > > > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > > > > ---
> > > > > > > >
> > > > > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > > > > >
> > > > > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhao1, Wei]

> -----Original Message-----
> From: Ananyev, Konstantin
> Sent: Friday, March 15, 2019 9:00 PM
> To: Zhao1, Wei <wei.zhao1@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>;
> David Harton (dharton) <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 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 more
> > > 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
> >
> > 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?

Maybe hassle to certain extent for implement, I think.


> 
> >
> > >
> > >
> > > >
> > > > Maybe we can change code to
> > > > "
> > > > 	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;
> > > >
> > > > 	if (ixgbe_is_vf(dev) == 0)
> > > > 		offloads |= 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 <dharton@cisco.com>
> > > > > > > > > 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)
> > > > > > > > > <dharton@cisco.com>
> > > > > > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan
> > > > > > > > > filter/extend for ixgbevf
> > > > > > > > >
> > > > > > > > > ixgbevf vlan strip and extend capabilities were removed
> > > > > > > > > when migrating to the bit flags implementation.
> > > > > > > > >
> > > > > > > > > Restoring the capbility to enable these offloads at
> > > > > > > > > configuration
> > > > > > > time.
> > > > > > > > >
> > > > > > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx
> > > > > > > > > offloads
> > > > > > > > > API\")
> > > > > > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > > > > > ---
> > > > > > > > >
> > > > > > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > > > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > > > > > >
> > > > > > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > > > > > -		offloads |= (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

Re: [dpdk-dev] [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf

[Zhao1, Wei]

> -----Original Message-----
> From: Ananyev, Konstantin
> Sent: Friday, March 15, 2019 9:00 PM
> To: Zhao1, Wei <wei.zhao1@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>;
> David Harton (dharton) <dharton@cisco.com>
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v2] net/ixgbe: Restore vlan filter/extend for ixgbevf
> 
> 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 more
> > > 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
> >
> > 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?

Maybe hassle to certain extent for implement, I think.


> 
> >
> > >
> > >
> > > >
> > > > Maybe we can change code to
> > > > "
> > > > 	offloads |= DEV_RX_OFFLOAD_VLAN_FILTER;
> > > >
> > > > 	if (ixgbe_is_vf(dev) == 0)
> > > > 		offloads |= 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 <dharton@cisco.com>
> > > > > > > > > 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)
> > > > > > > > > <dharton@cisco.com>
> > > > > > > > > Subject: [PATCH v2] net/ixgbe: Restore vlan
> > > > > > > > > filter/extend for ixgbevf
> > > > > > > > >
> > > > > > > > > ixgbevf vlan strip and extend capabilities were removed
> > > > > > > > > when migrating to the bit flags implementation.
> > > > > > > > >
> > > > > > > > > Restoring the capbility to enable these offloads at
> > > > > > > > > configuration
> > > > > > > time.
> > > > > > > > >
> > > > > > > > > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx
> > > > > > > > > offloads
> > > > > > > > > API\")
> > > > > > > > > Signed-off-by: David Harton <dharton@cisco.com>
> > > > > > > > > ---
> > > > > > > > >
> > > > > > > > > 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 = 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 == ixgbe_mac_82598EB)
> > > > > > > > >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > > > > > > > >
> > > > > > > > > -	if (ixgbe_is_vf(dev) == 0)
> > > > > > > > > -		offloads |= (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

[dpdk-dev] [PATCH v3] net/ixgbe: Restore vlan filter/extend for ixgbevf

[David Harton]

ixgbevf vlan strip and extend capabilities were removed when
migrating to the bit flags implementation.

Restoring the capbility to enable these offloads at
configuration time.

Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
Signed-off-by: David Harton <dharton@cisco.com>
---

v3: restored ixgbe_is_vf() and only restore VLAN_FILTER
v2: removed unused function ixgbe_is_vf()

 drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
index e92a70fb3..e71d3c188 100644
--- a/drivers/net/ixgbe/ixgbe_rxtx.c
+++ b/drivers/net/ixgbe/ixgbe_rxtx.c
@@ -2853,14 +2853,14 @@ 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_SCATTER;
 
 	if (hw->mac.type == ixgbe_mac_82598EB)
 		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
 
 	if (ixgbe_is_vf(dev) == 0)
-		offloads |= (DEV_RX_OFFLOAD_VLAN_FILTER |
-			     DEV_RX_OFFLOAD_VLAN_EXTEND);
+		offloads |= DEV_RX_OFFLOAD_VLAN_EXTEND;
 
 	/*
 	 * RSC is only supported by 82599 and x540 PF devices in a non-SR-IOV
-- 
2.19.1

[dpdk-dev] [PATCH v3] net/ixgbe: Restore vlan filter/extend for ixgbevf

[David Harton]

ixgbevf vlan strip and extend capabilities were removed when
migrating to the bit flags implementation.

Restoring the capbility to enable these offloads at
configuration time.

Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
Signed-off-by: David Harton <dharton@cisco.com>
---

v3: restored ixgbe_is_vf() and only restore VLAN_FILTER
v2: removed unused function ixgbe_is_vf()

 drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
index e92a70fb3..e71d3c188 100644
--- a/drivers/net/ixgbe/ixgbe_rxtx.c
+++ b/drivers/net/ixgbe/ixgbe_rxtx.c
@@ -2853,14 +2853,14 @@ 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_SCATTER;
 
 	if (hw->mac.type == ixgbe_mac_82598EB)
 		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
 
 	if (ixgbe_is_vf(dev) == 0)
-		offloads |= (DEV_RX_OFFLOAD_VLAN_FILTER |
-			     DEV_RX_OFFLOAD_VLAN_EXTEND);
+		offloads |= DEV_RX_OFFLOAD_VLAN_EXTEND;
 
 	/*
 	 * RSC is only supported by 82599 and x540 PF devices in a non-SR-IOV
-- 
2.19.1

[dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for ixgbevf

[David Harton]

ixgbevf vlan strip and extend capabilities were removed when
migrating to the bit flags implementation.

Restoring the capbility to enable the vlan strip offload at
configuration time.

Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
Signed-off-by: David Harton <dharton@cisco.com>
---

 v4: fixed commit headline/log
 v3: restored ixgbe_is_vf() and only restore VLAN_FILTER
 v2: removed unused function ixgbe_is_vf()

 drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
index e92a70fb3..e71d3c188 100644
--- a/drivers/net/ixgbe/ixgbe_rxtx.c
+++ b/drivers/net/ixgbe/ixgbe_rxtx.c
@@ -2853,14 +2853,14 @@ 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_SCATTER;
 
 	if (hw->mac.type == ixgbe_mac_82598EB)
 		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
 
 	if (ixgbe_is_vf(dev) == 0)
-		offloads |= (DEV_RX_OFFLOAD_VLAN_FILTER |
-			     DEV_RX_OFFLOAD_VLAN_EXTEND);
+		offloads |= DEV_RX_OFFLOAD_VLAN_EXTEND;
 
 	/*
 	 * RSC is only supported by 82599 and x540 PF devices in a non-SR-IOV
-- 
2.19.1

[dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for ixgbevf

[David Harton]

ixgbevf vlan strip and extend capabilities were removed when
migrating to the bit flags implementation.

Restoring the capbility to enable the vlan strip offload at
configuration time.

Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
Signed-off-by: David Harton <dharton@cisco.com>
---

 v4: fixed commit headline/log
 v3: restored ixgbe_is_vf() and only restore VLAN_FILTER
 v2: removed unused function ixgbe_is_vf()

 drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
index e92a70fb3..e71d3c188 100644
--- a/drivers/net/ixgbe/ixgbe_rxtx.c
+++ b/drivers/net/ixgbe/ixgbe_rxtx.c
@@ -2853,14 +2853,14 @@ 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_SCATTER;
 
 	if (hw->mac.type == ixgbe_mac_82598EB)
 		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
 
 	if (ixgbe_is_vf(dev) == 0)
-		offloads |= (DEV_RX_OFFLOAD_VLAN_FILTER |
-			     DEV_RX_OFFLOAD_VLAN_EXTEND);
+		offloads |= DEV_RX_OFFLOAD_VLAN_EXTEND;
 
 	/*
 	 * RSC is only supported by 82599 and x540 PF devices in a non-SR-IOV
-- 
2.19.1

Re: [dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for ixgbevf

[Zhao1, Wei]

Acked-by: Wei Zhao <wei.zhao1@intel.com>


> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Harton
> Sent: Saturday, March 16, 2019 12:09 AM
> To: dev@dpdk.org
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>;
> David Harton <dharton@cisco.com>
> Subject: [dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for ixgbevf
> 
> ixgbevf vlan strip and extend capabilities were removed when migrating to
> the bit flags implementation.
> 
> Restoring the capbility to enable the vlan strip offload at configuration time.
> 
> Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> Signed-off-by: David Harton <dharton@cisco.com>
> ---
> 
>  v4: fixed commit headline/log
>  v3: restored ixgbe_is_vf() and only restore VLAN_FILTER
>  v2: removed unused function ixgbe_is_vf()
> 
>  drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
> index e92a70fb3..e71d3c188 100644
> --- a/drivers/net/ixgbe/ixgbe_rxtx.c
> +++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> @@ -2853,14 +2853,14 @@ 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_SCATTER;
> 
>  	if (hw->mac.type == ixgbe_mac_82598EB)
>  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> 
>  	if (ixgbe_is_vf(dev) == 0)
> -		offloads |= (DEV_RX_OFFLOAD_VLAN_FILTER |
> -			     DEV_RX_OFFLOAD_VLAN_EXTEND);
> +		offloads |= DEV_RX_OFFLOAD_VLAN_EXTEND;
> 
>  	/*
>  	 * RSC is only supported by 82599 and x540 PF devices in a non-SR-
> IOV
> --
> 2.19.1

Re: [dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for ixgbevf

[Zhao1, Wei]

Acked-by: Wei Zhao <wei.zhao1@intel.com>


> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Harton
> Sent: Saturday, March 16, 2019 12:09 AM
> To: dev@dpdk.org
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>;
> David Harton <dharton@cisco.com>
> Subject: [dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for ixgbevf
> 
> ixgbevf vlan strip and extend capabilities were removed when migrating to
> the bit flags implementation.
> 
> Restoring the capbility to enable the vlan strip offload at configuration time.
> 
> Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> Signed-off-by: David Harton <dharton@cisco.com>
> ---
> 
>  v4: fixed commit headline/log
>  v3: restored ixgbe_is_vf() and only restore VLAN_FILTER
>  v2: removed unused function ixgbe_is_vf()
> 
>  drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
> index e92a70fb3..e71d3c188 100644
> --- a/drivers/net/ixgbe/ixgbe_rxtx.c
> +++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> @@ -2853,14 +2853,14 @@ 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_SCATTER;
> 
>  	if (hw->mac.type == ixgbe_mac_82598EB)
>  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> 
>  	if (ixgbe_is_vf(dev) == 0)
> -		offloads |= (DEV_RX_OFFLOAD_VLAN_FILTER |
> -			     DEV_RX_OFFLOAD_VLAN_EXTEND);
> +		offloads |= DEV_RX_OFFLOAD_VLAN_EXTEND;
> 
>  	/*
>  	 * RSC is only supported by 82599 and x540 PF devices in a non-SR-
> IOV
> --
> 2.19.1

Re: [dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for ixgbevf

[Zhang, Qi Z]

> -----Original Message-----
> From: Zhao1, Wei
> Sent: Monday, March 18, 2019 11:05 AM
> To: David Harton <dharton@cisco.com>; dev@dpdk.org
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>
> Subject: RE: [dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for ixgbevf
> 
> Acked-by: Wei Zhao <wei.zhao1@intel.com>

Applied to dpdk-next-net-intel.

Thanks
Qi
> 
> 
> > -----Original Message-----
> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Harton
> > Sent: Saturday, March 16, 2019 12:09 AM
> > To: dev@dpdk.org
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>;
> > David Harton <dharton@cisco.com>
> > Subject: [dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for
> > ixgbevf
> >
> > ixgbevf vlan strip and extend capabilities were removed when migrating
> > to the bit flags implementation.
> >
> > Restoring the capbility to enable the vlan strip offload at configuration time.
> >
> > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> > Signed-off-by: David Harton <dharton@cisco.com>
> > ---
> >
> >  v4: fixed commit headline/log
> >  v3: restored ixgbe_is_vf() and only restore VLAN_FILTER
> >  v2: removed unused function ixgbe_is_vf()
> >
> >  drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c
> > b/drivers/net/ixgbe/ixgbe_rxtx.c index e92a70fb3..e71d3c188 100644
> > --- a/drivers/net/ixgbe/ixgbe_rxtx.c
> > +++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> > @@ -2853,14 +2853,14 @@ 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_SCATTER;
> >
> >  	if (hw->mac.type == ixgbe_mac_82598EB)
> >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> >
> >  	if (ixgbe_is_vf(dev) == 0)
> > -		offloads |= (DEV_RX_OFFLOAD_VLAN_FILTER |
> > -			     DEV_RX_OFFLOAD_VLAN_EXTEND);
> > +		offloads |= DEV_RX_OFFLOAD_VLAN_EXTEND;
> >
> >  	/*
> >  	 * RSC is only supported by 82599 and x540 PF devices in a non-SR-
> > IOV
> > --
> > 2.19.1

Re: [dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for ixgbevf

[Zhang, Qi Z]

> -----Original Message-----
> From: Zhao1, Wei
> Sent: Monday, March 18, 2019 11:05 AM
> To: David Harton <dharton@cisco.com>; dev@dpdk.org
> Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>
> Subject: RE: [dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for ixgbevf
> 
> Acked-by: Wei Zhao <wei.zhao1@intel.com>

Applied to dpdk-next-net-intel.

Thanks
Qi
> 
> 
> > -----Original Message-----
> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Harton
> > Sent: Saturday, March 16, 2019 12:09 AM
> > To: dev@dpdk.org
> > Cc: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>;
> > David Harton <dharton@cisco.com>
> > Subject: [dpdk-dev] [PATCH v4] net/ixgbe: Restore vlan filter for
> > ixgbevf
> >
> > ixgbevf vlan strip and extend capabilities were removed when migrating
> > to the bit flags implementation.
> >
> > Restoring the capbility to enable the vlan strip offload at configuration time.
> >
> > Fixes: ec3b1124d14d (\"net/ixgbe: convert to new Rx offloads API\")
> > Signed-off-by: David Harton <dharton@cisco.com>
> > ---
> >
> >  v4: fixed commit headline/log
> >  v3: restored ixgbe_is_vf() and only restore VLAN_FILTER
> >  v2: removed unused function ixgbe_is_vf()
> >
> >  drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c
> > b/drivers/net/ixgbe/ixgbe_rxtx.c index e92a70fb3..e71d3c188 100644
> > --- a/drivers/net/ixgbe/ixgbe_rxtx.c
> > +++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> > @@ -2853,14 +2853,14 @@ 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_SCATTER;
> >
> >  	if (hw->mac.type == ixgbe_mac_82598EB)
> >  		offloads |= DEV_RX_OFFLOAD_VLAN_STRIP;
> >
> >  	if (ixgbe_is_vf(dev) == 0)
> > -		offloads |= (DEV_RX_OFFLOAD_VLAN_FILTER |
> > -			     DEV_RX_OFFLOAD_VLAN_EXTEND);
> > +		offloads |= DEV_RX_OFFLOAD_VLAN_EXTEND;
> >
> >  	/*
> >  	 * RSC is only supported by 82599 and x540 PF devices in a non-SR-
> > IOV
> > --
> > 2.19.1