<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div id="Signature">
<p>BR</p>
<p>Rongwei</p>
<p> </p>
</div>
<div id="appendonsend"></div>
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<hr style="display: inline-block; width: 98%;">
<div id="divRplyFwdMsg" dir="ltr"><span style="font-family: Calibri, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);"><b>From:</b> Thomas Monjalon <thomas@monjalon.net><br>
<b>Sent:</b> Tuesday, June 4, 2024 20:19<br>
<b>To:</b> rongwei liu <rongweil@nvidia.com><br>
<b>Cc:</b> dev@dpdk.org <dev@dpdk.org>; Matan Azrad <matan@nvidia.com>; Slava Ovsiienko <viacheslavo@nvidia.com>; Ori Kam <orika@nvidia.com>; Suanming Mou <suanmingm@nvidia.com>; Dariusz Sosnowski <dsosnowski@nvidia.com><br>
<b>Subject:</b> Re: [PATCH v2 4/4] net/mlx5: implement VXLAN last reserved modification</span>
<div> </div>
</div>
<div class="elementToProof" style="font-size: 11pt;">External email: Use caution opening links or attachments<br>
<br>
<br>
27/05/2024 04:47, Rongwei Liu:<br>
> --- a/doc/guides/nics/mlx5.rst<br>
> +++ b/doc/guides/nics/mlx5.rst<br>
> @@ -685,6 +685,11 @@ Limitations<br>
> - Second tunnel fields are not supported.<br>
><br>
> - Encapsulation levels greater than ``2`` are not supported.<br>
> + - Modification of the VXLAN header is supported with below limitations:<br>
> +<br>
> + - Only in HW steering (``dv_flow_en=2``).<br>
> + - Support VNI and rsvd1 modifications for traffic with default VXLAN/VXLAN-GPE/VXLAN-GBP<br>
> + UDP destination port.<br>
<br>
Don't add at the end of bullet lists without trying to order.<br>
It is limitation for a specific protocol, it should go above with other protocols,<br>
between "Modification of VXLAN network" and GENEVE limitations.<br>
<br>
"rsvd1" should be replaced with "last reserved byte".</div>
<div class="elementToProof" style="font-size: 11pt; color: rgb(0, 0, 0);">Ok.</div>
<div class="elementToProof" style="font-size: 11pt;"><br>
What do you mean with "default VXLAN/VXLAN-GPE/VXLAN-GBP UDP destination port"?<br>
Is it port 4789 for VXLAN and VXLAN-GBP, and port 4790 for VXLAN-GPE?<br>
We need to be more specific probably.</div>
<div class="elementToProof" style="font-size: 11pt;"><br>
</div>
<div class="elementToProof" style="font-size: 11pt; color: rgb(0, 0, 0);">Will change to the exact number.</div>
<div class="elementToProof" style="font-size: 11pt;"><br>
<br>
<br>
</div>
</body>
</html>