]> git.baikalelectronics.ru Git - kernel.git/commit
[SCSI]: Fix scsi_send_eh_cmnd scatterlist handling
authorDavid S. Miller <davem@sunset.davemloft.net>
Mon, 2 Apr 2007 21:21:55 +0000 (14:21 -0700)
committerDavid S. Miller <davem@sunset.davemloft.net>
Mon, 2 Apr 2007 21:26:22 +0000 (14:26 -0700)
commitfdfd67e3c2f2b2917f30baa35ef7998e28ca3af1
treeece3e6847a0b60f15594e90f1d567e9df6eefa5d
parentbf8530f80e98835cab335cddf6dc10b49b2e5f96
[SCSI]: Fix scsi_send_eh_cmnd scatterlist handling

This fixes a regression caused by commit:

0b31ad55477cae1113dca90b3eb249a8bf17efbb

The sense buffer code in scsi_send_eh_cmnd was changed to use
alloc_page() and a scatter list, but the sense data copy was not
updated to match so what we actually get in the sense buffer is total
grabage starting with the kernel address of the struct page we got.
Basically the stack frame of scsi_send_eh_cmd() is what ends up
in the sense buffer.

Depending upon how pointers look on a given platform, you can
end up getting sr_ioctl.c errors when you mount a cdrom.  If
the CDROM gives a check condition for GPCMD_GET_CONFIGURATION issued
by drivers/cdrom/cdrom.c:cdrom_mmc_profile(), sr_ioctl will
spit out this error message in sr_do_ioctl() with the way pointers
are on sparc64:

default:
printk(KERN_ERR "%s: CDROM (ioctl) error, command: ", cd->cdi.name);
__scsi_print_command(cgc->cmd);
scsi_print_sense_hdr("sr", &sshdr);
err = -EIO;

This is the error Tom Callaway reported in:

http://marc.info/?l=linux-sparc&m=117407453208101&w=2

Anyways, fix this by using page_address(sgl.page) which is OK
because we know this is low-mem due to GFP_ATOMIC.

Signed-off-by: David S. Miller <davem@davemloft.net>
Acked-by: Christoph Hellwig <hch@lst.de>
drivers/scsi/scsi_error.c