Skip to content

Further fix for dsa unbinding

Neil Armstrong lkml netdev 3 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest4 patches 3 repliesIn review
  2. v26 patches 9 repliesSuperseded
  3. v16 patches 8 repliesSuperseded

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4net: dsa: remove DSA link pollinginclude/net/dsa.h net/dsa/dsa.c+0 -550 replies
2/4net: dsa: cleanup resources upon module removalnet/dsa/dsa.c+8 -00 replies
3/4net: dsa: Add missing master netdev dev_put() callsnet/dsa/dsa.c+5 -10 replies
4/4net: dsa: move dsa slave destroy code to slave.cnet/dsa/dsa.c net/dsa/dsa_priv.h net/dsa/slave.c+13 -20 replies

Reviews trailers per patch

None of the 4 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchAndrew LunnFlorian Fainelli
SeriesA-b A-b R-b
1/4
2/4
3/4
4/4

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v3 thread

Cover letter [PATCH v3 net-next 0/4] Further fix for dsa unbinding

This serie fixes further issues for DSA dynamic unbinding. The first patch completely removes the PHY link state polling. The two following cleans up the dsa state upon removal. The last patch moves slave destroy code as slave function and adds missing netdev and phy cleanup calls.

v1: http://lkml.kernel.org/r/[email protected]
v2: http://lkml.kernel.org/r/[email protected]
remove phy fix and add missing calls in dsa_switch_destroy then add dedicated dsa_slave_destroy

v3: remove polling instead of fixing it, make single patch for dsa slave destroy

Neil Armstrong (4):
net: dsa: remove DSA link polling
net: dsa: cleanup resources upon module removal
net: dsa: Add missing master netdev dev_put() calls
net: dsa: move dsa slave destroy code to slave.c

 include/net/dsa.h  | 12 -----------
 net/dsa/dsa.c      | 60 +++++++++++++-----------------------------------------
 net/dsa/dsa_priv.h |  1 +
 net/dsa/slave.c    | 11 ++++++++++
 4 files changed, 26 insertions(+), 58 deletions(-)
Signature · 1 line
1.9.1