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

Reply via email to