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: sta...@dpdk.org
>
> Signed-off-by: Jerin Jacob <jerin.ja...@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;
>
