> -----Original Message----- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of John Daley (johndale) > Sent: Tuesday, July 19, 2016 6:44 PM > To: dev at dpdk.org > Cc: bruce.richardson at intel.com; John Daley (johndale) <johndale at > cisco.com> > Subject: [dpdk-dev] [PATCH] net/enic: heed VLAN strip flag in device > configure function > > The configure function enicpmd_dev_configure() was not paying attention to > the rxmode VLAN strip bit. Set the VLAN strip mode according to the bit. > > Fixes: fefed3d1e62c ("enic: new driver") > > Signed-off-by: John Daley <johndale at cisco.com> > --- > > Although not critical, it is low risk and would be nice to have in 16_07. > > drivers/net/enic/enic_ethdev.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/enic/enic_ethdev.c > b/drivers/net/enic/enic_ethdev.c index 59082d2..47b07c9 100644 > --- a/drivers/net/enic/enic_ethdev.c > +++ b/drivers/net/enic/enic_ethdev.c > @@ -356,6 +356,7 @@ static int enicpmd_dev_configure(struct rte_eth_dev > *eth_dev) > eth_dev->data->dev_conf.rxmode.split_hdr_size); > } > > + enicpmd_vlan_offload_set(eth_dev, ETH_VLAN_STRIP_MASK); > enic->hw_ip_checksum = eth_dev->data- > >dev_conf.rxmode.hw_ip_checksum; > return 0; > } > -- > 2.7.0
Reviewed-by: David Harton <dharton at cisco.com> Tested-by: David Harton <dharton at cisco.com>
