18/04/2019 09:38, Vivek Kumar Sharma:
> >> From: Vivek Sharma <viveksha...@marvell.com>
> >>
> >> Enable missing support for QinQ strip rx offload
> >> in vlan offload set/get methods.
> >>
> >> Fixes: cc9d0456b870 ("i40e: support double vlan stripping and insertion")
> >> Cc: sta...@dpdk.org
>
> >Not sure it is a fix.
> >The commit mentioned above allows some kind of offload config.
> >You are extending the offload config with support in
> >rte_eth_dev_set_vlan_offload().
>
> DEV_RX_OFFLOAD_QINQ_STRIP was introduced in cc9d0456b870 ("i40e: support
> double vlan stripping and insertion"). But, the means to utilize this
> capability by enabling this rx offload for an ethdev was missing from the
> patch. The current patch fixes that missing functionality so that user can
> enable QinQ strip rx offload for capable devices.
It was possible to utilize the offload by setting the flag in mbuf.
You are adding a way to configure it at port level. So it is not a fix.
