netdev - Re: [patch net-next v2] flow_offload: use flow_action_for_each in flow_action_mixed_hw_stats_types

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <cf6e79eb-c68a-d063-a6e4-3795e30f3543@solarflare.com>
Date:   Tue, 10 Mar 2020 17:44:47 +0000
From:   Edward Cree <ecree@...arflare.com>
To:     Jiri Pirko <jiri@...nulli.us>, <netdev@...r.kernel.org>
CC:     <davem@...emloft.net>, <kuba@...nel.org>, <pablo@...filter.org>,
        <mlxsw@...lanox.com>
Subject: Re: [patch net-next v2] flow_offload: use flow_action_for_each in
 flow_action_mixed_hw_stats_types_check()

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@...nulli.us>
Acked-by: Edward Cree <ecree@...arflare.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;