]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commit
net: pktgen: remove rcu locking in pktgen_change_name()
authorEric Dumazet <edumazet@google.com>
Sat, 15 Oct 2016 15:50:49 +0000 (17:50 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 17 Oct 2016 14:52:59 +0000 (10:52 -0400)
commit9a0b1e8ba4061778897b544afc898de2163382f7
tree910a94c53d211ba3a5a17f6dc39fad9649becd65
parenta04a480d4392ea6efd117be2de564117b2a009c0
net: pktgen: remove rcu locking in pktgen_change_name()

After Jesper commit back in linux-3.18, we trigger a lockdep
splat in proc_create_data() while allocating memory from
pktgen_change_name().

This patch converts t->if_lock to a mutex, since it is now only
used from control path, and adds proper locking to pktgen_change_name()

1) pktgen_thread_lock to protect the outer loop (iterating threads)
2) t->if_lock to protect the inner loop (iterating devices)

Note that before Jesper patch, pktgen_change_name() was lacking proper
protection, but lockdep was not able to detect the problem.

Fixes: 8788370a1d4b ("pktgen: RCU-ify "if_list" to remove lock in next_to_run()")
Reported-by: John Sperbeck <jsperbeck@google.com>
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Jesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/pktgen.c