rds: tcp: fix various rds-tcp issues during netns create/delete sequences
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest3 patches 4 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/3 | rds: tcp: Take explicit refcounts on struct netnet/ net/ net/ | +6 | 0 replies | |
| 2/3 | rds: tcp: Reorder initialization sequence in rds_tcp_init to avoid racesnet/ | +9 | 0 replies | |
| 3/3 | rds: tcp: Sequence teardown of listen and acceptor sockets to avoid racesnet/ net/ net/ | +18 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH net 0/3] rds: tcp: fix various rds-tcp issues during netns create/delete sequences
Dmitry Vyukov reported some syszkaller panics during netns deletion.
While I have not been able to reproduce those exact panics, my attempts to do so uncovered a few other problems, which are fixed patch 2 and patch 3 of this patch series. In addition, as mentioned in,
https://www.spinics.net/lists/netdev/msg422997.html
code-inspection points that the rds_connection needs to take an explicit refcnt on the struct net so that it is held down until all cleanup is completed for netns removal, and this is fixed by patch1.
The following scripts were run concurrently to uncover/test patch{2, 3} while simultaneously running rds-ping to 12.0.0.18 from another system:
# cat del.rds
while [ 1 ]; do
modprobe rds_tcp
modprobe -r rds-tcp
done # cat del.netns
while [ 1 ]; do
ip netns delete blue
ip netns add blue
ip link add link eth1 address a:b:c:d:e:f blue0 type macvlan
ip link set blue0 netns blue
ip netns exec blue ip addr add 12.0.0.18/24 dev blue0
ip netns exec blue ifconfig blue0 up
sleep 3;
doneSowmini Varadhan (3):
rds: tcp: Take explicit refcounts on struct net
rds: tcp: Reorder initialization sequence in rds_tcp_init to avoid
races
rds: tcp: Sequence teardown of listen and acceptor sockets to avoid
races
net/rds/connection.c | 1 + net/rds/rds.h | 6 +++--- net/rds/tcp.c | 38 +++++++++++++++++++++----------------- net/rds/tcp.h | 2 +- net/rds/tcp_listen.c | 9 +++++++-- 5 files changed, 33 insertions(+), 23 deletions(-)