On 17 February 2017 at 04:44, Alin Serdean <[email protected]> wrote:
> At the moment on Windows we suport only LE platforms. > > The option header is currently defined in BE. > > Found while testing. > > Signed-off-by: Alin Gabriel Serdean <[email protected]> > Would you mind changing the commit message to say something like "geneve option header should be in little endian". > --- > 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
