Acked-by: Sairam Venugopal <[email protected]>




On 3/6/17, 8:16 AM, "[email protected] on behalf of Alin Serdean" 
<[email protected] on behalf of [email protected]> 
wrote:

>From: Alin Serdean <[email protected]>
>
>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 <[email protected]>
>Acked-by: Yin Lin <[email protected]>
>---
>v2: Change commit title and message as suggested by:
>    Guru Shetty <[email protected]>
>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
>[email protected]
>https://mail.openvswitch.org/mailman/listinfo/ovs-dev
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to