summaryrefslogtreecommitdiffstats
path: root/net/ipv6
diff options
context:
space:
mode:
authorNeilBrown <neilb@suse.de>2010-11-15 11:27:01 +1100
committerJ. Bruce Fields <bfields@redhat.com>2010-12-17 15:48:18 -0500
commit7c96aef75949a56ec427fc6a2522dace2af33605 (patch)
treeca65368231d4c5b39e3adce558bfe15d712809c7 /net/ipv6
parent18b631f83810e95eeb2e1839889b27142bd8d6d8 (diff)
downloadop-kernel-dev-7c96aef75949a56ec427fc6a2522dace2af33605.zip
op-kernel-dev-7c96aef75949a56ec427fc6a2522dace2af33605.tar.gz
sunrpc: remove xpt_pool
The xpt_pool field is only used for reporting BUGs. And it isn't used correctly. In particular, when it is cleared in svc_xprt_received before XPT_BUSY is cleared, there is no guarantee that either the compiler or the CPU might not re-order to two assignments, just setting xpt_pool to NULL after XPT_BUSY is cleared. If a different cpu were running svc_xprt_enqueue at this moment, it might see XPT_BUSY clear and then xpt_pool non-NULL, and so BUG. This could be fixed by calling smp_mb__before_clear_bit() before the clear_bit. However as xpt_pool isn't really used, it seems safest to simply remove xpt_pool. Another alternate would be to change the clear_bit to clear_bit_unlock, and the test_and_set_bit to test_and_set_bit_lock. Signed-off-by: NeilBrown <neilb@suse.de> Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'net/ipv6')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud