On Fri, Apr 07, 2017 at 05:56:40PM -0300, Flavio Leitner wrote: > On Fri, Apr 07, 2017 at 04:12:48PM +0300, Roi Dayan wrote: > > From: Paul Blakey <[email protected]> [...] > > > > +static const struct nl_policy act_policy[] = { > > + [TCA_ACT_KIND] = { .type = NL_A_STRING, .optional = false, }, > > + [TCA_ACT_COOKIE] = { .type = NL_A_UNSPEC, .optional = true, }, > > + [TCA_ACT_OPTIONS] = { .type = NL_A_NESTED, .optional = false, }, > > + [TCA_ACT_STATS] = { .type = NL_A_NESTED, .optional = false, }, > > > This patch breaks with not so recent kernels because some defines > are not available yet. E.g.: TCA_ACT_COOKIE included in v4.11-rc1. > > I'd suggest to look at lib/netlink-conntrack.c as an example including > missing defines to allow userspace to compile with older kernels.
The issue is fixed by patch 23 in this series, but it should have been either before or together with this patch, otherwise it may break bisection in the future. -- Flavio _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
