Commit dac27072 authored by Scott Mayhew's avatar Scott Mayhew Committed by J. Bruce Fields

nfsd: make nfsd4_scsi_identify_device retry with a larger buffer

nfsd4_scsi_identify_device() performs a single IDENTIFY command for the
device identification VPD page using a small buffer.  If the reply is
too large to fit in this buffer then the GETDEVICEINFO reply will not
contain any info for the SCSI volume aside from the registration key.
This can happen for example if the device has descriptors using long
SCSI name strings.

When the initial reply from the device indicates a larger buffer is
needed, retry once using the page length from that reply.
Signed-off-by: default avatarScott Mayhew <smayhew@redhat.com>
Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Signed-off-by: default avatarJ. Bruce Fields <bfields@redhat.com>
parent 9c2ece6e
...@@ -216,13 +216,21 @@ static int nfsd4_scsi_identify_device(struct block_device *bdev, ...@@ -216,13 +216,21 @@ static int nfsd4_scsi_identify_device(struct block_device *bdev,
struct request_queue *q = bdev->bd_disk->queue; struct request_queue *q = bdev->bd_disk->queue;
struct request *rq; struct request *rq;
struct scsi_request *req; struct scsi_request *req;
size_t bufflen = 252, len, id_len; /*
* The allocation length (passed in bytes 3 and 4 of the INQUIRY
* command descriptor block) specifies the number of bytes that have
* been allocated for the data-in buffer.
* 252 is the highest one-byte value that is a multiple of 4.
* 65532 is the highest two-byte value that is a multiple of 4.
*/
size_t bufflen = 252, maxlen = 65532, len, id_len;
u8 *buf, *d, type, assoc; u8 *buf, *d, type, assoc;
int error; int retries = 1, error;
if (WARN_ON_ONCE(!blk_queue_scsi_passthrough(q))) if (WARN_ON_ONCE(!blk_queue_scsi_passthrough(q)))
return -EINVAL; return -EINVAL;
again:
buf = kzalloc(bufflen, GFP_KERNEL); buf = kzalloc(bufflen, GFP_KERNEL);
if (!buf) if (!buf)
return -ENOMEM; return -ENOMEM;
...@@ -255,6 +263,12 @@ static int nfsd4_scsi_identify_device(struct block_device *bdev, ...@@ -255,6 +263,12 @@ static int nfsd4_scsi_identify_device(struct block_device *bdev,
len = (buf[2] << 8) + buf[3] + 4; len = (buf[2] << 8) + buf[3] + 4;
if (len > bufflen) { if (len > bufflen) {
if (len <= maxlen && retries--) {
blk_put_request(rq);
kfree(buf);
bufflen = len;
goto again;
}
pr_err("pNFS: INQUIRY 0x83 response invalid (len = %zd)\n", pr_err("pNFS: INQUIRY 0x83 response invalid (len = %zd)\n",
len); len);
goto out_put_request; goto out_put_request;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment