diff options
| author |   Mike Christie <mchristi@redhat.com> | 2016-06-02 20:12:37 -0500 |
|---|---|---|
| committer |   Nicholas Bellinger <nab@linux-iscsi.org> | 2016-07-20 00:58:36 -0700 |
| commit | ea263c7fada4af8ec7fe5fcfd6e7d7705a89351b (patch) | |
| tree | 72beb3f878ba29726a189211cc4e6dacd9a6f339 /drivers/target/target_core_iblock.c | |
| parent | target: Fix race between iscsi-target connection shutdown + ABORT_TASK (diff) | |
| download | linux-dev-ea263c7fada4af8ec7fe5fcfd6e7d7705a89351b.tar.xz linux-dev-ea263c7fada4af8ec7fe5fcfd6e7d7705a89351b.zip | |
target: Fix max_unmap_lba_count calc overflow
max_discard_sectors only 32bits, and some non scsi backend
devices will set this to the max 0xffffffff, so we can end up
overflowing during the max_unmap_lba_count calculation.
This fixes a regression caused by my patch:
commit 8a9ebe717a133ba7bc90b06047f43cc6b8bcb8b3
Author: Mike Christie <mchristi@redhat.com>
Date: Mon Jan 18 14:09:27 2016 -0600
target: Fix WRITE_SAME/DISCARD conversion to linux 512b sectors
which can result in extra discards being sent to due the overflow
causing max_unmap_lba_count to be smaller than what the backing
device can actually support.
Signed-off-by: Mike Christie <mchristi@redhat.com>
Reviewed-by: Bart Van Assche <bart.vanassche@sandisk.com>
Cc: stable@vger.kernel.org
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target/target_core_iblock.c')
| -rw-r--r-- | drivers/target/target_core_iblock.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/target/target_core_iblock.c b/drivers/target/target_core_iblock.c index 7c4efb4417b0..2077bc28640a 100644 --- a/drivers/target/target_core_iblock.c +++ b/drivers/target/target_core_iblock.c @@ -121,8 +121,7 @@ static int iblock_configure_device(struct se_device *dev) dev->dev_attrib.hw_max_sectors = queue_max_hw_sectors(q); dev->dev_attrib.hw_queue_depth = q->nr_requests; - if (target_configure_unmap_from_queue(&dev->dev_attrib, q, - dev->dev_attrib.hw_block_size)) + if (target_configure_unmap_from_queue(&dev->dev_attrib, q)) pr_debug("IBLOCK: BLOCK Discard support available," " disabled by default\n"); |