On Fri, 14 Dec 2018 16:09:33 +0000
Ferruh Yigit <ferruh.yi...@intel.com> wrote:
> On 12/14/2018 1:26 AM, Stephen Hemminger wrote:
> > In some cases, VLAN stripping is not desireable. If necessary
> > re-insert stripped vlan tag.
> >
> > Signed-off-by: Stephen Hemminger <sthem...@microsoft.com>
>
> <...>
>
> > @@ -501,6 +501,14 @@ static void hn_rxpkt(struct hn_rx_queue *rxq, struct
> > hn_rx_bufinfo *rxb,
> > if (info->vlan_info != HN_NDIS_VLAN_INFO_INVALID) {
> > m->vlan_tci = info->vlan_info;
> > m->ol_flags |= PKT_RX_VLAN_STRIPPED | PKT_RX_VLAN;
> > +
> > + /* NDIS always strips tag, put it back if necessary */
> > + if (!hv->vlan_strip && rte_vlan_insert(&m)) {
>
> Should 'PKT_RX_VLAN_STRIPPED' flag removed when vlan inserted back?
This is already done by rte_vlan_insert()
