[ovs-dev] [PATCH v2] datapath-windows: Fix GENEVE option header

Guru Shetty guru at ovn.org
Mon Mar 6 17:19:02 UTC 2017


On 6 March 2017 at 08:16, Alin Serdean <aserdean at cloudbasesolutions.com>
wrote:

> From: Alin Serdean <aserdean at 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 <aserdean at cloudbasesolutions.com>
> Acked-by: Yin Lin <linyi at vmware.com>
>
Applied, thanks.


> ---
> v2: Change commit title and message as suggested by:
>     Guru Shetty <guru at 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
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>


More information about the dev mailing list