|
 |
902636 |
From 23b92512d7f11b3a38cf24a5c2fe7848f1e550e8 Mon Sep 17 00:00:00 2001
|
|
|
902636 |
From: Maxim Levitsky <mlevitsk@redhat.com>
|
|
|
902636 |
Date: Sun, 31 May 2020 16:40:34 +0100
|
|
|
902636 |
Subject: [PATCH 6/7] block.c: adding bdrv_co_delete_file
|
|
|
902636 |
MIME-Version: 1.0
|
|
|
902636 |
Content-Type: text/plain; charset=UTF-8
|
|
|
902636 |
Content-Transfer-Encoding: 8bit
|
|
|
902636 |
|
|
|
902636 |
RH-Author: Maxim Levitsky <mlevitsk@redhat.com>
|
|
|
902636 |
Message-id: <20200531164035.34188-3-mlevitsk@redhat.com>
|
|
|
902636 |
Patchwork-id: 97058
|
|
|
902636 |
O-Subject: [RHEL-AV-8.2.1 qemu-kvm PATCH 2/3] block.c: adding bdrv_co_delete_file
|
|
|
902636 |
Bugzilla: 1827630
|
|
|
902636 |
RH-Acked-by: Sergio Lopez Pascual <slp@redhat.com>
|
|
|
902636 |
RH-Acked-by: John Snow <jsnow@redhat.com>
|
|
|
902636 |
RH-Acked-by: Eric Blake <eblake@redhat.com>
|
|
|
902636 |
|
|
|
902636 |
From: Daniel Henrique Barboza <danielhb413@gmail.com>
|
|
|
902636 |
|
|
|
902636 |
Using the new 'bdrv_co_delete_file' interface, a pure co_routine function
|
|
|
902636 |
'bdrv_co_delete_file' inside block.c can can be used in a way similar of
|
|
|
902636 |
the existing bdrv_create_file to to clean up a created file.
|
|
|
902636 |
|
|
|
902636 |
We're creating a pure co_routine because the only caller of
|
|
|
902636 |
'bdrv_co_delete_file' will be already in co_routine context, thus there
|
|
|
902636 |
is no need to add all the machinery to check for qemu_in_coroutine() and
|
|
|
902636 |
create a separated co_routine to do the job.
|
|
|
902636 |
|
|
|
902636 |
Suggested-by: Daniel P. Berrangé <berrange@redhat.com>
|
|
|
902636 |
Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
|
|
|
902636 |
Message-Id: <20200130213907.2830642-3-danielhb413@gmail.com>
|
|
|
902636 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
902636 |
(cherry picked from commit e1d7f8bb1ec0c6911dcea81641ce6139dbded02d)
|
|
|
902636 |
Signed-off-by: Maxim Levitsky <mlevitsk@redhat.com>
|
|
|
902636 |
Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>
|
|
|
902636 |
---
|
|
|
902636 |
block.c | 26 ++++++++++++++++++++++++++
|
|
|
902636 |
include/block/block.h | 1 +
|
|
|
902636 |
2 files changed, 27 insertions(+)
|
|
|
902636 |
|
|
|
902636 |
diff --git a/block.c b/block.c
|
|
|
902636 |
index ba3b40d7..d6a05da 100644
|
|
|
902636 |
--- a/block.c
|
|
|
902636 |
+++ b/block.c
|
|
|
902636 |
@@ -672,6 +672,32 @@ int bdrv_create_file(const char *filename, QemuOpts *opts, Error **errp)
|
|
|
902636 |
return bdrv_create(drv, filename, opts, errp);
|
|
|
902636 |
}
|
|
|
902636 |
|
|
|
902636 |
+int coroutine_fn bdrv_co_delete_file(BlockDriverState *bs, Error **errp)
|
|
|
902636 |
+{
|
|
|
902636 |
+ Error *local_err = NULL;
|
|
|
902636 |
+ int ret;
|
|
|
902636 |
+
|
|
|
902636 |
+ assert(bs != NULL);
|
|
|
902636 |
+
|
|
|
902636 |
+ if (!bs->drv) {
|
|
|
902636 |
+ error_setg(errp, "Block node '%s' is not opened", bs->filename);
|
|
|
902636 |
+ return -ENOMEDIUM;
|
|
|
902636 |
+ }
|
|
|
902636 |
+
|
|
|
902636 |
+ if (!bs->drv->bdrv_co_delete_file) {
|
|
|
902636 |
+ error_setg(errp, "Driver '%s' does not support image deletion",
|
|
|
902636 |
+ bs->drv->format_name);
|
|
|
902636 |
+ return -ENOTSUP;
|
|
|
902636 |
+ }
|
|
|
902636 |
+
|
|
|
902636 |
+ ret = bs->drv->bdrv_co_delete_file(bs, &local_err);
|
|
|
902636 |
+ if (ret < 0) {
|
|
|
902636 |
+ error_propagate(errp, local_err);
|
|
|
902636 |
+ }
|
|
|
902636 |
+
|
|
|
902636 |
+ return ret;
|
|
|
902636 |
+}
|
|
|
902636 |
+
|
|
|
902636 |
/**
|
|
|
902636 |
* Try to get @bs's logical and physical block size.
|
|
|
902636 |
* On success, store them in @bsz struct and return 0.
|
|
|
902636 |
diff --git a/include/block/block.h b/include/block/block.h
|
|
|
902636 |
index 92685d2..b2a3074 100644
|
|
|
902636 |
--- a/include/block/block.h
|
|
|
902636 |
+++ b/include/block/block.h
|
|
|
902636 |
@@ -373,6 +373,7 @@ bool bdrv_is_backing_chain_frozen(BlockDriverState *bs, BlockDriverState *base,
|
|
|
902636 |
int bdrv_freeze_backing_chain(BlockDriverState *bs, BlockDriverState *base,
|
|
|
902636 |
Error **errp);
|
|
|
902636 |
void bdrv_unfreeze_backing_chain(BlockDriverState *bs, BlockDriverState *base);
|
|
|
902636 |
+int coroutine_fn bdrv_co_delete_file(BlockDriverState *bs, Error **errp);
|
|
|
902636 |
|
|
|
902636 |
|
|
|
902636 |
typedef struct BdrvCheckResult {
|
|
|
902636 |
--
|
|
|
902636 |
1.8.3.1
|
|
|
902636 |
|