diff options
| author |   Douglas Gilbert <dougg@torque.net> | 2005-09-09 20:07:32 +1000 |
|---|---|---|
| committer |   James Bottomley <jejb@mulgrave.(none)> | 2005-09-09 16:42:44 -0500 |
| commit | 942fc2fb73f2cac53484ebaf1c4f9af7aefaca83 (patch) | |
| tree | 7d399d5847dd3be95368cf66860439ae0954ae19 /drivers/block/scsi_ioctl.c | |
| parent | [SCSI] sg: do not set VM_IO flag on mmap-ed pages (diff) | |
| download | linux-dev-942fc2fb73f2cac53484ebaf1c4f9af7aefaca83.tar.xz linux-dev-942fc2fb73f2cac53484ebaf1c4f9af7aefaca83.zip | |
[SCSI] permit READ DEFECT DATA in block/scsi_ioctl
The soon to be released smartmontools 5.34 uses the
READ DEFECT DATA command on SCSI disks. A disk that
has defect list entries (or worse, an increasing number
of them) is at risk.
Currently the first invocation of smartctl causes this:
scsi: unknown opcode 0x37
message to appear the console and in the log.
The READ DEFECT DATA SCSI command does not change
the state of a disk. Its opcode (0x37) is valid for
SBC devices (e.g. disks) and SMC-2 devices (media
changers) where it is called INITIALIZE STATUS ELEMENT
WITH RANGE and again doesn't change the external state
of the device.
Changelog:
- mark SCSI opcode 0x37 (READ DEFECT DATA) as
safe_for_read
Signed-off-by: Douglas Gilbert <dougg@torque.net>
Acked-by: Jens Axboe <axboe@suse.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/block/scsi_ioctl.c')
| -rw-r--r-- | drivers/block/scsi_ioctl.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/block/scsi_ioctl.c b/drivers/block/scsi_ioctl.c index abb2df249fd3..856c2278e9d0 100644 --- a/drivers/block/scsi_ioctl.c +++ b/drivers/block/scsi_ioctl.c @@ -123,6 +123,7 @@ static int verify_command(struct file *file, unsigned char *cmd) safe_for_read(READ_12), safe_for_read(READ_16), safe_for_read(READ_BUFFER), + safe_for_read(READ_DEFECT_DATA), safe_for_read(READ_LONG), safe_for_read(INQUIRY), safe_for_read(MODE_SENSE), |