From: Alin Serdean <aserd...@cloudbasesolutions.com>

The GENEVE option header is defined in big endian, however we support only
little endian on Windows at the moment.

This patch changes the GENEVE option header into little endian.

Found while testing.

Signed-off-by: Alin Gabriel Serdean <aserd...@cloudbasesolutions.com>
Acked-by: Yin Lin <li...@vmware.com>
---
v2: Change commit title and message as suggested by:
    Guru Shetty <g...@ovn.org>
intended for: master, branch-2.7
---
 datapath-windows/ovsext/Geneve.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/datapath-windows/ovsext/Geneve.h b/datapath-windows/ovsext/Geneve.h
index be8a834..019c0dd 100644
--- a/datapath-windows/ovsext/Geneve.h
+++ b/datapath-windows/ovsext/Geneve.h
@@ -71,10 +71,10 @@ typedef struct GeneveOptionHdr {
     UINT32   optionClass:16;
     /* Format of data contained in the option. */
     UINT32   type:8;
-    /* Reserved. */
-    UINT32   reserved:3;
     /* Length of option in int32 excluding the option header. */
     UINT32   length:5;
+    /* Reserved. */
+    UINT32   reserved:3;
 } GeneveOptionHdr;
 
 #define GENEVE_CRIT_OPT_TYPE (1 << 7)
-- 
2.10.2.windows.1
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to