On 9/26/2018 4:06 AM, John Daley wrote: > From: Hyong Youb Kim <hyon...@cisco.com> > > Since the following commit, PKT_RX_VLAN indicates the presence of > mbuf's vlan_tci, not PKT_RX_VLAN_STRIPPED. > > commit 380a7aab1ae2 ("mbuf: rename deprecated VLAN flags") > Cc: olivier.m...@6wind.com > > Signed-off-by: Hyong Youb Kim <hyon...@cisco.com> > Reviewed-by: John Daley <johnd...@cisco.com> > --- > app/test-pmd/rxonly.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/app/test-pmd/rxonly.c b/app/test-pmd/rxonly.c > index a93d80612..e8d226624 100644 > --- a/app/test-pmd/rxonly.c > +++ b/app/test-pmd/rxonly.c > @@ -130,7 +130,7 @@ pkt_burst_receive(struct fwd_stream *fs) > } > if (ol_flags & PKT_RX_TIMESTAMP) > printf(" - timestamp %"PRIu64" ", mb->timestamp); > - if (ol_flags & PKT_RX_VLAN_STRIPPED) > + if (ol_flags & PKT_RX_VLAN) > printf(" - VLAN tci=0x%x", mb->vlan_tci); > if (ol_flags & PKT_RX_QINQ_STRIPPED) > printf(" - QinQ VLAN tci=0x%x, VLAN tci outer=0x%x",
Isn't same also correct for QinQ, PKT_RX_QINQ means mb->vlan_tci & mb->vlan_tci_outer are set?