Re: [ovs-dev] ofproto: Handle OpenFlow version mismatch for requestforward with groups.
Ben Pfaff writes: > On Thu, Sep 27, 2018 at 08:57:23AM -0400, Aaron Conole wrote: >> I think the issue here on the bot side is that the compiler is not able >> to deal with this initialization semantic. I'll follow up and get it >> fixed. > > I sent a fix yesterday: > https://patchwork.ozlabs.org/patch/975494/ Thanks for the quick turnaround! Since it's likely caused by a compiler bug, I also submitted a request at: https://bugzilla.redhat.com/show_bug.cgi?id=1633716 ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Re: [ovs-dev] ofproto: Handle OpenFlow version mismatch for requestforward with groups.
On Thu, Sep 27, 2018 at 08:57:23AM -0400, Aaron Conole wrote: > I think the issue here on the bot side is that the compiler is not able > to deal with this initialization semantic. I'll follow up and get it > fixed. I sent a fix yesterday: https://patchwork.ozlabs.org/patch/975494/ ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Re: [ovs-dev] ofproto: Handle OpenFlow version mismatch for requestforward with groups.
0-day Robot writes: > Bleep bloop. Greetings Ben Pfaff, I am a robot and I have tried out your > patch. > Thanks for your contribution. > > I encountered some error that I wasn't expecting. See the details below. > > > build: > /bin/sh ./libtool --tag=CC --mode=compile gcc -std=gnu99 -DHAVE_CONFIG_H > -I.-I ./include -I ./include -I ./lib -I ./lib-Wstrict-prototypes > -Wall -Wextra -Wno-sign-compare -Wpointer-arith -Wformat -Wformat-security > -Wswitch-enum -Wunused-parameter -Wbad-function-cast -Wcast-align > -Wstrict-prototypes -Wold-style-definition -Wmissing-prototypes > -Wmissing-field-initializers -fno-strict-aliasing -Wshadow > -Wno-null-pointer-arithmetic -Werror -Werror -g -O2 -MT > ofproto/ofproto_libofproto_la-ofproto.lo -MD -MP -MF > ofproto/.deps/ofproto_libofproto_la-ofproto.Tpo -c -o > ofproto/ofproto_libofproto_la-ofproto.lo `test -f 'ofproto/ofproto.c' || echo > './'`ofproto/ofproto.c > libtool: compile: gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I ./include -I > ./include -I ./lib -I ./lib -Wstrict-prototypes -Wall -Wextra > -Wno-sign-compare -Wpointer-arith -Wformat -Wformat-security -Wswitch-enum > -Wunused-parameter -Wbad-function-cast -Wcast-align -Wstrict-prototypes > -Wold-style-definition -Wmissing-prototypes -Wmissing-field-initializers > -fno-strict-aliasing -Wshadow -Wno-null-pointer-arithmetic -Werror -Werror -g > -O2 -MT ofproto/ofproto_libofproto_la-ofproto.lo -MD -MP -MF > ofproto/.deps/ofproto_libofproto_la-ofproto.Tpo -c ofproto/ofproto.c -o > ofproto/ofproto_libofproto_la-ofproto.o > ofproto/ofproto.c: In function 'ofproto_group_mod_finish': > ofproto/ofproto.c:7400:13: error: missing initializer for field 'new_buckets' > of 'struct ' [-Werror=missing-field-initializers] > .new_buckets = new_group ? _group->buckets : NULL, > ^ > In file included from ofproto/ofproto.c:48:0: > ./include/openvswitch/ofp-monitor.h:135:36: note: 'new_buckets' declared here > const struct ovs_list *new_buckets; > ^ > ofproto/ofproto.c:7401:13: error: missing initializer for field > 'group_existed' of 'struct ' [-Werror=missing-field-initializers] > .group_existed = group_collection_n(>old_groups) > 0, > ^ > In file included from ofproto/ofproto.c:48:0: > ./include/openvswitch/ofp-monitor.h:140:17: note: 'group_existed' declared > here > int group_existed; > ^ > ofproto/ofproto.c: At top level: > cc1: error: unrecognized command line option "-Wno-null-pointer-arithmetic" > [-Werror] > cc1: all warnings being treated as errors > make[2]: *** [ofproto/ofproto_libofproto_la-ofproto.lo] Error 1 > make[2]: Leaving directory > `/var/lib/jenkins/jobs/upstream_build_from_pw/workspace' > make[1]: *** [all-recursive] Error 1 > make[1]: Leaving directory > `/var/lib/jenkins/jobs/upstream_build_from_pw/workspace' > make: *** [all] Error 2 > > > Please check this out. If you feel there has been an error, please email > acon...@bytheb.org > > Thanks, > 0-day Robot I think the issue here on the bot side is that the compiler is not able to deal with this initialization semantic. I'll follow up and get it fixed. Meanwhile, I've disabled the --enable-Werror, and I'll work on pulling out the warnings that are 'new' vs. 'not-new'. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Re: [ovs-dev] ofproto: Handle OpenFlow version mismatch for requestforward with groups.
Bleep bloop. Greetings Ben Pfaff, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. build: /bin/sh ./libtool --tag=CC --mode=compile gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I ./include -I ./include -I ./lib -I ./lib-Wstrict-prototypes -Wall -Wextra -Wno-sign-compare -Wpointer-arith -Wformat -Wformat-security -Wswitch-enum -Wunused-parameter -Wbad-function-cast -Wcast-align -Wstrict-prototypes -Wold-style-definition -Wmissing-prototypes -Wmissing-field-initializers -fno-strict-aliasing -Wshadow -Wno-null-pointer-arithmetic -Werror -Werror -g -O2 -MT ofproto/ofproto_libofproto_la-ofproto.lo -MD -MP -MF ofproto/.deps/ofproto_libofproto_la-ofproto.Tpo -c -o ofproto/ofproto_libofproto_la-ofproto.lo `test -f 'ofproto/ofproto.c' || echo './'`ofproto/ofproto.c libtool: compile: gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I ./include -I ./include -I ./lib -I ./lib -Wstrict-prototypes -Wall -Wextra -Wno-sign-compare -Wpointer-arith -Wformat -Wformat-security -Wswitch-enum -Wunused-parameter -Wbad-function-cast -Wcast-align -Wstrict-prototypes -Wold-style-definition -Wmissing-prototypes -Wmissing-field-initializers -fno-strict-aliasing -Wshadow -Wno-null-pointer-arithmetic -Werror -Werror -g -O2 -MT ofproto/ofproto_libofproto_la-ofproto.lo -MD -MP -MF ofproto/.deps/ofproto_libofproto_la-ofproto.Tpo -c ofproto/ofproto.c -o ofproto/ofproto_libofproto_la-ofproto.o ofproto/ofproto.c: In function 'ofproto_group_mod_finish': ofproto/ofproto.c:7400:13: error: missing initializer for field 'new_buckets' of 'struct ' [-Werror=missing-field-initializers] .new_buckets = new_group ? _group->buckets : NULL, ^ In file included from ofproto/ofproto.c:48:0: ./include/openvswitch/ofp-monitor.h:135:36: note: 'new_buckets' declared here const struct ovs_list *new_buckets; ^ ofproto/ofproto.c:7401:13: error: missing initializer for field 'group_existed' of 'struct ' [-Werror=missing-field-initializers] .group_existed = group_collection_n(>old_groups) > 0, ^ In file included from ofproto/ofproto.c:48:0: ./include/openvswitch/ofp-monitor.h:140:17: note: 'group_existed' declared here int group_existed; ^ ofproto/ofproto.c: At top level: cc1: error: unrecognized command line option "-Wno-null-pointer-arithmetic" [-Werror] cc1: all warnings being treated as errors make[2]: *** [ofproto/ofproto_libofproto_la-ofproto.lo] Error 1 make[2]: Leaving directory `/var/lib/jenkins/jobs/upstream_build_from_pw/workspace' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/var/lib/jenkins/jobs/upstream_build_from_pw/workspace' make: *** [all] Error 2 Please check this out. If you feel there has been an error, please email acon...@bytheb.org Thanks, 0-day Robot ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev