Subject: Re: [net-next PATCH v2 5/6] net sched: skbedit action fix late binding

From: Jamal Hadi Salim <[email-masked]>
Date: Mon, 9 May 2016 07:53:43 -0400

On 16-05-08 11:28 PM, Cong Wang wrote:

>> +       /* if action exists and this is a late filter bind, no need
>> +        * to continue processing
>> +       */
>
> This comment looks useless, at least for me, because the code
> is already clear.
>

I will get rid of it. Note: I added it to one patch only incase
the feature of late binding was foreign to some people.

>> +       if (aexists && bind)
>> +                       return 0;
> One extra tab?

thanks for catching it.

cheers,
jamal