[dpdk-dev] mbuf: fix comment about vxlan destination port used

Message ID 1499437060-7894-1-git-send-email-cian.ferriter@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation fail Compilation issues

Commit Message

Cian Ferriter July 7, 2017, 2:17 p.m. UTC
  IANA assigns a destination port of 4789 for the VXLAN in the Service
Name and Transport Protocol Port Number Registry. This is mentioned in
RFC 7348.

Signed-off-by: Cian Ferriter <cian.ferriter@intel.com>
---
 lib/librte_mbuf/rte_mbuf_ptype.h |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
  

Comments

Olivier Matz July 10, 2017, 12:33 p.m. UTC | #1
On Fri,  7 Jul 2017 15:17:40 +0100, Cian Ferriter <cian.ferriter@intel.com> wrote:
> IANA assigns a destination port of 4789 for the VXLAN in the Service
> Name and Transport Protocol Port Number Registry. This is mentioned in
> RFC 7348.
> 
> Signed-off-by: Cian Ferriter <cian.ferriter@intel.com>

Fixes: f295a00a2b44 ("mbuf: add definitions of unified packet types")
CC: stable@dpdk.org

Acked-by: Olivier Matz <olivier.matz@6wind.com>
  
Thomas Monjalon July 19, 2017, 7:04 a.m. UTC | #2
10/07/2017 15:33, Olivier Matz:
> On Fri,  7 Jul 2017 15:17:40 +0100, Cian Ferriter <cian.ferriter@intel.com> wrote:
> > IANA assigns a destination port of 4789 for the VXLAN in the Service
> > Name and Transport Protocol Port Number Registry. This is mentioned in
> > RFC 7348.
> > 
> > Signed-off-by: Cian Ferriter <cian.ferriter@intel.com>
> 
> Fixes: f295a00a2b44 ("mbuf: add definitions of unified packet types")
> CC: stable@dpdk.org
> 
> Acked-by: Olivier Matz <olivier.matz@6wind.com>

Applied, thanks
  

Patch

diff --git a/lib/librte_mbuf/rte_mbuf_ptype.h b/lib/librte_mbuf/rte_mbuf_ptype.h
index a3269c4..acd70bb 100644
--- a/lib/librte_mbuf/rte_mbuf_ptype.h
+++ b/lib/librte_mbuf/rte_mbuf_ptype.h
@@ -341,11 +341,11 @@ 
  * Packet format:
  * <'ether type'=0x0800
  * | 'version'=4, 'protocol'=17
- * | 'destination port'=4798>
+ * | 'destination port'=4789>
  * or,
  * <'ether type'=0x86DD
  * | 'version'=6, 'next header'=17
- * | 'destination port'=4798>
+ * | 'destination port'=4789>
  */
 #define RTE_PTYPE_TUNNEL_VXLAN              0x00003000
 /**