On Wed, Feb 15, 2017 at 8:15 PM, David Miller <da...@davemloft.net> wrote: > From: Or Gerlitz <ogerl...@mellanox.com> > Date: Wed, 15 Feb 2017 10:52:35 +0200 > >> @@ -194,6 +199,9 @@ static int mall_change(struct net *net, struct sk_buff >> *in_skb, >> } >> } >> >> + if (!(tc_in_hw(new->flags))) >> + new->flags |= TCA_CLS_FLAGS_NOT_IN_HW; > > Too many parenthesis, please make this: > > if (!tc_in_hw(new->flags)) > new->flags |= TCA_CLS_FLAGS_NOT_IN_HW;
OKay, Dave, I will fix that along with the other places
