I got this error:
/********/
security -Wswitch-enum -Wunused-parameter -Wstrict-aliasing -Wbad-function-cast 
-Wcast-align -Wmissing-prototypes -Wmissing-field-initializers  -g -O2 -MT 
lib/bfd.o -MD -MP -MF $depbase.Tpo -c -o lib/bfd.o lib/bfd.c &&\
In file included from lib/bfd.c:27:0:
lib/flow.h:136:1: error: negative width in bit-field 'build_assert_failed'
mv -f $depbase.Tpo $depbase.Po
make[2]: *** [lib/bfd.o] Error 1
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2
/********/
Is this mean the struct flow I changed has something error?
first I thought the reason is the bit field I used in the new struct flow, but 
I change 
/*****/
u_int8_t doff:4;
    u_int8_t ack:1;
    u_int8_t rst:1;
    u_int8_t syn:1;
    u_int8_t fin:1;
/*****/
to  u_int8_t tcp_ctl; or ovs_be16 tcp_lrf;
still get the same error.
don't understand.




linaishan

From: Ben Pfaff
Date: 2014-01-25 05:31
To: linaishan
CC: discuss
Subject: Re: [ovs-discuss] the meaning of FLOW_WC_SEQ
On Fri, Jan 24, 2014 at 09:14:52PM +0800, linaishan wrote:
> I want to extend the struct flow in lib/flow.h.
> at line 130 there is a description:
> /* Remember to update FLOW_WC_SEQ when changing 'struct flow'. */
> BUILD_ASSERT_DECL(sizeof(struct flow) == sizeof(struct flow_tnl) + 160 &&
>                   FLOW_WC_SEQ == 20);
> 
> what's the meaning of FLOW_WC_SEQ?

You update it, to any other value, to get compiler errors wherever there
is code that might need to be updated.
_______________________________________________
discuss mailing list
discuss@openvswitch.org
http://openvswitch.org/mailman/listinfo/discuss

Reply via email to