On 07/02/2021 21:47, Vladimir Oltean wrote: > From: Vladimir Oltean <vladimir.oltean@xxxxxxx> > > Looking through patchwork I don't see that there was any consensus to > use switchdev notifiers only in case of netlink provided port flags but > not sysfs (as a sort of deprecation, punishment or anything like that), > so we should probably keep the user interface consistent in terms of > functionality. > > http://patchwork.ozlabs.org/project/netdev/patch/20170605092043.3523-3-jiri@xxxxxxxxxxx/ > http://patchwork.ozlabs.org/project/netdev/patch/20170608064428.4785-3-jiri@xxxxxxxxxxx/ > > Fixes: 3922285d96e7 ("net: bridge: Add support for offloading port attributes") > Signed-off-by: Vladimir Oltean <vladimir.oltean@xxxxxxx> > --- > net/bridge/br_sysfs_if.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/net/bridge/br_sysfs_if.c b/net/bridge/br_sysfs_if.c > index 96ff63cde1be..5aea9427ffe1 100644 > --- a/net/bridge/br_sysfs_if.c > +++ b/net/bridge/br_sysfs_if.c > @@ -59,9 +59,8 @@ static BRPORT_ATTR(_name, 0644, \ > static int store_flag(struct net_bridge_port *p, unsigned long v, > unsigned long mask) > { > - unsigned long flags; > - > - flags = p->flags; > + unsigned long flags = p->flags; > + int err; > > if (v) > flags |= mask; > @@ -69,6 +68,10 @@ static int store_flag(struct net_bridge_port *p, unsigned long v, > flags &= ~mask; > > if (flags != p->flags) { > + err = br_switchdev_set_port_flag(p, flags, mask); > + if (err) > + return err; > + > p->flags = flags; > br_port_flags_change(p, mask); > } > Acked-by: Nikolay Aleksandrov <nikolay@xxxxxxxxxx>
