|
 |
31f061 |
lib/metadata/writecache_manip.c | 10 +++++++---
|
|
|
31f061 |
tools/lvconvert.c | 2 ++
|
|
|
31f061 |
2 files changed, 9 insertions(+), 3 deletions(-)
|
|
|
31f061 |
|
|
|
31f061 |
diff --git a/lib/metadata/writecache_manip.c b/lib/metadata/writecache_manip.c
|
|
|
31f061 |
index 5004aa9..8150d07 100644
|
|
|
31f061 |
--- a/lib/metadata/writecache_manip.c
|
|
|
31f061 |
+++ b/lib/metadata/writecache_manip.c
|
|
|
31f061 |
@@ -75,7 +75,7 @@ static int _get_writecache_kernel_status(struct cmd_context *cmd,
|
|
|
31f061 |
return 0;
|
|
|
31f061 |
}
|
|
|
31f061 |
|
|
|
31f061 |
- if (!lv_info_with_seg_status(cmd, first_seg(lv), &status, 1, 1)) {
|
|
|
31f061 |
+ if (!lv_info_with_seg_status(cmd, first_seg(lv), &status, 0, 0)) {
|
|
|
31f061 |
log_error("Failed to get device mapper status for %s", display_lvname(lv));
|
|
|
31f061 |
goto fail;
|
|
|
31f061 |
}
|
|
|
31f061 |
@@ -434,8 +434,12 @@ int lv_writecache_set_cleaner(struct logical_volume *lv)
|
|
|
31f061 |
seg->writecache_settings.cleaner_set = 1;
|
|
|
31f061 |
|
|
|
31f061 |
if (lv_is_active(lv)) {
|
|
|
31f061 |
- if (!lv_update_and_reload(lv)) {
|
|
|
31f061 |
- log_error("Failed to update VG and reload LV.");
|
|
|
31f061 |
+ if (!vg_write(lv->vg) || !vg_commit(lv->vg)) {
|
|
|
31f061 |
+ log_error("Failed to update VG.");
|
|
|
31f061 |
+ return 0;
|
|
|
31f061 |
+ }
|
|
|
31f061 |
+ if (!lv_writecache_message(lv, "cleaner")) {
|
|
|
31f061 |
+ log_error("Failed to set writecache cleaner for %s.", display_lvname(lv));
|
|
|
31f061 |
return 0;
|
|
|
31f061 |
}
|
|
|
31f061 |
} else {
|
|
|
31f061 |
diff --git a/tools/lvconvert.c b/tools/lvconvert.c
|
|
|
31f061 |
index 4323965..7b74afb 100644
|
|
|
31f061 |
--- a/tools/lvconvert.c
|
|
|
31f061 |
+++ b/tools/lvconvert.c
|
|
|
31f061 |
@@ -5720,6 +5720,8 @@ static int _lvconvert_detach_writecache_when_clean(struct cmd_context *cmd,
|
|
|
31f061 |
return 0;
|
|
|
31f061 |
}
|
|
|
31f061 |
|
|
|
31f061 |
+ log_debug("detach writecache check clean reading vg %s", id->vg_name);
|
|
|
31f061 |
+
|
|
|
31f061 |
vg = vg_read(cmd, id->vg_name, NULL, READ_FOR_UPDATE, lockd_state, &error_flags, NULL);
|
|
|
31f061 |
|
|
|
31f061 |
if (!vg) {
|