netdev - Re: [PATCH net] switchdev: bridge: Check return code is not EOPNOTSUPP

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: <f9593252b8a9d4733dfa7d22dd71d803@eikelenboom.it>
Date:	Fri, 13 Nov 2015 13:34:45 +0100
From:	Sander Eikelenboom <linux@...elenboom.it>
To:	Ido Schimmel <idosch@...lanox.com>
Cc:	netdev@...r.kernel.org, stephen@...workplumber.org,
	davem@...emloft.net, eladr@...lanox.com,
	Jiri Pirko <jiri@...lanox.com>
Subject: Re: [PATCH net] switchdev: bridge: Check return code is not
 EOPNOTSUPP

On 2015-11-13 12:06, Ido Schimmel wrote:
> When NET_SWITCHDEV=n, switchdev_port_attr_set simply returns 
> EOPNOTSUPP.
> In this case we should not emit errors and warnings to the kernel log.

Hi Ido,

Thanks for your patch!

It fixes these:
[  207.245442] vif vif-1-0 vif1.0: failed to set HW ageing time
[  207.245443] xen_bridge: error setting offload STP state on 
port1(vif1.0)

But i still have these:
[  335.412194] vif19.0-emu: set_features() failed (-1); wanted 
0x00000080000048c1, left 0x00000080001b48c9
[  335.412204] vif19.0-emu: set_features() failed (-1); wanted 
0x00000080000048c1, left 0x00000080001b48c9
[  335.412311] vif19.0-emu: set_features() failed (-1); wanted 
0x00000080000248c9, left 0x00000080001b48c9
[  335.412319] vif19.0-emu: set_features() failed (-1); wanted 
0x00000080000048c1, left 0x00000080001b48c9
[  335.412326] vif19.0-emu: set_features() failed (-1); wanted 
0x00000080000048c1, left 0x00000080001b48c9
[  335.535955] vif vif-19-0 vif19.0: set_features() failed (-1); wanted 
0x0000000400004803, left 0x0000000400114813
[  335.535965] vif vif-19-0 vif19.0: set_features() failed (-1); wanted 
0x0000000400004803, left 0x0000000400114813
[  335.615392] vif vif-19-0 vif19.0: set_features() failed (-1); wanted 
0x0000000400004803, left 0x0000000400114813
[  335.615401] xen_bridge: set_features() failed (-1); wanted 
0x000000801fdb78c9, left 0x000000801fff78e9

--
Sander

> Reported-by: Sander Eikelenboom <linux@...elenboom.it>
> Fixes: 0bc05d585d38 ("switchdev: allow caller to explicitly request
> attr_set as deferred")
> Fixes: 6ac311ae8bfb ("Adding switchdev ageing notification on port
> bridged")
> Signed-off-by: Ido Schimmel <idosch@...lanox.com>
> Signed-off-by: Jiri Pirko <jiri@...lanox.com>
> ---
>  net/bridge/br_stp.c    | 2 +-
>  net/bridge/br_stp_if.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/net/bridge/br_stp.c b/net/bridge/br_stp.c
> index f7e8dee..5f3f645 100644
> --- a/net/bridge/br_stp.c
> +++ b/net/bridge/br_stp.c
> @@ -48,7 +48,7 @@ void br_set_state(struct net_bridge_port *p,
> unsigned int state)
> 
>  	p->state = state;
>  	err = switchdev_port_attr_set(p->dev, &attr);
> -	if (err)
> +	if (err && err != -EOPNOTSUPP)
>  		br_warn(p->br, "error setting offload STP state on port %u(%s)\n",
>  				(unsigned int) p->port_no, p->dev->name);
>  }
> diff --git a/net/bridge/br_stp_if.c b/net/bridge/br_stp_if.c
> index fa53d7a..5396ff08 100644
> --- a/net/bridge/br_stp_if.c
> +++ b/net/bridge/br_stp_if.c
> @@ -50,7 +50,7 @@ void br_init_port(struct net_bridge_port *p)
>  	p->config_pending = 0;
> 
>  	err = switchdev_port_attr_set(p->dev, &attr);
> -	if (err)
> +	if (err && err != -EOPNOTSUPP)
>  		netdev_err(p->dev, "failed to set HW ageing time\n");
>  }

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html