On Fri, Oct 31, 2014 at 4:29 PM, Ben Pfaff <b...@nicira.com> wrote:
> packets.h uses a LITTLE_ENDIAN macro for Geneve:
>
>     struct geneve_opt {
>         ovs_be16  opt_class;
>         uint8_t   type;
>     #ifdef LITTLE_ENDIAN
>         uint8_t   length:5;
>         uint8_t   r3:1;
>         uint8_t   r2:1;
>         uint8_t   r1:1;
>     #else
>         uint8_t   r1:1;
>         uint8_t   r2:1;
>         uint8_t   r3:1;
>         uint8_t   length:5;
>     #endif
>         uint8_t   opt_data[];
>     };
>
> I don't think we have such a macro.  The rest of the userspace code
> tests for WORDS_BIGENDIAN.

It is defined on my machine but I agree that it is not portable. I'll
send out a patch to fix it.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to