summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Teigland <teigland@redhat.com>2020-02-11 12:59:39 -0600
committerDavid Teigland <teigland@redhat.com>2020-02-11 13:01:13 -0600
commitcba06012acc589888ef88221f1a580b5b81b4100 (patch)
treee4b1db1c56edb8dcd6a9af952561abe79bed2f78
parent892a182975049ebe732860c30c866fe2de5b7717 (diff)
downloadlvm2-cba06012acc589888ef88221f1a580b5b81b4100.tar.gz
writecache: check if cachevol is writable
before trying to initialize it (since wipe_lv does not return an error if it fails to write.)
-rw-r--r--tools/lvconvert.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/tools/lvconvert.c b/tools/lvconvert.c
index cd9a3e8f9..aa2dca7f5 100644
--- a/tools/lvconvert.c
+++ b/tools/lvconvert.c
@@ -5351,6 +5351,11 @@ static int _writecache_zero(struct cmd_context *cmd, struct logical_volume *lv)
};
int ret;
+ if (!(lv->status & LVM_WRITE)) {
+ log_error("Cannot initialize readonly LV %s", display_lvname(lv));
+ return 0;
+ }
+
if (test_mode())
return 1;