* [patch net-next v2] flow_offload: use flow_action_for_each in flow_action_mixed_hw_stats_types_check()
@ 2020-03-10 10:11 Jiri Pirko
2020-03-10 11:30 ` Pablo Neira Ayuso
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Jiri Pirko @ 2020-03-10 10:11 UTC (permalink / raw)
To: netdev; +Cc: davem, kuba, pablo, ecree, mlxsw
Instead of manually iterating over entries, use flow_action_for_each
helper. Move the helper and wrap it to fit to 80 cols on the way.
Signed-off-by: Jiri Pirko <jiri@resnulli.us>
---
v1->v2:
- removed action_entry init in loop
---
include/net/flow_offload.h | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/include/net/flow_offload.h b/include/net/flow_offload.h
index 64807aa03cee..891e15055708 100644
--- a/include/net/flow_offload.h
+++ b/include/net/flow_offload.h
@@ -256,6 +256,11 @@ static inline bool flow_offload_has_one_action(const struct flow_action *action)
return action->num_entries == 1;
}
+#define flow_action_for_each(__i, __act, __actions) \
+ for (__i = 0, __act = &(__actions)->entries[0]; \
+ __i < (__actions)->num_entries; \
+ __act = &(__actions)->entries[++__i])
+
static inline bool
flow_action_mixed_hw_stats_types_check(const struct flow_action *action,
struct netlink_ext_ack *extack)
@@ -267,8 +272,7 @@ flow_action_mixed_hw_stats_types_check(const struct flow_action *action,
if (flow_offload_has_one_action(action))
return true;
- for (i = 0; i < action->num_entries; i++) {
- action_entry = &action->entries[i];
+ flow_action_for_each(i, action_entry, action) {
if (i && action_entry->hw_stats_type != last_hw_stats_type) {
NL_SET_ERR_MSG_MOD(extack, "Mixing HW stats types for actions is not supported");
return false;
@@ -316,9 +320,6 @@ flow_action_basic_hw_stats_types_check(const struct flow_action *action,
return flow_action_hw_stats_types_check(action, extack, 0);
}
-#define flow_action_for_each(__i, __act, __actions) \
- for (__i = 0, __act = &(__actions)->entries[0]; __i < (__actions)->num_entries; __act = &(__actions)->entries[++__i])
-
struct flow_rule {
struct flow_match match;
struct flow_action action;
--
2.21.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [patch net-next v2] flow_offload: use flow_action_for_each in flow_action_mixed_hw_stats_types_check()
2020-03-10 10:11 [patch net-next v2] flow_offload: use flow_action_for_each in flow_action_mixed_hw_stats_types_check() Jiri Pirko
@ 2020-03-10 11:30 ` Pablo Neira Ayuso
2020-03-10 17:44 ` Edward Cree
2020-03-10 22:43 ` David Miller
2 siblings, 0 replies; 4+ messages in thread
From: Pablo Neira Ayuso @ 2020-03-10 11:30 UTC (permalink / raw)
To: Jiri Pirko; +Cc: netdev, davem, kuba, ecree, mlxsw
On Tue, Mar 10, 2020 at 11:11:57AM +0100, Jiri Pirko wrote:
> Instead of manually iterating over entries, use flow_action_for_each
> helper. Move the helper and wrap it to fit to 80 cols on the way.
>
> Signed-off-by: Jiri Pirko <jiri@resnulli.us>
Acked-by: Pablo Neira Ayuso <pablo@netfilter.org>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [patch net-next v2] flow_offload: use flow_action_for_each in flow_action_mixed_hw_stats_types_check()
2020-03-10 10:11 [patch net-next v2] flow_offload: use flow_action_for_each in flow_action_mixed_hw_stats_types_check() Jiri Pirko
2020-03-10 11:30 ` Pablo Neira Ayuso
@ 2020-03-10 17:44 ` Edward Cree
2020-03-10 22:43 ` David Miller
2 siblings, 0 replies; 4+ messages in thread
From: Edward Cree @ 2020-03-10 17:44 UTC (permalink / raw)
To: Jiri Pirko, netdev; +Cc: davem, kuba, pablo, mlxsw
On 10/03/2020 10:11, Jiri Pirko wrote:
> Instead of manually iterating over entries, use flow_action_for_each
> helper. Move the helper and wrap it to fit to 80 cols on the way.
>
> Signed-off-by: Jiri Pirko <jiri@resnulli.us>
Acked-by: Edward Cree <ecree@solarflare.com>
> ---
> v1->v2:
> - removed action_entry init in loop
> ---
> include/net/flow_offload.h | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/include/net/flow_offload.h b/include/net/flow_offload.h
> index 64807aa03cee..891e15055708 100644
> --- a/include/net/flow_offload.h
> +++ b/include/net/flow_offload.h
> @@ -256,6 +256,11 @@ static inline bool flow_offload_has_one_action(const struct flow_action *action)
> return action->num_entries == 1;
> }
>
> +#define flow_action_for_each(__i, __act, __actions) \
> + for (__i = 0, __act = &(__actions)->entries[0]; \
> + __i < (__actions)->num_entries; \
> + __act = &(__actions)->entries[++__i])
> +
> static inline bool
> flow_action_mixed_hw_stats_types_check(const struct flow_action *action,
> struct netlink_ext_ack *extack)
> @@ -267,8 +272,7 @@ flow_action_mixed_hw_stats_types_check(const struct flow_action *action,
> if (flow_offload_has_one_action(action))
> return true;
>
> - for (i = 0; i < action->num_entries; i++) {
> - action_entry = &action->entries[i];
> + flow_action_for_each(i, action_entry, action) {
> if (i && action_entry->hw_stats_type != last_hw_stats_type) {
> NL_SET_ERR_MSG_MOD(extack, "Mixing HW stats types for actions is not supported");
> return false;
> @@ -316,9 +320,6 @@ flow_action_basic_hw_stats_types_check(const struct flow_action *action,
> return flow_action_hw_stats_types_check(action, extack, 0);
> }
>
> -#define flow_action_for_each(__i, __act, __actions) \
> - for (__i = 0, __act = &(__actions)->entries[0]; __i < (__actions)->num_entries; __act = &(__actions)->entries[++__i])
> -
> struct flow_rule {
> struct flow_match match;
> struct flow_action action;
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [patch net-next v2] flow_offload: use flow_action_for_each in flow_action_mixed_hw_stats_types_check()
2020-03-10 10:11 [patch net-next v2] flow_offload: use flow_action_for_each in flow_action_mixed_hw_stats_types_check() Jiri Pirko
2020-03-10 11:30 ` Pablo Neira Ayuso
2020-03-10 17:44 ` Edward Cree
@ 2020-03-10 22:43 ` David Miller
2 siblings, 0 replies; 4+ messages in thread
From: David Miller @ 2020-03-10 22:43 UTC (permalink / raw)
To: jiri; +Cc: netdev, kuba, pablo, ecree, mlxsw
From: Jiri Pirko <jiri@resnulli.us>
Date: Tue, 10 Mar 2020 11:11:57 +0100
> Instead of manually iterating over entries, use flow_action_for_each
> helper. Move the helper and wrap it to fit to 80 cols on the way.
>
> Signed-off-by: Jiri Pirko <jiri@resnulli.us>
> ---
> v1->v2:
> - removed action_entry init in loop
Applied, thank you.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-03-10 22:43 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-10 10:11 [patch net-next v2] flow_offload: use flow_action_for_each in flow_action_mixed_hw_stats_types_check() Jiri Pirko
2020-03-10 11:30 ` Pablo Neira Ayuso
2020-03-10 17:44 ` Edward Cree
2020-03-10 22:43 ` David Miller
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.