[PATCH] net: add macro for VLAN header length
    Thomas Monjalon 
    thomas at monjalon.net
       
    Wed Nov 17 00:14:10 CET 2021
    
    
  
10/11/2021 18:40, Ferruh Yigit:
> Multiple drivers are defining macros for VLAN header length, to remove
> the redundancy defining macro in the ether header.
> And updated drivers to use the new macro.
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit at intel.com>
[...]
> --- a/lib/net/rte_ether.h
> +++ b/lib/net/rte_ether.h
> +#define RTE_VLAN_HLEN       4
Please could you add a doxygen comment for this constant?
> +/** Maximum VLAN frame length, including CRC. */
>  #define RTE_ETHER_MAX_VLAN_FRAME_LEN \
> -	(RTE_ETHER_MAX_LEN + 4)
> -	/**< Maximum VLAN frame length, including CRC. */
> +	(RTE_ETHER_MAX_LEN + RTE_VLAN_HLEN)
What about QinQ?
    
    
More information about the dev
mailing list