[dpdk-dev] [PATCH] app/testpmd: check Rx VLAN offload flag to print VLAN TCI

Ferruh Yigit ferruh.yigit at intel.com
Mon Oct 1 16:01:40 CEST 2018


On 9/26/2018 4:06 AM, John Daley wrote:
> From: Hyong Youb Kim <hyonkim at 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.matz at 6wind.com
> 
> Signed-off-by: Hyong Youb Kim <hyonkim at cisco.com>
> Reviewed-by: John Daley <johndale at 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?



More information about the dev mailing list