[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 6/6] block/iscsi: Do not force-cap *pnum
From: |
Hanna Reitz |
Subject: |
[PATCH v3 6/6] block/iscsi: Do not force-cap *pnum |
Date: |
Thu, 12 Aug 2021 10:41:48 +0200 |
bdrv_co_block_status() does it for us, we do not need to do it here.
The advantage of not capping *pnum is that bdrv_co_block_status() can
cache larger data regions than requested by its caller.
Signed-off-by: Hanna Reitz <hreitz@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
---
block/iscsi.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/block/iscsi.c b/block/iscsi.c
index 4d2a416ce7..852384086b 100644
--- a/block/iscsi.c
+++ b/block/iscsi.c
@@ -781,9 +781,6 @@ retry:
iscsi_allocmap_set_allocated(iscsilun, offset, *pnum);
}
- if (*pnum > bytes) {
- *pnum = bytes;
- }
out_unlock:
qemu_mutex_unlock(&iscsilun->mutex);
g_free(iTask.err_str);
--
2.31.1
- [PATCH v3 0/6] block-status cache for data regions, Hanna Reitz, 2021/08/12
- [PATCH v3 3/6] block: Clarify that @bytes is no limit on *pnum, Hanna Reitz, 2021/08/12
- [PATCH v3 1/6] block: Drop BDS comment regarding bdrv_append(), Hanna Reitz, 2021/08/12
- [PATCH v3 2/6] block: block-status cache for data regions, Hanna Reitz, 2021/08/12
- [PATCH v3 4/6] block/file-posix: Do not force-cap *pnum, Hanna Reitz, 2021/08/12
- [PATCH v3 6/6] block/iscsi: Do not force-cap *pnum,
Hanna Reitz <=
- [PATCH v3 5/6] block/gluster: Do not force-cap *pnum, Hanna Reitz, 2021/08/12