On Thu, Sep 24, 2015 at 2:05 PM, Stephen Hemminger <[email protected]> wrote: > On Thu, 24 Sep 2015 13:59:26 -0700 > [email protected] wrote: > >> From: Scott Feldman <[email protected]> >> >> Push bridge-level attributes down to switchdev drivers. This patchset >> adds the infrastructure and then pushes, as an example, ageing_time attribute >> down from bridge to switchdev (rocker) driver. Add some range-checking >> for ageing_time. >> >> # ip link set dev br0 type bridge ageing_time 1000 >> >> # ip link set dev br0 type bridge ageing_time 999 >> RTNETLINK answers: Numerical result out of range >> >> Up until now, switchdev attrs where port-level attrs, so the netdev used in >> switchdev_attr_set() would be a switch port or bond of switch ports. With >> bridge-level attrs, the netdev passed to switchdev_attr_set() is the bridge >> netdev. The same recusive algo is used to visit the leaves of the stacked >> drivers to set the attr, it's just in this case we start one layer higher in >> the stack. One note is not all ports in the bridge may support setting a >> bridge-level attribute, so rather than failing the entire set, we'll skip >> over >> those ports returning -EOPNOTSUPP. > > > Rather than having more in bridge, shouldn't this just be a netlink event?
You lost me? Oh, do you mean netdev notifier? Jiri (privately) had suggested that also. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
