Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1366144 > unrolled thread
| Started by | Haishuang Yan <yanhaishuang@cmss.chinamobile.com> |
|---|---|
| First post | 2016-03-29 12:50 +0200 |
| Last post | 2016-03-30 21:50 +0200 |
| Articles | 3 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] bridge: Allow set bridge ageing time when switchdev disabled Haishuang Yan <yanhaishuang@cmss.chinamobile.com> - 2016-03-29 12:50 +0200
Re: [PATCH] bridge: Allow set bridge ageing time when switchdev disabled Ido Schimmel <idosch@mellanox.com> - 2016-03-30 00:10 +0200
Re: [PATCH] bridge: Allow set bridge ageing time when switchdev disabled David Miller <davem@davemloft.net> - 2016-03-30 21:50 +0200
| From | Haishuang Yan <yanhaishuang@cmss.chinamobile.com> |
|---|---|
| Date | 2016-03-29 12:50 +0200 |
| Subject | [PATCH] bridge: Allow set bridge ageing time when switchdev disabled |
| Message-ID | <rhZnH-4Qd-5@gated-at.bofh.it> |
When NET_SWITCHDEV=n, switchdev_port_attr_set will return -EOPNOTSUPP, we should ignore this error code and continue to set the ageing time. Signed-off-by: Haishuang Yan <yanhaishuang@cmss.chinamobile.com> --- net/bridge/br_stp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/bridge/br_stp.c b/net/bridge/br_stp.c index e234490..9cb7044 100644 --- a/net/bridge/br_stp.c +++ b/net/bridge/br_stp.c @@ -582,7 +582,7 @@ int br_set_ageing_time(struct net_bridge *br, u32 ageing_time) int err; err = switchdev_port_attr_set(br->dev, &attr); - if (err) + if (err && err != -EOPNOTSUPP) return err; br->ageing_time = t; -- 1.8.3.1
[toc] | [next] | [standalone]
| From | Ido Schimmel <idosch@mellanox.com> |
|---|---|
| Date | 2016-03-30 00:10 +0200 |
| Subject | Re: [PATCH] bridge: Allow set bridge ageing time when switchdev disabled |
| Message-ID | <ri9ZM-46r-17@gated-at.bofh.it> |
| In reply to | #1366144 |
Tue, Mar 29, 2016 at 01:48:08PM IDT, yanhaishuang@cmss.chinamobile.com wrote:
>When NET_SWITCHDEV=n, switchdev_port_attr_set will return -EOPNOTSUPP,
>we should ignore this error code and continue to set the ageing time.
>
>Signed-off-by: Haishuang Yan <yanhaishuang@cmss.chinamobile.com>
Fixes: c62987bbd8a1 ("bridge: push bridge setting ageing_time down to switchdev")
Acked-by: Ido Schimmel <idosch@mellanox.com>
Thank you.
[toc] | [prev] | [next] | [standalone]
| From | David Miller <davem@davemloft.net> |
|---|---|
| Date | 2016-03-30 21:50 +0200 |
| Subject | Re: [PATCH] bridge: Allow set bridge ageing time when switchdev disabled |
| Message-ID | <riuhQ-1Jv-13@gated-at.bofh.it> |
| In reply to | #1366144 |
From: Haishuang Yan <yanhaishuang@cmss.chinamobile.com> Date: Tue, 29 Mar 2016 18:48:08 +0800 > When NET_SWITCHDEV=n, switchdev_port_attr_set will return -EOPNOTSUPP, > we should ignore this error code and continue to set the ageing time. > > Signed-off-by: Haishuang Yan <yanhaishuang@cmss.chinamobile.com> Applied and queued up for -stable, thanks. > @@ -582,7 +582,7 @@ int br_set_ageing_time(struct net_bridge *br, u32 ageing_time) > int err; > > err = switchdev_port_attr_set(br->dev, &attr); > - if (err) > + if (err && err != -EOPNOTSUPP) > return err; > > br->ageing_time = t; This is at least the 5th time I've had to apply a bug fix exactly like this one. If this doesn't convince everyone that this whole -EOPNOTSUPP mechanism is extremely error prone, I don't know what will. There should be only one or two strictly controlled locations that reinterpret the return value from all of these routines that have this special way of signalling lack of switchdev support via -EOPNOTSUPP. It can't be spread all over the place like this, it simply does NOT work.
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web