[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v9 03/10] qcow2_format.py: change Qcow2BitmapExt initialization m
From: |
Andrey Shinkevich |
Subject: |
[PATCH v9 03/10] qcow2_format.py: change Qcow2BitmapExt initialization method |
Date: |
Mon, 13 Jul 2020 10:27:49 +0300 |
There are two ways to initialize a class derived from Qcow2Struct:
1. Pass a block of binary data to the constructor.
2. Pass the file descriptor to allow reading the file from constructor.
Let's change the Qcow2BitmapExt initialization method from 1 to 2 to
support a scattered reading in the initialization chain.
The implementation comes with the patch that follows.
Suggested-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Signed-off-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com>
---
tests/qemu-iotests/qcow2_format.py | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/tests/qemu-iotests/qcow2_format.py
b/tests/qemu-iotests/qcow2_format.py
index 2f3681b..9eb6b5b 100644
--- a/tests/qemu-iotests/qcow2_format.py
+++ b/tests/qemu-iotests/qcow2_format.py
@@ -62,8 +62,8 @@ class Qcow2StructMeta(type):
class Qcow2Struct(metaclass=Qcow2StructMeta):
-
- """Qcow2Struct: base class for qcow2 data structures
+ """
+ Qcow2Struct: base class for qcow2 data structures
Successors should define fields class variable, which is: list of tuples,
each of three elements:
@@ -173,7 +173,13 @@ class QcowHeaderExtension(Qcow2Struct):
self.data_str = data_str
if self.magic == QCOW2_EXT_MAGIC_BITMAPS:
- self.obj = Qcow2BitmapExt(data=self.data)
+ assert fd is not None
+ position = fd.tell()
+ # Step back to reread data
+ padded = (self.length + 7) & ~7
+ fd.seek(-padded, 1)
+ self.obj = Qcow2BitmapExt(fd=fd)
+ fd.seek(position)
else:
self.obj = None
--
1.8.3.1
- [PATCH v9 00/10] iotests: Dump QCOW2 dirty bitmaps metadata, Andrey Shinkevich, 2020/07/13
- [PATCH v9 02/10] qcow2_format.py: make printable data an extension class member, Andrey Shinkevich, 2020/07/13
- [PATCH v9 08/10] qcow2.py: Introduce '-j' key to dump in JSON format, Andrey Shinkevich, 2020/07/13
- [PATCH v9 04/10] qcow2_format.py: dump bitmap flags in human readable way., Andrey Shinkevich, 2020/07/13
- [PATCH v9 03/10] qcow2_format.py: change Qcow2BitmapExt initialization method,
Andrey Shinkevich <=
- [PATCH v9 09/10] qcow2_format.py: collect fields to dump in JSON format, Andrey Shinkevich, 2020/07/13
- [PATCH v9 06/10] qcow2_format.py: pass cluster size to substructures, Andrey Shinkevich, 2020/07/13
- [PATCH v9 10/10] qcow2_format.py: support dumping metadata in JSON format, Andrey Shinkevich, 2020/07/13
- [PATCH v9 07/10] qcow2_format.py: Dump bitmap table serialized entries, Andrey Shinkevich, 2020/07/13
- [PATCH v9 05/10] qcow2_format.py: Dump bitmap directory information, Andrey Shinkevich, 2020/07/13
- [PATCH v9 01/10] qcow2: Fix capitalization of header extension constant., Andrey Shinkevich, 2020/07/13
- Re: [PATCH v9 00/10] iotests: Dump QCOW2 dirty bitmaps metadata, Eric Blake, 2020/07/13