[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v6 6/9] scsi: move unmap error checking to the compl
From: |
Anton Nefedov |
Subject: |
[Qemu-devel] [PATCH v6 6/9] scsi: move unmap error checking to the complete callback |
Date: |
Fri, 30 Nov 2018 14:47:23 +0000 |
This will help to account the operation in the following commit.
The difference is that we don't call scsi_disk_req_check_error() before
the 1st discard iteration anymore. That function also checks if
the request is cancelled, however it shouldn't get canceled until it
yields in blk_aio() functions anyway.
Same approach is already used for emulate_write_same.
Signed-off-by: Anton Nefedov <address@hidden>
Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Reviewed-by: Alberto Garcia <address@hidden>
---
hw/scsi/scsi-disk.c | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-disk.c
index b1133e6293..daf37d117c 100644
--- a/hw/scsi/scsi-disk.c
+++ b/hw/scsi/scsi-disk.c
@@ -1606,9 +1606,6 @@ static void scsi_unmap_complete_noio(UnmapCBData *data,
int ret)
SCSIDiskState *s = DO_UPCAST(SCSIDiskState, qdev, r->req.dev);
assert(r->req.aiocb == NULL);
- if (scsi_disk_req_check_error(r, ret, false)) {
- goto done;
- }
if (data->count > 0) {
r->sector = ldq_be_p(&data->inbuf[0]);
@@ -1644,7 +1641,12 @@ static void scsi_unmap_complete(void *opaque, int ret)
r->req.aiocb = NULL;
aio_context_acquire(blk_get_aio_context(s->qdev.conf.blk));
- scsi_unmap_complete_noio(data, ret);
+ if (scsi_disk_req_check_error(r, ret, false)) {
+ scsi_req_unref(&r->req);
+ g_free(data);
+ } else {
+ scsi_unmap_complete_noio(data, ret);
+ }
aio_context_release(blk_get_aio_context(s->qdev.conf.blk));
}
--
2.17.1
- [Qemu-devel] [PATCH v6 0/9] discard blockstats, Anton Nefedov, 2018/11/30
- [Qemu-devel] [PATCH v6 3/9] block: add empty account cookie type, Anton Nefedov, 2018/11/30
- [Qemu-devel] [PATCH v6 2/9] qapi: add unmap to BlockDeviceStats, Anton Nefedov, 2018/11/30
- [Qemu-devel] [PATCH v6 1/9] qapi: group BlockDeviceStats fields, Anton Nefedov, 2018/11/30
- [Qemu-devel] [PATCH v6 4/9] ide: account UNMAP (TRIM) operations, Anton Nefedov, 2018/11/30
- [Qemu-devel] [PATCH v6 6/9] scsi: move unmap error checking to the complete callback,
Anton Nefedov <=
- [Qemu-devel] [PATCH v6 9/9] qapi: query-blockstat: add driver specific file-posix stats, Anton Nefedov, 2018/11/30
- [Qemu-devel] [PATCH v6 7/9] scsi: account unmap operations, Anton Nefedov, 2018/11/30
- [Qemu-devel] [PATCH v6 5/9] scsi: store unmap offset and nb_sectors in request struct, Anton Nefedov, 2018/11/30
- [Qemu-devel] [PATCH v6 8/9] file-posix: account discard operations, Anton Nefedov, 2018/11/30