diff options
| author | <jejb@titanic.il.steeleye.com> | 2005-04-12 17:38:09 -0400 |
|---|---|---|
| committer | James Bottomley <jejb@titanic> | 2005-04-16 21:10:34 -0400 |
| commit | c2a9331c62216e222a08de8dd58559367fef0af6 (patch) | |
| tree | f25b06ee61ffa8b000a45d0ec5f8a92ebb67f0ce /include/scsi/scsi_device.h | |
| parent | 152587deb8903c0edf483a5b889f975bc6bea7e0 (diff) | |
updates for CFQ oops fix
- add a comment to the device structure that the device_busy field
is now protected by the request_queue->queue_lock
- null out sdev->request_queue after the queue is released to trap
any (and there shouldn't be any) use after the queue is freed.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'include/scsi/scsi_device.h')
| -rw-r--r-- | include/scsi/scsi_device.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/scsi/scsi_device.h b/include/scsi/scsi_device.h index f6d051318299..fe9571d2e306 100644 --- a/include/scsi/scsi_device.h +++ b/include/scsi/scsi_device.h | |||
| @@ -43,6 +43,7 @@ struct scsi_device { | |||
| 43 | struct list_head siblings; /* list of all devices on this host */ | 43 | struct list_head siblings; /* list of all devices on this host */ |
| 44 | struct list_head same_target_siblings; /* just the devices sharing same target id */ | 44 | struct list_head same_target_siblings; /* just the devices sharing same target id */ |
| 45 | 45 | ||
| 46 | /* this is now protected by the request_queue->queue_lock */ | ||
| 46 | volatile unsigned short device_busy; /* commands actually active on low-level */ | 47 | volatile unsigned short device_busy; /* commands actually active on low-level */ |
| 47 | spinlock_t list_lock; | 48 | spinlock_t list_lock; |
| 48 | struct list_head cmd_list; /* queue of in use SCSI Command structures */ | 49 | struct list_head cmd_list; /* queue of in use SCSI Command structures */ |