On 4/5/2019 1:04 PM, Nithin Dabilpuram wrote: > From: Nithin Dabilpuram <[email protected]> > > Tx VLAN & QinQ insert enable need not depend on > Rx VLAN offload ETH_VLAN_EXTEND_OFFLOAD. For Tx VLAN > insert enable, error check is now to see if QinQ was enabled > but only single VLAN id is set. > > Fixes: 6a34f91690d0 ("app/testpmd: fix error message when setting Tx VLAN") > Cc: [email protected] > > Signed-off-by: Nithin Dabilpuram <[email protected]> > Acked-by: Bernard Iremonger <[email protected]> > --- > v4: > * Resend v3 from different mailserver to avoid > CRLF > v3: > * Add back error check in tx_vlan_set() to check if QinQ is > already enabled. Also fix headline. > v2: > * Split change into two seperate patches as suggested. >
Hi Nithin, I just merged the v3 and about to send the mail :) What is different in v4? avoid CRLF?

