diff options
| author | Boaz Harrosh <bharrosh@panasas.com> | 2009-03-24 07:37:50 -0400 |
|---|---|---|
| committer | Jens Axboe <jens.axboe@oracle.com> | 2009-03-26 06:01:25 -0400 |
| commit | e7cbbf1bf17e3c706f874e867f7b744e1c86fed9 (patch) | |
| tree | 93076e12227e81f711402641146a4644d31a3c57 | |
| parent | 1cd96c242a829d52f7a5ae98f554ca9775429685 (diff) | |
bsg: Remove bogus check against request_queue->max_sectors
bsg submits REQ_TYPE_BLOCK_PC so the right check is max_hw_sectors.
But I've removed this check because right after, bsg proceeds with
calling blk_rq_map_user() which does all the right checks.
Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
| -rw-r--r-- | block/bsg.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/block/bsg.c b/block/bsg.c index 0f63b91d0af6..206060e795da 100644 --- a/block/bsg.c +++ b/block/bsg.c | |||
| @@ -218,9 +218,6 @@ bsg_validate_sgv4_hdr(struct request_queue *q, struct sg_io_v4 *hdr, int *rw) | |||
| 218 | 218 | ||
| 219 | if (hdr->guard != 'Q') | 219 | if (hdr->guard != 'Q') |
| 220 | return -EINVAL; | 220 | return -EINVAL; |
| 221 | if (hdr->dout_xfer_len > (q->max_sectors << 9) || | ||
| 222 | hdr->din_xfer_len > (q->max_sectors << 9)) | ||
| 223 | return -EIO; | ||
| 224 | 221 | ||
| 225 | switch (hdr->protocol) { | 222 | switch (hdr->protocol) { |
| 226 | case BSG_PROTOCOL_SCSI: | 223 | case BSG_PROTOCOL_SCSI: |
