From a23588d77cb1585e642946fe84c3eaafe7e2d270 Mon Sep 17 00:00:00 2001 From: David Teigland Date: Fri, 30 Sep 2022 12:16:32 -0500 Subject: lvresize: move the lockd_lv earlier the lock should cover any potential activation, not just the resizing changes --- lib/metadata/lv_manip.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c index 99045c220..d6452e849 100644 --- a/lib/metadata/lv_manip.c +++ b/lib/metadata/lv_manip.c @@ -6814,6 +6814,13 @@ int lv_resize(struct cmd_context *cmd, struct logical_volume *lv, return 0; } + /* + * If the LV is locked due to being active, this lock call is a no-op. + * Otherwise, this acquires a transient lock on the lv (not PERSISTENT) + */ + if (!lockd_lv_resize(cmd, lv_top, "ex", 0, lp)) + return_0; + /* * Active 'hidden' -tpool can be waiting for resize, but the pool LV * itself might be inactive. Here plain suspend/resume would not work. @@ -6909,13 +6916,6 @@ int lv_resize(struct cmd_context *cmd, struct logical_volume *lv, if (is_reduce && !lp->fsopt[0] && !_lv_reduce_confirmation(lv_top, lp)) goto_out; - /* - * If the LV is locked due to being active, this lock call is a no-op. - * Otherwise, this acquires a transient lock on the lv (not PERSISTENT) - */ - if (!lockd_lv_resize(cmd, lv_top, "ex", 0, lp)) - goto_out; - /* Part of old approach to fs handling using fsadm. */ if (!strcmp(lp->fsopt, "resize_fsadm") && !lp->nofsck && !_fsadm_cmd(FSADM_CMD_CHECK, lv_top, 0, lp->yes, lp->force, &status)) { -- cgit v1.2.1