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?

Reply via email to