[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ef9d16e6-186d-4ee3-9888-1c4aba1b4b88@linux.intel.com>
Date: Mon, 10 Jun 2024 11:06:13 +0200
From: Marcin Szycik <marcin.szycik@...ux.intel.com>
To: Asbjørn Sloth Tønnesen <ast@...erby.net>,
netdev@...r.kernel.org
Cc: Louis Peens <louis.peens@...igine.com>,
Davide Caratti <dcaratti@...hat.com>, Leon Romanovsky <leon@...nel.org>,
linux-net-drivers@....com, intel-wired-lan@...ts.osuosl.org,
oss-drivers@...igine.com, i.maximets@....org,
Tariq Toukan <tariqt@...dia.com>, linux-kernel@...r.kernel.org,
Edward Cree <ecree.xilinx@...il.com>, Eric Dumazet <edumazet@...gle.com>,
Tony Nguyen <anthony.l.nguyen@...el.com>,
Martin Habets <habetsm.xilinx@...il.com>, Jakub Kicinski <kuba@...nel.org>,
Paolo Abeni <pabeni@...hat.com>, Saeed Mahameed <saeedm@...dia.com>,
"David S. Miller" <davem@...emloft.net>, linux-rdma@...r.kernel.org
Subject: Re: [Intel-wired-lan] [PATCH net-next 5/5] ice: flower: validate
encapsulation control flags
On 09.06.2024 19:33, Asbjørn Sloth Tønnesen wrote:
> Encapsulation control flags are currently not used anywhere,
> so all flags are currently unsupported by all drivers.
>
> This patch adds validation of this assumption, so that
> encapsulation flags may be used in the future.
>
> In case any encapsulation control flags are masked,
> flow_rule_match_has_enc_control_flags() sets a NL extended
> error message, and we return -EOPNOTSUPP.
>
> Only compile tested.
Reviewed-by: Marcin Szycik <marcin.szycik@...ux.intel.com>
> Signed-off-by: Asbjørn Sloth Tønnesen <ast@...erby.net>
> ---
> drivers/net/ethernet/intel/ice/ice_tc_lib.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/net/ethernet/intel/ice/ice_tc_lib.c b/drivers/net/ethernet/intel/ice/ice_tc_lib.c
> index 8bd24b33f3a67..e6923f8121a99 100644
> --- a/drivers/net/ethernet/intel/ice/ice_tc_lib.c
> +++ b/drivers/net/ethernet/intel/ice/ice_tc_lib.c
> @@ -1353,6 +1353,7 @@ ice_parse_tunnel_attr(struct net_device *dev, struct flow_rule *rule,
> struct ice_tc_flower_fltr *fltr)
> {
> struct ice_tc_flower_lyr_2_4_hdrs *headers = &fltr->outer_headers;
> + struct netlink_ext_ack *extack = fltr->extack;
> struct flow_match_control enc_control;
>
> fltr->tunnel_type = ice_tc_tun_get_type(dev);
> @@ -1373,6 +1374,9 @@ ice_parse_tunnel_attr(struct net_device *dev, struct flow_rule *rule,
>
> flow_rule_match_enc_control(rule, &enc_control);
>
> + if (flow_rule_has_enc_control_flags(enc_control.mask->flags, extack))
> + return -EOPNOTSUPP;
> +
> if (enc_control.key->addr_type == FLOW_DISSECTOR_KEY_IPV4_ADDRS) {
> struct flow_match_ipv4_addrs match;
>
Powered by blists - more mailing lists