[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 10/19] parallels: Let image extensions work in RW mode
From: |
Alexander Ivanov |
Subject: |
[PATCH 10/19] parallels: Let image extensions work in RW mode |
Date: |
Mon, 2 Oct 2023 10:57:29 +0200 |
Now we support extensions saving and can let to work with them in
read-write mode.
Signed-off-by: Alexander Ivanov <alexander.ivanov@virtuozzo.com>
---
block/parallels-ext.c | 4 ----
block/parallels.c | 17 ++++-------------
2 files changed, 4 insertions(+), 17 deletions(-)
diff --git a/block/parallels-ext.c b/block/parallels-ext.c
index 0a632a2331..a0de4f1b07 100644
--- a/block/parallels-ext.c
+++ b/block/parallels-ext.c
@@ -177,10 +177,6 @@ static BdrvDirtyBitmap
*parallels_load_bitmap(BlockDriverState *bs,
return NULL;
}
- /* We support format extension only for RO parallels images. */
- assert(!(bs->open_flags & BDRV_O_RDWR));
- bdrv_dirty_bitmap_set_readonly(bitmap, true);
-
return bitmap;
}
diff --git a/block/parallels.c b/block/parallels.c
index d2a45e0c04..772a8ca1eb 100644
--- a/block/parallels.c
+++ b/block/parallels.c
@@ -1371,19 +1371,10 @@ static int parallels_open(BlockDriverState *bs, QDict
*options, int flags,
}
if (ph.ext_off) {
- if (flags & BDRV_O_RDWR) {
- /*
- * It's unsafe to open image RW if there is an extension (as we
- * don't support it). But parallels driver in QEMU historically
- * ignores the extension, so print warning and don't care.
- */
- warn_report("Format Extension ignored in RW mode");
- } else {
- ret = parallels_read_format_extension(
- bs, le64_to_cpu(ph.ext_off) << BDRV_SECTOR_BITS, errp);
- if (ret < 0) {
- goto fail;
- }
+ ret = parallels_read_format_extension(
+ bs, le64_to_cpu(ph.ext_off) << BDRV_SECTOR_BITS, errp);
+ if (ret < 0) {
+ goto fail;
}
}
--
2.34.1
- [PATCH 06/19] parallels: Add a note about used bitmap in parallels_check_duplicate(), (continued)
- [PATCH 06/19] parallels: Add a note about used bitmap in parallels_check_duplicate(), Alexander Ivanov, 2023/10/02
- [PATCH 03/19] parallels: Move host clusters allocation to a separate function, Alexander Ivanov, 2023/10/02
- [PATCH 05/19] parallels: Recreate used bitmap in parallels_check_leak(), Alexander Ivanov, 2023/10/02
- [PATCH 01/19] parallels: Move inactivation code to a separate function, Alexander Ivanov, 2023/10/02
- [PATCH 11/19] parallels: Handle L1 entries equal to one, Alexander Ivanov, 2023/10/02
- [PATCH 07/19] parallels: Create used bitmap even if checks needed, Alexander Ivanov, 2023/10/02
- [PATCH 17/19] tests: Add parallels images support to test 165, Alexander Ivanov, 2023/10/02
- [PATCH 09/19] parallels: Add dirty bitmaps saving, Alexander Ivanov, 2023/10/02
- [PATCH 08/19] parallels: Make mark_used() and mark_unused() global functions, Alexander Ivanov, 2023/10/02
- [PATCH 14/19] parallels: Truncate images on the last used cluster, Alexander Ivanov, 2023/10/02
- [PATCH 10/19] parallels: Let image extensions work in RW mode,
Alexander Ivanov <=
- [PATCH 12/19] parallels: Make a loaded dirty bitmap persistent, Alexander Ivanov, 2023/10/02
- [PATCH 15/19] parallels: Remove unnecessary data_end field, Alexander Ivanov, 2023/10/02
- Re: [PATCH 15/19] parallels: Remove unnecessary data_end field, Mike Maslenkin, 2023/10/06
- Re: [PATCH 15/19] parallels: Remove unnecessary data_end field, Alexander Ivanov, 2023/10/07
- Re: [PATCH 15/19] parallels: Remove unnecessary data_end field, Mike Maslenkin, 2023/10/07
- Re: [PATCH 15/19] parallels: Remove unnecessary data_end field, Alexander Ivanov, 2023/10/07
- Re: [PATCH 15/19] parallels: Remove unnecessary data_end field, Mike Maslenkin, 2023/10/07
- Re: [PATCH 15/19] parallels: Remove unnecessary data_end field, Alexander Ivanov, 2023/10/08
[PATCH 13/19] parallels: Reverse a conditional in parallels_check_leak() to reduce indents, Alexander Ivanov, 2023/10/02
[PATCH 19/19] tests: Add parallels format support to image-fleecing, Alexander Ivanov, 2023/10/02