On Fri, Apr 09, 2021 at 09:35:04AM -0400, Aaron Conole wrote: > +static int > +probe_insert_ct_state_rule(int ifindex, uint16_t ct_state, struct tcf_id *id) > +{ > + int prio = TC_RESERVED_PRIORITY_MAX + 1; > + struct tc_flower flower; > + > + memset(&flower, 0, sizeof flower); > + flower.key.ct_state = ct_state; > + flower.mask.ct_state = ct_state; > + flower.tc_policy = TC_POLICY_SKIP_HW;
This field is introduced in patch 3/3 here and with that, if doing a bisect or so, the line above will fail to compile. It's best if patch 3 is moved to the front of the patchset, like it is on the 2.13 patchset. Btw, I liked how the backports are going to be probe for the new features but not use them. That might give useful debugging information later on. Marcelo _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev