diff options
author | J. Bruce Fields <bfields@redhat.com> | 2013-02-10 17:33:48 +0100 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2013-02-17 16:53:16 +0100 |
commit | e75bafbff2270993926abcc31358361db74a9bc2 (patch) | |
tree | 2a59f30307bfa469aab049f9b804107f679460b6 /net/802 | |
parent | lockd: nlmclnt_reclaim(): avoid stack overflow (diff) | |
download | linux-e75bafbff2270993926abcc31358361db74a9bc2.tar.xz linux-e75bafbff2270993926abcc31358361db74a9bc2.zip |
svcrpc: make svc_age_temp_xprts enqueue under sv_lock
svc_age_temp_xprts expires xprts in a two-step process: first it takes
the sv_lock and moves the xprts to expire off their server-wide list
(sv_tempsocks or sv_permsocks) to a local list. Then it drops the
sv_lock and enqueues and puts each one.
I see no reason for this: svc_xprt_enqueue() will take sp_lock, but the
sv_lock and sp_lock are not otherwise nested anywhere (and documentation
at the top of this file claims it's correct to nest these with sp_lock
inside.)
Cc: stable@kernel.org
Tested-by: Jason Tibbitts <tibbs@math.uh.edu>
Tested-by: Paweł Sikora <pawel.sikora@agmk.net>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'net/802')
0 files changed, 0 insertions, 0 deletions