linux-stable.git/net/dsa, branch linux-4.6.y

net: dsa: refine netdev event notifier

2016-03-14T20:05:32+00:00

Rework the netdev event handler, similar to what the Mellanox Spectrum
driver does, to easily welcome more events later (for example
NETDEV_PRECHANGEUPPER) and use netdev helpers (such as
netif_is_bridge_master).

Signed-off-by: Vivien Didelot 
Acked-by: Jiri Pirko 
Acked-by: Ido Schimmel 
Signed-off-by: David S. Miller

net: dsa: make port_bridge_leave return void

2016-03-14T20:05:31+00:00

netdev_upper_dev_unlink() which notifies NETDEV_CHANGEUPPER, returns
void, as well as del_nbp(). So there's no advantage to catch an eventual
error from the port_bridge_leave routine at the DSA level.

Make this routine void for the DSA layer and its existing drivers.

Signed-off-by: Vivien Didelot 
Acked-by: Jiri Pirko 
Signed-off-by: David S. Miller

net: dsa: rename port_*_bridge routines

2016-03-14T20:05:31+00:00

Rename DSA port_join_bridge and port_leave_bridge routines to
respectively port_bridge_join and port_bridge_leave in order to respect
an implicit Port::Bridge namespace.

Signed-off-by: Vivien Didelot 
Acked-by: Jiri Pirko 
Signed-off-by: David S. Miller

phy: fixed: Fix removal of phys.

2016-03-14T19:43:11+00:00

The fixed phys delete function simply removed the fixed phy from the
internal linked list and freed the memory. It however did not
unregister the associated phy device. This meant it was still possible
to find the phy device on the mdio bus.

Make fixed_phy_del() an internal function and add a
fixed_phy_unregister() to unregisters the phy device and then uses
fixed_phy_del() to free resources.

Modify DSA to use this new API function, so we don't leak phys.

Signed-off-by: Andrew Lunn 
Signed-off-by: David S. Miller

dsa: dsa: Fix freeing of fixed-phys from user ports.

2016-03-14T19:43:11+00:00

All ports types can have a fixed PHY associated with it. Remove the
check which limits removal to only CPU and DSA ports.

Signed-off-by: Andrew Lunn 
Signed-off-by: David S. Miller

dsa: Destroy fixed link phys after the phy has been disconnected

2016-03-14T19:43:11+00:00

The phy is disconnected from the slave in dsa_slave_destroy(). Don't
destroy fixed link phys until after this, since there can be fixed
linked phys connected to ports.

Signed-off-by: Andrew Lunn 
Signed-off-by: David S. Miller

dsa: slave: Don't reference NULL pointer during phy_disconnect

2016-03-14T19:43:10+00:00

When the phy is disconnected, the parent pointer to the netdev it was
attached to is set to NULL. The code then tries to suspend the phy,
but dsa_slave_fixed_link_update needs the parent pointer to determine
which switch the phy is connected to. So it dereferenced a NULL
pointer. Check for this condition.

Signed-off-by: Andrew Lunn 
Signed-off-by: David S. Miller

net: dsa: Fix cleanup resources upon module removal

2016-03-10T21:20:17+00:00

The initial commit badly merged into the dsa_resume method instead
of the dsa_remove_dst method.
As consequence, the dst->master_netdev->dsa_ptr is not set to NULL on
removal and re-bind of the dsa device fails with error -17.

Fixes: b0dc635d923c ("net: dsa: cleanup resources upon module removal ")
Signed-off-by: Neil Armstrong 
Acked-by: Andrew Lunn 
Signed-off-by: David S. Miller

net: dsa: support VLAN filtering switchdev attr

2016-03-01T21:24:51+00:00

When a user explicitly requests VLAN filtering with something like:

    # echo 1 > /sys/class/net//bridge/vlan_filtering

Switchdev propagates a SWITCHDEV_ATTR_ID_BRIDGE_VLAN_FILTERING port
attribute.

Add support for it in the DSA layer with a new port_vlan_filtering
function to let drivers toggle 802.1Q filtering on user demand.

Signed-off-by: Vivien Didelot 
Signed-off-by: David S. Miller

net: dsa: drop vlan_getnext

2016-02-25T20:20:21+00:00

The VLAN GetNext operation is specific to some switches, and thus can be
complicated to implement for some drivers.

Remove the support for the vlan_getnext/port_pvid_get approach in favor
of the generic and simpler port_vlan_dump function.

Signed-off-by: Vivien Didelot 
Signed-off-by: David S. Miller