On 6 March 2017 at 08:16, Alin Serdean <[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]>
>
Applied, thanks.


> ---
> 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