From c813db8fc2ec15a8370bb291f662e15ba26682e8 Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac <zkabelac@redhat.com> Date: Mon, 30 Sep 2019 13:22:42 +0200 Subject: [PATCH] vdo: deactivate forgotten vdo pool If the linear mapping is lost (for whatever reason, i.e. test suite forcible 'dmsetup remove' linear LV, lvm2 had hard times figuring out how to deactivate such DM table. So add function which is in case inactive VDO pool LV checks if the pool is actually still active (-vpool device present) and it has open count == 0. In this case deactivation is allowed to continue and cleanup DM table. --- lib/activate/activate.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/lib/activate/activate.c b/lib/activate/activate.c index 29cd2d3aa..258881633 100644 --- a/lib/activate/activate.c +++ b/lib/activate/activate.c @@ -2434,6 +2434,17 @@ int lv_deactivate(struct cmd_context *cmd, const char *lvid_s, const struct logi } } + if (lv_is_vdo_pool(lv)) { + /* If someone has remove 'linear' mapping over VDO device + * we may still be able to deactivate the rest of the tree + * i.e. in test-suite we simulate this via 'dmsetup remove' */ + if (!lv_info(cmd, lv, 1, &info, 1, 0)) + goto_out; + + if (info.exists && !info.open_count) + r = 0; /* Unused VDO device left in table? */ + } + if (r) goto out; }