Hi Nithin
> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Thursday, March 21, 2019 5:11 PM
> To: Nithin Kumar Dabilpuram <ndabilpu...@marvell.com>; Lu, Wenzhuo
> <wenzhuo...@intel.com>; Wu, Jingjing <jingjing...@intel.com>; Iremonger,
> Bernard <bernard.iremon...@intel.com>
> Cc: dev@dpdk.org; Wang, Xiao W <xiao.w.w...@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v2 1/2] app/testpmd: fix tx vlan and qinq
> dependency
>
> On 3/18/2019 9:56 AM, Nithin Kumar Dabilpuram wrote:
> > Tx VLAN & QinQ insert enable need not depend on Rx VLAN offload
> > ETH_VLAN_EXTEND_OFFLOAD.
> >
> > Fixes: 6a34f91690d0 ("app/testpmd: fix error message when setting Tx
> > VLAN")
> > Cc: xiao.w.w...@intel.com
> >
> > Signed-off-by: Nithin Dabilpuram <ndabilpu...@marvell.com>
>
> <...>
>
> > @@ -2963,11 +2962,6 @@ tx_vlan_set(portid_t port_id, uint16_t vlan_id)
> > if (vlan_id_is_invalid(vlan_id))
> > return;
> >
> > - vlan_offload = rte_eth_dev_get_vlan_offload(port_id);
> > - if (vlan_offload & ETH_VLAN_EXTEND_OFFLOAD) {
> > - printf("Error, as QinQ has been enabled.\n");
> > - return;
> > - }
>
> It looks like you didn't take account comment on this in previous version, can
> you please check it?
./dpdk/devtools/check-git-log.sh -1
Wrong headline lowercase:
app/testpmd: fix tx vlan and qinq dependency
otherwise
acked-by: Bernard Iremonger <bernard.iremon...@intel.com>
