> On Dec 8, 2016, at 9:38 PM, Andy Zhou <[email protected]> wrote: > > > > On Wed, Dec 7, 2016 at 4:31 PM, Jarno Rajahalme <[email protected] > <mailto:[email protected]>> wrote: > This patch allows openvswitch kernel module in the OVS tree to be > compiled against the current net-next Linux kernel. The changes are > due to these upstream commits: > > 56989f6d856 ("genetlink: mark families as __ro_after_init") > 489111e5c25 ("genetlink: statically initialize families") > a07ea4d9941 ("genetlink: no longer support using static family IDs") > > struct genl_family initialization is changed be completely static and > to include the new (in Linux 4.6) __ro_after_init attribute. Compat > code defines it as an empty macro if not defined already. > > GENL_ID_GENERATE is no longer defined, but since it was defined as 0, > it is safe to drop it from all initializers also on older Linux > versions. > > Tested with current Linux net-next (4.9) and 3.16. > > Signed-off-by: Jarno Rajahalme <[email protected] <mailto:[email protected]>> > > Tested building linux 4.0-4.9. > > Acked-by: Andy Zhou <[email protected] <mailto:[email protected]>> >
Thanks! > When compiling with 4.0, I got the following warnning. It does not seem to be > related to this patch. > > /home/azhou/projs/ovs/datapath/linux/datapath.c: In function > ‘ovs_flow_cmd_set’: > /home/azhou/projs/ovs/datapath/linux/datapath.c:1232:1: warning: the frame > size of 1040 bytes is larger than 1024 bytes [-Wframe-larger-than=] > } > ^ Right, not related. Upstream commit 190aa3e778 (“openvswitch: Fix Frame-size larger than 1024 bytes warning.”) fixes similar problem for ovs_flow_cmd_new(). Jarno _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
