diff options
author | Nicholas Bellinger <nab@linux-iscsi.org> | 2011-03-02 18:52:51 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2011-03-14 19:31:08 -0400 |
commit | 904f0bc482201fa86e75c330d79dfd11be494cf8 (patch) | |
tree | 150fa17a137ecc22c22accf126c61e9dd79d354b /drivers/target | |
parent | 853e2bd2103aaa91d1ba1c0b57ba17628d836f03 (diff) |
[SCSI] target: Fix volume size misreporting for volumes > 2TB
the target infrastructure fails to send the correct conventional size
to READ_CAPACITY that force a retry with READ_CAPACITY_16, which reads
the capacity for devices > 2TB. Fix by adding the correct return to
trigger RC(16).
Reported-by: Ben Jarvis <bjarvismn@gmail.com>
Signed-off-by: Signed-off-by: Nicholas A. Bellinger <nab@linux-iscsi.org>
Cc: stable@kernel.org
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/target')
-rw-r--r-- | drivers/target/target_core_cdb.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/drivers/target/target_core_cdb.c b/drivers/target/target_core_cdb.c index 366080baf474..7f19c8b7b84c 100644 --- a/drivers/target/target_core_cdb.c +++ b/drivers/target/target_core_cdb.c | |||
@@ -667,7 +667,13 @@ target_emulate_readcapacity(struct se_cmd *cmd) | |||
667 | { | 667 | { |
668 | struct se_device *dev = SE_DEV(cmd); | 668 | struct se_device *dev = SE_DEV(cmd); |
669 | unsigned char *buf = cmd->t_task->t_task_buf; | 669 | unsigned char *buf = cmd->t_task->t_task_buf; |
670 | u32 blocks = dev->transport->get_blocks(dev); | 670 | unsigned long long blocks_long = dev->transport->get_blocks(dev); |
671 | u32 blocks; | ||
672 | |||
673 | if (blocks_long >= 0x00000000ffffffff) | ||
674 | blocks = 0xffffffff; | ||
675 | else | ||
676 | blocks = (u32)blocks_long; | ||
671 | 677 | ||
672 | buf[0] = (blocks >> 24) & 0xff; | 678 | buf[0] = (blocks >> 24) & 0xff; |
673 | buf[1] = (blocks >> 16) & 0xff; | 679 | buf[1] = (blocks >> 16) & 0xff; |