From 1404dc2fbf6c8bb482825b74dc4f1ac1d9bf841d Mon Sep 17 00:00:00 2001 From: Alasdair Kergon Date: Wed, 16 May 2012 12:50:14 +0000 Subject: [PATCH] Re-enable partial activation of non-thin LVs until it can be fixed. (2.02.90) - The test should be checking the LV as a whole, not just individual segments. --- WHATS_NEW | 1 + lib/activate/dev_manager.c | 16 ---------------- 2 files changed, 1 insertion(+), 16 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index 03cb07dd7..c19646896 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,5 +1,6 @@ Version 2.02.96 - ================================ + Re-enable partial activation of non-thin LVs until it can be fixed. (2.02.90) Fix alloc cling to cling to PVs already found with contiguous policy. Fix cling policy not to behave like normal policy if no previous LV seg. Fix allocation loop not to use later policies when --alloc cling without tags. diff --git a/lib/activate/dev_manager.c b/lib/activate/dev_manager.c index be99e8cfd..40b0edc8b 100644 --- a/lib/activate/dev_manager.c +++ b/lib/activate/dev_manager.c @@ -1581,22 +1581,6 @@ int add_areas_line(struct dev_manager *dm, struct lv_segment *seg, seg->lv->vg->name, seg->lv->name); return 0; } - - /* - * Mirrors activate LVs replaced with error targets and - * RAID can handle non-accessible sub-LVs. - * - * TODO: Can we eventually skip to activate such LVs ? - */ - if (!num_existing_areas && - !strstr(seg->lv->name, "_rmeta_") && - !strstr(seg->lv->name, "_rimage_") && - !strstr(seg->lv->name, "_mimage_") && - !((name = strstr(seg->lv->name, "_mlog")) && !name[5])) { - log_error("Cannot activate %s/%s: all segments missing.", - seg->lv->vg->name, seg->lv->name); - return 0; - } } return 1;