Re: [dpdk-dev] [PATCH 1/2] mbuf: fix missing Tx offload flag name and list update

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 10/11/2018 8:34 PM, Jerin Jacob wrote:
> Fix missing PKT_TX_UDP_SEG ol_flag name and list updatation.

Hi Jerin,

Patchset looks good, only there are a few more missings, would you mind updating
them too since you already touched:

In Tx:
PKT_TX_QINQ
s/PKT_TX_VLAN_PKT/PKT_TX_VLAN

In Rx:
PKT_RX_FDIR_ID
PKT_RX_FDIR_FLX


In rte_get_tx_ol_flag_list(), in `tx_flags`, s/"PKT_TX_TUNNEL_NONE"/NULL

Thanks,
ferruh

> 
> Fixes: 6d18505efaa6 ("vhost: support UDP Fragmentation Offload")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
> ---
>  lib/librte_mbuf/rte_mbuf.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/lib/librte_mbuf/rte_mbuf.c b/lib/librte_mbuf/rte_mbuf.c
> index 5297beb51..a93995184 100644
> --- a/lib/librte_mbuf/rte_mbuf.c
> +++ b/lib/librte_mbuf/rte_mbuf.c
> @@ -408,6 +408,7 @@ const char *rte_get_tx_ol_flag_name(uint64_t mask)
>  	case PKT_TX_TUNNEL_UDP: return "PKT_TX_TUNNEL_UDP";
>  	case PKT_TX_MACSEC: return "PKT_TX_MACSEC";
>  	case PKT_TX_SEC_OFFLOAD: return "PKT_TX_SEC_OFFLOAD";
> +	case PKT_TX_UDP_SEG: return "PKT_TX_UDP_SEG";
>  	default: return NULL;
>  	}
>  }
> @@ -448,6 +449,7 @@ rte_get_tx_ol_flag_list(uint64_t mask, char *buf, size_t buflen)
>  		  "PKT_TX_TUNNEL_NONE" },
>  		{ PKT_TX_MACSEC, PKT_TX_MACSEC, NULL },
>  		{ PKT_TX_SEC_OFFLOAD, PKT_TX_SEC_OFFLOAD, NULL },
> +		{ PKT_TX_UDP_SEG, PKT_TX_UDP_SEG, NULL },
>  		{ PKT_TX_OUTER_UDP_CKSUM, PKT_TX_OUTER_UDP_CKSUM, NULL },
>  	};
>  	const char *name;
>