On Fri, Mar 04, 2016 at 10:20:18AM -0800, Cong Wang wrote:
> On Thu, Mar 3, 2016 at 11:51 AM, Amir Vadai wrote:
> > On Thu, Mar 03, 2016 at 09:45:28AM -0800, Cong Wang wrote:
> >>
> >> So why?
> > The struct will not be used, and without exposing it, the compiler will
> > complain on code like I h
On Thu, Mar 3, 2016 at 11:51 AM, Amir Vadai wrote:
> On Thu, Mar 03, 2016 at 09:45:28AM -0800, Cong Wang wrote:
>>
>> So why?
> The struct will not be used, and without exposing it, the compiler will
> complain on code like I have in patch 9/10 ("net/mlx5e: Support offload
> cls_flower with drop a
On 16-03-03 06:55 AM, Amir Vadai wrote:
> Introduce the macros tc_no_actions and tc_for_each_action to make code
> clearer.
>
> Suggested-by: Jiri Pirko
> Signed-off-by: Amir Vadai
> ---
Acked-by: John Fastabend
This should clean up the driver implementation a bit thanks.
On Thu, Mar 03, 2016 at 09:45:28AM -0800, Cong Wang wrote:
> On Thu, Mar 3, 2016 at 6:55 AM, Amir Vadai wrote:
> > Introduce the macros tc_no_actions and tc_for_each_action to make code
> > clearer.
> >
> > Suggested-by: Jiri Pirko
> > Signed-off-by: Amir Vadai
> > ---
> > include/net/act_api.h
On Thu, Mar 3, 2016 at 6:55 AM, Amir Vadai wrote:
> Introduce the macros tc_no_actions and tc_for_each_action to make code
> clearer.
>
> Suggested-by: Jiri Pirko
> Signed-off-by: Amir Vadai
> ---
> include/net/act_api.h| 21 -
> include/net/tc_act/tc_gact.h | 4 ++-
Thu, Mar 03, 2016 at 03:55:53PM CET, a...@vadai.me wrote:
>Introduce the macros tc_no_actions and tc_for_each_action to make code
>clearer.
>
>Suggested-by: Jiri Pirko
>Signed-off-by: Amir Vadai
Acked-by: Jiri Pirko
Introduce the macros tc_no_actions and tc_for_each_action to make code
clearer.
Suggested-by: Jiri Pirko
Signed-off-by: Amir Vadai
---
include/net/act_api.h| 21 -
include/net/tc_act/tc_gact.h | 4 ++--
2 files changed, 18 insertions(+), 7 deletions(-)
diff --git a