[PATCH v2 2/4] ethdev: add VXLAN last reserved field

[rongwei liu]

BR

Rongwei



________________________________
From: Thomas Monjalon <thomas at monjalon.net>
Sent: Tuesday, June 4, 2024 19:59
To: rongwei liu <rongweil at nvidia.com>
Cc: dev at dpdk.org <dev at dpdk.org>; Matan Azrad <matan at nvidia.com>; Slava Ovsiienko <viacheslavo at nvidia.com>; Ori Kam <orika at nvidia.com>; Suanming Mou <suanmingm at nvidia.com>; Dariusz Sosnowski <dsosnowski at nvidia.com>; Ferruh Yigit <ferruh.yigit at amd.com>; Andrew Rybchenko <andrew.rybchenko at oktetlabs.ru>
Subject: Re: [PATCH v2 2/4] ethdev: add VXLAN last reserved field

External email: Use caution opening links or attachments


27/05/2024 04:47, Rongwei Liu:
> Add VXLAN last reserved byte in rte flow packet field.

You should squash addition in testpmd in the same patch.
Adding a field in rte_flow.h should be atomic with testpmd update.

Sure. Squash into one commit.

> @@ -2428,6 +2428,7 @@ enum rte_flow_field_id {
>       RTE_FLOW_FIELD_ESP_SEQ_NUM,     /**< ESP Sequence Number. */
>       RTE_FLOW_FIELD_ESP_PROTO,       /**< ESP next protocol value. */
>       RTE_FLOW_FIELD_RANDOM,          /**< Random value. */
> +     RTE_FLOW_FIELD_VXLAN_RSVD1,     /**< VXLAN reserved1. */

It isn't clear what reserved1 is.
You should say it is the last byte of the VXLAN header.

ACK.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mails.dpdk.org/archives/dev/attachments/20240604/4446a3c5/attachment.htm>