ethdev: fix ESP packet type description

Message ID 20231025203215.1546091-1-akozyrev@nvidia.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series ethdev: fix ESP packet type description |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-compile-amd64-testing success Testing PASS
ci/github-robot: build success github build: passed
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-unit-amd64-testing success Testing PASS
ci/iol-sample-apps-testing success Testing PASS
ci/intel-Functional success Functional PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-compile-arm64-testing success Testing PASS

Commit Message

Alexander Kozyrev Oct. 25, 2023, 8:32 p.m. UTC
  The correct protocol number for ESP (IP Encapsulating Security Payload)
packet type is 50. 51 is IPSec AH (Authentication Header).

Fixes: 1e84afd3906b ("mbuf: add security crypto flags and fields")
Signed-off-by: Alexander Kozyrev <akozyrev@nvidia.com>
---
 lib/mbuf/rte_mbuf_ptype.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Morten Brørup Oct. 26, 2023, 5:49 a.m. UTC | #1
> From: Alexander Kozyrev [mailto:akozyrev@nvidia.com]
> Sent: Wednesday, 25 October 2023 22.32
> 
> The correct protocol number for ESP (IP Encapsulating Security Payload)
> packet type is 50. 51 is IPSec AH (Authentication Header).
> 
> Fixes: 1e84afd3906b ("mbuf: add security crypto flags and fields")
> Signed-off-by: Alexander Kozyrev <akozyrev@nvidia.com>
> ---

Reviewed-by: Morten Brørup <mb@smartsharesystems.com>
  
Thomas Monjalon Nov. 27, 2023, 3:32 p.m. UTC | #2
>> The correct protocol number for ESP (IP Encapsulating Security Payload)
>> packet type is 50. 51 is IPSec AH (Authentication Header).
>> 
>> Fixes: 1e84afd3906b ("mbuf: add security crypto flags and fields")
>> Signed-off-by: Alexander Kozyrev <akozyrev@nvidia.com>
> Reviewed-by: Morten Brørup <mb@smartsharesystems.com>

Applied, thanks
  

Patch

diff --git a/lib/mbuf/rte_mbuf_ptype.h b/lib/mbuf/rte_mbuf_ptype.h
index 17a2dd3576..f2276e2909 100644
--- a/lib/mbuf/rte_mbuf_ptype.h
+++ b/lib/mbuf/rte_mbuf_ptype.h
@@ -419,10 +419,10 @@  extern "C" {
  *
  * Packet format:
  * <'ether type'=0x0800
- * | 'version'=4, 'protocol'=51>
+ * | 'version'=4, 'protocol'=50>
  * or,
  * <'ether type'=0x86DD
- * | 'version'=6, 'next header'=51>
+ * | 'version'=6, 'next header'=50>
  */
 #define RTE_PTYPE_TUNNEL_ESP                0x00009000
 /**