On Wed, Nov 21, 2018 at 07:15:41PM -0200, Marcelo Ricardo Leitner wrote: > On Wed, Nov 21, 2018 at 03:51:24AM +0100, Pablo Neira Ayuso wrote: [...] > > diff --git a/net/sched/cls_flower.c b/net/sched/cls_flower.c > > index d2971fbfc3d9..8898943b8ee6 100644 > > --- a/net/sched/cls_flower.c > > +++ b/net/sched/cls_flower.c > > @@ -382,7 +382,7 @@ static int fl_hw_replace_filter(struct tcf_proto *tp, > > bool skip_sw = tc_skip_sw(f->flags); > > int err; > > > > - cls_flower.rule = flow_rule_alloc(); > > + cls_flower.rule = flow_rule_alloc(tcf_exts_num_actions(&f->exts)); > > As previous patch did: > -struct flow_rule *flow_rule_alloc(void); > +struct flow_rule *flow_rule_alloc(unsigned int num_actions); > the build is broken without this change (bisect-ability). > (applies to similar lines too)
Sorry about this, I'll fix this, thanks Marcelo.