[Devel] [PATCH RHEL8 COMMIT] ploop: Kill "add_delta" interface
Konstantin Khorenko
khorenko at virtuozzo.com
Tue May 4 16:59:25 MSK 2021
The commit is pushed to "branch-rh8-4.18.0-240.1.1.vz8.5.x-ovz" and will appear at https://src.openvz.org/scm/ovz/vzkernel.git
after rh8-4.18.0-240.1.1.vz8.5.25
------>
commit b91a8cb526fab5e5ff4ef47614af8e7d1f59f753
Author: Kirill Tkhai <ktkhai at virtuozzo.com>
Date: Tue May 4 16:59:25 2021 +0300
ploop: Kill "add_delta" interface
This was used at development time, but now this is
useless. Userspace never adds a delta into middle
of deltas stack.
To_merge: 3e892fb36c57 ("dm: Interpreter of ploop1 format (ploop driver)")
Signed-off-by: Kirill Tkhai <ktkhai at virtuozzo.com>
=====================
Patchset description:
ploop: Preparation to kill-loop from dm-ploop (series 1)
Currently, ploop requires underlining loop for top delta. This is overkill,
since we may submit bios directly to fs, without using it.
Initially there was a possibility, we may use device-mapper format snapshots
instead of ploop-based. But we don't do that, so we have to use loop for top
delta and directly submit bios to the rest of deltas. So, these two ways to do
the same actions are wrong.
This is preparations#1 for further series.
Let's this goes to testing earlier.
Kirill Tkhai (7):
ploop: Kill "add_delta" interface
ploop: Refactor ploop_add_delta()
ploop: Move check in ploop_ctr()
ploop: Call add_delta directly
ploop: Move add_delta iterating code to separate function
ploop: Allocate deltas in only place
ploop: Add ignore_signature_disk_in_use parameter
https://jira.sw.ru/browse/PSBM-123654
Signed-off-by: Kirill Tkhai <ktkhai at virtuozzo.com>
---
drivers/md/dm-ploop-cmd.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/drivers/md/dm-ploop-cmd.c b/drivers/md/dm-ploop-cmd.c
index 31fa424a88f5..85ba713749cf 100644
--- a/drivers/md/dm-ploop-cmd.c
+++ b/drivers/md/dm-ploop-cmd.c
@@ -1846,10 +1846,6 @@ int ploop_message(struct dm_target *ti, unsigned int argc, char **argv,
if (argc != 2 || kstrtou64(argv[1], 10, &val) < 0)
goto unlock;
ret = ploop_resize(ploop, val);
- } else if (!strcmp(argv[0], "add_delta")) {
- if (argc != 2)
- goto unlock;
- ret = ploop_add_delta(ploop, argv[1]);
} else if (!strcmp(argv[0], "merge")) {
if (argc == 1)
ret = ploop_merge_latest_snapshot(ploop);
More information about the Devel
mailing list