diff options
| author | Christof Schmitt <[email protected]> | 2009-09-17 07:10:14 +0000 |
|---|---|---|
| committer | James Bottomley <[email protected]> | 2009-10-02 14:45:58 +0000 |
| commit | e27168f8c337b12b8aa8d59c3123c79d2f83603d (patch) | |
| tree | 947d5e483ca1d4ee0bbbbdc7abf317f1f591dbd0 /drivers/scsi/mpt2sas/mpi/mpi2_init.h | |
| parent | [SCSI] sr: consider the last written sector when determining media size (diff) | |
| download | kernel-e27168f8c337b12b8aa8d59c3123c79d2f83603d.tar.gz kernel-e27168f8c337b12b8aa8d59c3123c79d2f83603d.zip | |
[SCSI] sg: Free data buffers after calling blk_rq_unmap_user
Running sg_luns on s390x with CONFIG_DEBUG_PAGEALLOC enabled fails
with EFAULT from the SG_IO ioctl. The EFAULT is the result from
copy_to_user failing in this call chain:
sg_ioctl
sg_new_read
sg_finish_rem_req
blk_rq_unmap_user
__blk_rq_unmap_user
bio_uncopy_user
__bio_copy_iov
copy_to_user
The sg driver calls sg_remove_scat to free the memory pages before
calling blk_rq_unmap_user that tries to copy the data back to
userspace. Change the order to first call blk_rq_unmap_user before
freeing the pages in sg_remove_scat.
Acked-by: FUJITA Tomonori <[email protected]>
Cc: [email protected]
Signed-off-by: Christof Schmitt <[email protected]>
Acked-by: Douglas Gilbert <[email protected]>
Signed-off-by: James Bottomley <[email protected]>
Diffstat (limited to 'drivers/scsi/mpt2sas/mpi/mpi2_init.h')
0 files changed, 0 insertions, 0 deletions