[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCHv5 12/17] iscsi: add bdrv_co_write_zeroes
From: |
Peter Lieven |
Subject: |
[Qemu-devel] [PATCHv5 12/17] iscsi: add bdrv_co_write_zeroes |
Date: |
Sun, 20 Oct 2013 17:43:06 +0200 |
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Peter Lieven <address@hidden>
---
block/iscsi.c | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 59 insertions(+)
diff --git a/block/iscsi.c b/block/iscsi.c
index c0465aa..1845fc8 100644
--- a/block/iscsi.c
+++ b/block/iscsi.c
@@ -56,6 +56,7 @@ typedef struct IscsiLun {
uint8_t lbprz;
struct scsi_inquiry_logical_block_provisioning lbp;
struct scsi_inquiry_block_limits bl;
+ unsigned char *zeroblock;
} IscsiLun;
typedef struct IscsiTask {
@@ -959,6 +960,62 @@ retry:
return 0;
}
+
+static int
+coroutine_fn iscsi_co_write_zeroes(BlockDriverState *bs, int64_t sector_num,
+ int nb_sectors, BdrvRequestFlags flags)
+{
+ IscsiLun *iscsilun = bs->opaque;
+ struct IscsiTask iTask;
+ uint64_t lba;
+ uint32_t nb_blocks;
+
+ if (!is_request_lun_aligned(sector_num, nb_sectors, iscsilun)) {
+ return -EINVAL;
+ }
+
+ if (!iscsilun->lbp.lbpws) {
+ /* WRITE SAME is not supported by the target */
+ return -ENOTSUP;
+ }
+
+ lba = sector_qemu2lun(sector_num, iscsilun);
+ nb_blocks = sector_qemu2lun(nb_sectors, iscsilun);
+
+ if (iscsilun->zeroblock == NULL) {
+ iscsilun->zeroblock = g_malloc0(iscsilun->block_size);
+ }
+
+ iscsi_co_init_iscsitask(iscsilun, &iTask);
+retry:
+ if (iscsi_writesame16_task(iscsilun->iscsi, iscsilun->lun, lba,
+ iscsilun->zeroblock, iscsilun->block_size,
+ nb_blocks, 0, !!(flags & BDRV_REQ_MAY_UNMAP),
+ 0, 0, iscsi_co_generic_cb, &iTask) == NULL) {
+ return -EIO;
+ }
+
+ while (!iTask.complete) {
+ iscsi_set_events(iscsilun);
+ qemu_coroutine_yield();
+ }
+
+ if (iTask.task != NULL) {
+ scsi_free_scsi_task(iTask.task);
+ iTask.task = NULL;
+ }
+
+ if (iTask.do_retry) {
+ goto retry;
+ }
+
+ if (iTask.status != SCSI_STATUS_GOOD) {
+ return -EIO;
+ }
+
+ return 0;
+}
+
static int parse_chap(struct iscsi_context *iscsi, const char *target)
{
QemuOptsList *list;
@@ -1421,6 +1478,7 @@ static void iscsi_close(BlockDriverState *bs)
}
qemu_aio_set_fd_handler(iscsi_get_fd(iscsi), NULL, NULL, NULL);
iscsi_destroy_context(iscsi);
+ g_free(iscsilun->zeroblock);
memset(iscsilun, 0, sizeof(IscsiLun));
}
@@ -1539,6 +1597,7 @@ static BlockDriver bdrv_iscsi = {
.bdrv_co_get_block_status = iscsi_co_get_block_status,
#endif
.bdrv_co_discard = iscsi_co_discard,
+ .bdrv_co_write_zeroes = iscsi_co_write_zeroes,
.bdrv_aio_readv = iscsi_aio_readv,
.bdrv_aio_writev = iscsi_aio_writev,
--
1.7.9.5
- [Qemu-devel] [PATCHv5 01/17] block: make BdrvRequestFlags public, (continued)
- [Qemu-devel] [PATCHv5 01/17] block: make BdrvRequestFlags public, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 03/17] block: introduce BDRV_REQ_MAY_UNMAP request flag, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 04/17] block: add logical block provisioning info to BlockDriverInfo, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 06/17] block/iscsi: add .bdrv_get_info, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 05/17] block: add wrappers for logical block provisioning information, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 07/17] block: add BlockLimits structure to BlockDriverState, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 09/17] block: honour BlockLimits in bdrv_co_discard, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 11/17] iscsi: set limits in BlockDriverState, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 08/17] block: honour BlockLimits in bdrv_co_do_write_zeroes, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 13/17] block: introduce bdrv_make_zero, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 12/17] iscsi: add bdrv_co_write_zeroes,
Peter Lieven <=
- [Qemu-devel] [PATCHv5 14/17] block/get_block_status: fix BDRV_BLOCK_ZERO for unallocated blocks, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 10/17] iscsi: simplify iscsi_co_discard, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 15/17] qemu-img: add support for fully allocated images, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 16/17] qemu-img: conditionally zero out target on convert, Peter Lieven, 2013/10/20
- [Qemu-devel] [PATCHv5 17/17] block/raw: copy BlockLimits on raw_open, Peter Lieven, 2013/10/20