diff options
author | James Bottomley <James.Bottomley@suse.de> | 2011-04-22 10:39:59 -0500 |
---|---|---|
committer | Willy Tarreau <w@1wt.eu> | 2012-02-11 15:37:12 +0100 |
commit | b6c5c5aea17e03b89ace3fb5f138902d3349c8f0 (patch) | |
tree | 6365028915cd5c92c11030a5fa3bc1d02de28372 /scripts/patch-kernel | |
parent | ba07b72d2e7ee9d6041dea436d9856d5d798264c (diff) |
put stricter guards on queue dead checks
commit 86cbfb5607d4b81b1a993ff689bbd2addd5d3a9b upstream.
SCSI uses request_queue->queuedata == NULL as a signal that the queue
is dying. We set this state in the sdev release function. However,
this allows a small window where we release the last reference but
haven't quite got to this stage yet and so something will try to take
a reference in scsi_request_fn and oops. It's very rare, but we had a
report here, so we're pushing this as a bug fix
The actual fix is to set request_queue->queuedata to NULL in
scsi_remove_device() before we drop the reference. This causes
correct automatic rejects from scsi_request_fn as people who hold
additional references try to submit work and prevents anything from
getting a new reference to the sdev that way.
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Signed-off-by: Willy Tarreau <w@1wt.eu>
Diffstat (limited to 'scripts/patch-kernel')
0 files changed, 0 insertions, 0 deletions