Re: [PATCH net-next 1/2] cxgb4/cxgb4vf: Add support for ndo_set_vf_vlan

[Hariprasad Shenai <hariprasad@chelsio.com>]

On Wednesday, August 08/24/16, 2016 at 07:08:14 +0000, Yuval Mintz wrote:
> > @@ -1202,6 +1202,10 @@ int t4vf_eth_xmit(struct sk_buff *skb, struct
> > net_device *dev)
> >  	BUG_ON(qidx >= pi->nqsets);
> >  	txq = &adapter->sge.ethtxq[pi->first_qset + qidx];
> > 
> > +	if (pi->vlan_id && !skb_vlan_tag_present(skb))
> > +		__vlan_hwaccel_put_tag(skb, cpu_to_be16(ETH_P_8021Q),
> > +				       pi->vlan_id);
> > +
> 
> So it's a purely SW implementation of the feature on the VF side?
> Does the HW enforces the configuration in any way on the VF?
Basically the PF driver passes the VLAN ID it got through ndo_set_vf_vlan
to the VF driver. And then the VF driver reads it and requests hardware to
tag it.

> 
> Also, looks like an already tagged packet would be processed with
> the original vlan-id [instead of the one of PF has provided].
> Is that intentional?
No, this isn't intentional. I thought VST and VGT cannot co-exist.
What should be the behavior? 

Thanks,
Hariprasad Shenai