[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v5 0/6] Discrad blocks during block-stream operation
From: |
Andrey Shinkevich |
Subject: |
[Qemu-devel] [PATCH v5 0/6] Discrad blocks during block-stream operation |
Date: |
Sun, 30 Dec 2018 23:09:11 +0300 |
Dear collegues,
The given feature discards blocks with copy-on-read operation while the
streaming process runs. Adding the 'discard' argument to the QMP block-stream
command allows dropping a block in the backing chain after it has been copied
to the active layer. That will elude the block duplication in the intermediate
backing file. It saves the disk space while external snapshots are being
merged.
The method involves the filter insertion above the active layer to allow write
operation in the backing chain. The method is similar to that in the 'commit
active' command (mirror.c).
The permission to write into an inactive layer can not be obtained due to the
existing child permission mechanism.
v5:
The existing copy-on-read filter driver was applied to the block-stream
operation in the separated patch. The code refactoring in 'stream.c' is big.
Unfortunately, there are issues with getting "write" lock in iotests -
subject to further resolution.
v4:
"[PATCH v4 0/5] Discrad blocks during block-stream operation"
The targeted version of the release at 'Since' word of the comment to 'discard'
argument changed to 4.0 in the file qapi/block-core.json as Eric Blake noticed.
v3:
"[PATCH v3 0/5] Discrad blocks during block-stream operation"
The series version number and the given changelog were added. 'Signed-off-by'
subscription of the patch [PATCH 3/5] amended as Peter Krempa requested.
v2:
"[PATCH 0/5] Discrad blocks during block-stream operation"
The suggestions of Dr. David Alan Gilbert and Alberto Garcia after their first
review have been applied.
The abort() callback function have been added to the filter driver and some
refactoring work done.
The test suite for the 'block-stream' command with 'discard' option was added
to the 030 file of iotests.
v1:
"[PATCH 0/2] Discrad blocks during block-stream operation"
The initial implementation of the feature.
Sincerely,
Andrey Shinkevich (6):
Stream block job involves copy-on-read filter driver
Discard blocks while copy-on-read
The discard flag for block stream operation
iotests: allow resume_drive by node name
iotests: prepare 030 for graph change
iotests: 030 with block-stream discard
block/copy-on-read.c | 24 ++++
block/stream.c | 290 ++++++++++++++++++++++++++++++++++++++++--
blockdev.c | 8 +-
hmp-commands.hx | 4 +-
hmp.c | 4 +-
include/block/block_int.h | 2 +-
qapi/block-core.json | 5 +-
tests/qemu-iotests/030 | 163 ++++++++++++++++++------
tests/qemu-iotests/030.out | 4 +-
tests/qemu-iotests/iotests.py | 9 +-
10 files changed, 448 insertions(+), 65 deletions(-)
--
1.8.3.1
- [Qemu-devel] [PATCH v5 0/6] Discrad blocks during block-stream operation,
Andrey Shinkevich <=
- [Qemu-devel] [PATCH v5 4/6] iotests: allow resume_drive by node name, Andrey Shinkevich, 2018/12/30
- [Qemu-devel] [PATCH v5 3/6] The discard flag for block stream operation, Andrey Shinkevich, 2018/12/30
- [Qemu-devel] [PATCH v5 1/6] Stream block job involves copy-on-read filter driver, Andrey Shinkevich, 2018/12/30
- [Qemu-devel] [PATCH v5 5/6] iotests: prepare 030 for graph change, Andrey Shinkevich, 2018/12/30
- [Qemu-devel] [PATCH v5 2/6] Discard blocks while copy-on-read, Andrey Shinkevich, 2018/12/30
- [Qemu-devel] [PATCH v5 6/6] iotests: 030 with block-stream discard, Andrey Shinkevich, 2018/12/30