lvm2-commits December 2018

lvm2-commits@lists.fedorahosted.org

5 participants
96 discussions

master - libdm: use libdm header
by Zdenek Kabelac 14 Dec '18

14 Dec '18

Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=3c377643339be97940554… Commit: 3c377643339be97940554323909d51b12c933f9e Parent: 46a8d2e898a48b1df86fe07d13df8aa994e32808 Author: Zdenek Kabelac <zkabelac(a)redhat.com> AuthorDate: Fri Dec 14 13:11:48 2018 +0100 Committer: Zdenek Kabelac <zkabelac(a)redhat.com> CommitterDate: Fri Dec 14 15:08:34 2018 +0100 libdm: use libdm header User libdm header file. --- WHATS_NEW_DM | 1 + libdm/datastruct/list.c | 2 +- 2 files changed, 2 insertions(+), 1 deletions(-) diff --git a/WHATS_NEW_DM b/WHATS_NEW_DM index aa1734c..3d45bc3 100644 --- a/WHATS_NEW_DM +++ b/WHATS_NEW_DM @@ -1,5 +1,6 @@ Version 1.02.155 - ==================================== + Include correct internal header inside libdm list.c. Enhance ioctl flattening and add parameters only when needed. Add DM_DEVICE_ARM_POLL for API completness matching kernel. Do not add parameters for RESUME with DM_DEVICE_CREATE dm task. diff --git a/libdm/datastruct/list.c b/libdm/datastruct/list.c index d80cb9a..74053c5 100644 --- a/libdm/datastruct/list.c +++ b/libdm/datastruct/list.c @@ -13,7 +13,7 @@ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#include "lib/misc/lib.h" +#include "libdm/misc/dmlib.h" #include <assert.h> /*

1 0

master - dmeventd: Fix libdevmapper-event linking
by Marian Csontos 12 Dec '18

12 Dec '18

Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=46a8d2e898a48b1df86fe… Commit: 46a8d2e898a48b1df86fe07d13df8aa994e32808 Parent: 627f0e2bd84753a7bfbda517f22c54a929e3a458 Author: Marian Csontos <mcsontos(a)redhat.com> AuthorDate: Wed Dec 12 10:50:36 2018 +0100 Committer: Marian Csontos <mcsontos(a)redhat.com> CommitterDate: Wed Dec 12 15:43:03 2018 +0100 dmeventd: Fix libdevmapper-event linking --- daemons/dmeventd/Makefile.in | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/daemons/dmeventd/Makefile.in b/daemons/dmeventd/Makefile.in index 2d856f7..0a03d66 100644 --- a/daemons/dmeventd/Makefile.in +++ b/daemons/dmeventd/Makefile.in @@ -58,11 +58,12 @@ device-mapper: $(TARGETS) CFLAGS_dmeventd.o += $(EXTRA_EXEC_CFLAGS) LIBS += $(PTHREAD_LIBS) +LIBS += $(INTERNAL_LIBS) dmeventd: $(LIB_SHARED) dmeventd.o @echo " [CC] $@" $(Q) $(CC) $(CFLAGS) -L. $(LDFLAGS) $(EXTRA_EXEC_LDFLAGS) $(ELDFLAGS) dmeventd.o \ - -o $@ $(DL_LIBS) $(DMEVENT_LIBS) $(INTERNAL_LIBS) $(LIBS) -lm + -o $@ $(DL_LIBS) $(DMEVENT_LIBS) $(LIBS) -lm dmeventd.static: $(LIB_STATIC) dmeventd.o @echo " [CC] $@"

1 0

master - man: document 's' RAID attribute bit
by Heinz Mauelshagen 11 Dec '18

11 Dec '18

Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=627f0e2bd84753a7bfbda… Commit: 627f0e2bd84753a7bfbda517f22c54a929e3a458 Parent: dd5716ddf258c4a44819fa90d3356833ccf767b4 Author: Heinz Mauelshagen <heinzm(a)redhat.com> AuthorDate: Tue Dec 11 16:54:17 2018 +0100 Committer: Heinz Mauelshagen <heinzm(a)redhat.com> CommitterDate: Tue Dec 11 16:54:17 2018 +0100 man: document 's' RAID attribute bit --- man/lvs.8_end | 2 ++ man/lvs.8_pregen | 2 ++ 2 files changed, 4 insertions(+), 0 deletions(-) diff --git a/man/lvs.8_end b/man/lvs.8_end index f24ffe9..4f83516 100644 --- a/man/lvs.8_end +++ b/man/lvs.8_end @@ -56,6 +56,8 @@ Inconsistencies are detected by initiating a "check" on a RAID logical volume. (The scrubbing operations, "check" and "repair", can be performed on a RAID logical volume via the 'lvchange' command.) (w)ritemostly signifies the devices in a RAID 1 logical volume that have been marked write-mostly. +Re(s)haping signifies a RAID Logical Volume is either undergoing a stripe +addition/removal, a stripe size or RAID algorithm change. (R)emove after reshape signifies freed striped raid images to be removed. .IP Related to Thin pool Logical Volumes: (F)ailed, out of (D)ata space, diff --git a/man/lvs.8_pregen b/man/lvs.8_pregen index cbc0ed8..377afc2 100644 --- a/man/lvs.8_pregen +++ b/man/lvs.8_pregen @@ -569,6 +569,8 @@ Inconsistencies are detected by initiating a "check" on a RAID logical volume. (The scrubbing operations, "check" and "repair", can be performed on a RAID logical volume via the 'lvchange' command.) (w)ritemostly signifies the devices in a RAID 1 logical volume that have been marked write-mostly. +Re(s)haping shows a Raid Logical Volume is undergoing a reshape (ie. striped + addition/removal, stripe size or RAID algorithm change). (R)emove after reshape signifies freed striped raid images to be removed. .IP Related to Thin pool Logical Volumes: (F)ailed, out of (D)ata space,

1 0

master - raid: fix (de)activation of RaidLVs with visible SubLVs
by Heinz Mauelshagen 11 Dec '18

11 Dec '18

Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=dd5716ddf258c4a44819f… Commit: dd5716ddf258c4a44819fa90d3356833ccf767b4 Parent: b2d1facd960e192b38ae4a9866f3bea9646e6cd5 Author: Heinz Mauelshagen <heinzm(a)redhat.com> AuthorDate: Wed Oct 31 23:05:08 2018 +0100 Committer: Heinz Mauelshagen <heinzm(a)redhat.com> CommitterDate: Tue Dec 11 16:35:34 2018 +0100 raid: fix (de)activation of RaidLVs with visible SubLVs There's a small window during creation of a new RaidLV when rmeta SubLVs are made visible to wipe them in order to prevent erroneous discovery of stale RAID metadata. In case a crash prevents the SubLVs from being committed hidden after such wiping, the RaidLV can still be activated with the SubLVs visible. During deactivation though, a deadlock occurs because the visible SubLVs are deactivated before the RaidLV. The patch adds _check_raid_sublvs to the raid validation in merge.c, an activation check to activate.c (paranoid, because the merge.c check will prevent activation in case of visible SubLVs) and shares the existing wiping function _clear_lvs in raid_manip.c moved to lv_manip.c and renamed to activate_and_wipe_lvlist to remove code duplication. Whilst on it, introduce activate_and_wipe_lv to share with (lvconvert|lvchange).c. Resolves: rhbz1633167 --- WHATS_NEW | 1 + lib/activate/activate.c | 6 + lib/metadata/lv_manip.c | 190 ++++++++++++++++++++++++++++---------- lib/metadata/merge.c | 30 ++++++ lib/metadata/metadata-exported.h | 8 ++ lib/metadata/raid_manip.c | 91 ++++-------------- tools/lvchange.c | 32 +------ tools/lvconvert.c | 20 +--- 8 files changed, 213 insertions(+), 165 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index ffc0d6a..6856d66 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,5 +1,6 @@ Version 2.03.02 - =================================== + Fix (de)activation of RaidLVs with visible SubLVs Prohibit mirrored 'mirror' log via lvcreate and lvconvert Use sync io if async io_setup fails, or use_aio=0 is set in config. diff --git a/lib/activate/activate.c b/lib/activate/activate.c index ce9d67c..678dec9 100644 --- a/lib/activate/activate.c +++ b/lib/activate/activate.c @@ -2532,6 +2532,12 @@ static int _lv_activate(struct cmd_context *cmd, const char *lvid_s, goto out; } + if (lv_raid_has_visible_sublvs(lv)) { + log_error("Refusing activation of RAID LV %s with " + "visible SubLVs.", display_lvname(lv)); + goto out; + } + if (test_mode()) { _skip("Activating %s.", display_lvname(lv)); r = 1; diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c index d5603c7..2c569d8 100644 --- a/lib/metadata/lv_manip.c +++ b/lib/metadata/lv_manip.c @@ -4051,6 +4051,25 @@ bad: return 0; } +/* Add all rmeta SubLVs for @seg to @lvs and return allocated @lvl to free by caller. */ +static struct lv_list *_raid_list_metalvs(struct lv_segment *seg, struct dm_list *lvs) +{ + uint32_t s; + struct lv_list *lvl; + + dm_list_init(lvs); + + if (!(lvl = dm_pool_alloc(seg->lv->vg->vgmem, sizeof(*lvl) * seg->area_count))) + return_NULL; + + for (s = 0; s < seg->area_count; s++) { + lvl[s].lv = seg_metalv(seg, s); + dm_list_add(lvs, &lvl[s].list); + } + + return lvl; +} + static int _lv_extend_layered_lv(struct alloc_handle *ah, struct logical_volume *lv, uint32_t extents, uint32_t first_area, @@ -4062,7 +4081,6 @@ static int _lv_extend_layered_lv(struct alloc_handle *ah, uint32_t fa, s; int clear_metadata = 0; uint32_t area_multiple = 1; - int fail; if (!(segtype = get_segtype_from_string(lv->vg->cmd, SEG_TYPE_NAME_STRIPED))) return_0; @@ -4140,74 +4158,50 @@ static int _lv_extend_layered_lv(struct alloc_handle *ah, return_0; if (clear_metadata) { + struct volume_group *vg = lv->vg; + /* * We must clear the metadata areas upon creation. */ - /* FIXME VG is not in a fully-consistent state here and should not be committed! */ - if (!vg_write(lv->vg) || !vg_commit(lv->vg)) - return_0; - - if (test_mode()) - log_verbose("Test mode: Skipping wiping of metadata areas."); - else { - fail = 0; - /* Activate all rmeta devices locally first (more efficient) */ - for (s = 0; !fail && s < seg->area_count; s++) { - meta_lv = seg_metalv(seg, s); - - if (!activate_lv(meta_lv->vg->cmd, meta_lv)) { - log_error("Failed to activate %s for clearing.", - display_lvname(meta_lv)); - fail = 1; - } - } - - /* Clear all rmeta devices */ - for (s = 0; !fail && s < seg->area_count; s++) { - meta_lv = seg_metalv(seg, s); - log_verbose("Clearing metadata area of %s.", - display_lvname(meta_lv)); - /* - * Rather than wiping meta_lv->size, we can simply - * wipe '1' to remove the superblock of any previous - * RAID devices. It is much quicker. - */ - if (!wipe_lv(meta_lv, (struct wipe_params) - { .do_zero = 1, .zero_sectors = 1 })) { - stack; - fail = 1; - } - } - - /* Deactivate all rmeta devices */ - for (s = 0; s < seg->area_count; s++) { - meta_lv = seg_metalv(seg, s); + /* + * Declare the new RaidLV as temporary to avoid visible SubLV + * failures on activation until after we wiped them so that + * we can avoid activating crashed, potentially partially + * wiped RaidLVs. + */ + lv->status |= LV_ACTIVATION_SKIP; - if (!deactivate_lv(meta_lv->vg->cmd, meta_lv)) { - log_error("Failed to deactivate %s after clearing.", - display_lvname(meta_lv)); - fail = 1; - } + if (test_mode()) { + /* FIXME VG is not in a fully-consistent state here and should not be committed! */ + if (!vg_write(vg) || !vg_commit(vg)) + return_0; - /* Wipe any temporary tags required for activation. */ - str_list_wipe(&meta_lv->tags); - } + log_verbose("Test mode: Skipping wiping of metadata areas."); + } else { + struct dm_list meta_lvs; + struct lv_list *lvl; - if (fail) { - /* Fail, after trying to deactivate all we could */ - struct volume_group *vg = lv->vg; + if (!(lvl = _raid_list_metalvs(seg, &meta_lvs))) + return 0; + /* Wipe lv list committing metadata */ + if (!activate_and_wipe_lvlist(&meta_lvs, 1)) { + /* If we failed clearing rmeta SubLVs, try removing the new RaidLV */ if (!lv_remove(lv)) log_error("Failed to remove LV"); else if (!vg_write(vg) || !vg_commit(vg)) log_error("Failed to commit VG %s", vg->name); return_0; } + + dm_pool_free(vg->vgmem, lvl); } for (s = 0; s < seg->area_count; s++) lv_set_hidden(seg_metalv(seg, s)); + + lv->status &= ~LV_ACTIVATION_SKIP; } return 1; @@ -7316,6 +7310,100 @@ out: return 1; } +/* + * Optionally makes on-disk metadata changes if @commit + * + * If LV is active: + * wipe any signatures and clear first sector of LVs listed on @lv_list + * otherwise: + * activate, wipe (as above), deactivate + * + * Returns: 1 on success, 0 on failure + */ +int activate_and_wipe_lvlist(struct dm_list *lv_list, int commit) +{ + struct lv_list *lvl; + struct volume_group *vg = NULL; + unsigned i = 0, sz = dm_list_size(lv_list); + char *was_active; + int r = 1; + + if (!sz) { + log_debug_metadata(INTERNAL_ERROR "Empty list of LVs given for wiping."); + return 1; + } + + dm_list_iterate_items(lvl, lv_list) { + if (!lv_is_visible(lvl->lv)) { + log_error(INTERNAL_ERROR + "LVs must be set visible before wiping."); + return 0; + } + vg = lvl->lv->vg; + } + + if (test_mode()) + return 1; + + /* + * FIXME: only vg_[write|commit] if LVs are not already written + * as visible in the LVM metadata (which is never the case yet). + */ + if (commit && + (!vg || !vg_write(vg) || !vg_commit(vg))) + return_0; + + was_active = alloca(sz); + + dm_list_iterate_items(lvl, lv_list) + if (!(was_active[i++] = lv_is_active(lvl->lv))) { + lvl->lv->status |= LV_TEMPORARY; + if (!activate_lv(vg->cmd, lvl->lv)) { + log_error("Failed to activate localy %s for wiping.", + display_lvname(lvl->lv)); + r = 0; + goto out; + } + lvl->lv->status &= ~LV_TEMPORARY; + } + + dm_list_iterate_items(lvl, lv_list) { + log_verbose("Wiping metadata area %s.", display_lvname(lvl->lv)); + /* Wipe any know signatures */ + if (!wipe_lv(lvl->lv, (struct wipe_params) { .do_wipe_signatures = 1, .do_zero = 1, .zero_sectors = 1 })) { + log_error("Failed to wipe %s.", display_lvname(lvl->lv)); + r = 0; + goto out; + } + } +out: + /* TODO: deactivation is only needed with clustered locking + * in normal case we should keep device active + */ + sz = 0; + dm_list_iterate_items(lvl, lv_list) + if ((i > sz) && !was_active[sz++] && + !deactivate_lv(vg->cmd, lvl->lv)) { + log_error("Failed to deactivate %s.", display_lvname(lvl->lv)); + r = 0; /* Continue deactivating as many as possible. */ + } + + return r; +} + +/* Wipe logical volume @lv, optionally with @commit of metadata */ +int activate_and_wipe_lv(struct logical_volume *lv, int commit) +{ + struct dm_list lv_list; + struct lv_list lvl; + + lvl.lv = lv; + dm_list_init(&lv_list); + dm_list_add(&lv_list, &lvl.list); + + return activate_and_wipe_lvlist(&lv_list, commit); +} + static struct logical_volume *_create_virtual_origin(struct cmd_context *cmd, struct volume_group *vg, const char *lv_name, diff --git a/lib/metadata/merge.c b/lib/metadata/merge.c index bdd9c67..d3d1d95 100644 --- a/lib/metadata/merge.c +++ b/lib/metadata/merge.c @@ -234,6 +234,30 @@ static void _check_non_raid_seg_members(struct lv_segment *seg, int *error_count /* .... more members? */ } +static void _check_raid_sublvs(struct lv_segment *seg, int *error_count) +{ + unsigned s; + + for (s = 0; s < seg->area_count; s++) { + if (seg_type(seg, s) != AREA_LV) + raid_seg_error("no raid image SubLV"); + + if ((seg_lv(seg, s)->status & LVM_WRITE) && + !(seg->lv->status & LV_ACTIVATION_SKIP) && + lv_is_visible(seg_lv(seg, s))) + raid_seg_error("visible raid image LV"); + + if (!seg_is_raid_with_meta(seg) || !seg->meta_areas) + continue; + + if (seg_metatype(seg, s) != AREA_LV) + raid_seg_error("no raid meta SubLV"); + else if (!(seg->lv->status & LV_ACTIVATION_SKIP) && + lv_is_visible(seg_metalv(seg, s))) + raid_seg_error("visible raid meta LV"); + } +} + /* * Check RAID segment struct members of @seg for acceptable * properties and increment @error_count for any bogus ones. @@ -287,10 +311,14 @@ static void _check_raid_seg(struct lv_segment *seg, int *error_count) /* Check for any MetaLV flaws like non-existing ones or size variations */ if (seg->meta_areas) for (area_len = s = 0; s < seg->area_count; s++) { + if (seg_metatype(seg, s) == AREA_UNASSIGNED) + continue; + if (seg_metatype(seg, s) != AREA_LV) { raid_seg_error("no MetaLV"); continue; } + if (!lv_is_raid_metadata(seg_metalv(seg, s))) raid_seg_error("MetaLV without RAID metadata flag"); if (area_len && @@ -314,6 +342,8 @@ static void _check_raid_seg(struct lv_segment *seg, int *error_count) _check_raid45610_seg(seg, error_count); else raid_seg_error("bogus RAID segment type"); + + _check_raid_sublvs(seg, error_count); } /* END: RAID segment property checks. */ diff --git a/lib/metadata/metadata-exported.h b/lib/metadata/metadata-exported.h index 763b91b..f867231 100644 --- a/lib/metadata/metadata-exported.h +++ b/lib/metadata/metadata-exported.h @@ -800,6 +800,12 @@ struct wipe_params { /* Zero out LV and/or wipe signatures */ int wipe_lv(struct logical_volume *lv, struct wipe_params params); +/* Wipe any signatures and zero first sector on @lv */ +int activate_and_wipe_lv(struct logical_volume *lv, int commit); + +/* Wipe any signatures and zero first sector of LVs listed on @lv_list */ +int activate_and_wipe_lvlist(struct dm_list *lv_list, int commit); + int lv_change_tag(struct logical_volume *lv, const char *tag, int add_tag); /* Reduce the size of an LV by extents */ @@ -1221,6 +1227,8 @@ int lv_raid_change_region_size(struct logical_volume *lv, int lv_raid_in_sync(const struct logical_volume *lv); uint32_t lv_raid_data_copies(const struct segment_type *segtype, uint32_t area_count); int lv_raid_free_reshape_space(const struct logical_volume *lv); +int lv_raid_clear_lv(struct logical_volume *lv, int commit); +int lv_raid_has_visible_sublvs(const struct logical_volume *lv); /* -- metadata/raid_manip.c */ /* ++ metadata/cache_manip.c */ diff --git a/lib/metadata/raid_manip.c b/lib/metadata/raid_manip.c index 3944dc4..349d926 100644 --- a/lib/metadata/raid_manip.c +++ b/lib/metadata/raid_manip.c @@ -689,86 +689,33 @@ static int _lv_update_and_reload_list(struct logical_volume *lv, int origin_only return r; } -/* Makes on-disk metadata changes - * If LV is active: - * clear first block of device - * otherwise: - * activate, clear, deactivate - * - * Returns: 1 on success, 0 on failure - */ +/* Wipe all LVs listsed on @lv_list committing lvm metadata */ static int _clear_lvs(struct dm_list *lv_list) { - struct lv_list *lvl; - struct volume_group *vg = NULL; - unsigned i = 0, sz = dm_list_size(lv_list); - char *was_active; - int r = 1; - - if (!sz) { - log_debug_metadata(INTERNAL_ERROR "Empty list of LVs given for clearing."); - return 1; - } - - dm_list_iterate_items(lvl, lv_list) { - if (!lv_is_visible(lvl->lv)) { - log_error(INTERNAL_ERROR - "LVs must be set visible before clearing."); - return 0; - } - vg = lvl->lv->vg; - } - - if (test_mode()) - return 1; + return activate_and_wipe_lvlist(lv_list, 1); +} - /* - * FIXME: only vg_[write|commit] if LVs are not already written - * as visible in the LVM metadata (which is never the case yet). - */ - if (!vg || !vg_write(vg) || !vg_commit(vg)) - return_0; +/* External interface to clear logical volumes on @lv_list */ +int lv_raid_has_visible_sublvs(const struct logical_volume *lv) +{ + unsigned s; + struct lv_segment *seg = first_seg(lv); - was_active = alloca(sz); + if (!lv_is_raid(lv) || (lv->status & LV_TEMPORARY) || !seg) + return 0; - dm_list_iterate_items(lvl, lv_list) - if (!(was_active[i++] = lv_is_active(lvl->lv))) { - lvl->lv->status |= LV_TEMPORARY; - if (!activate_lv(vg->cmd, lvl->lv)) { - log_error("Failed to activate localy %s for clearing.", - display_lvname(lvl->lv)); - r = 0; - goto out; - } - lvl->lv->status &= ~LV_TEMPORARY; - } + if (lv_is_raid_image(lv) || lv_is_raid_metadata(lv)) + return 0; - dm_list_iterate_items(lvl, lv_list) { - log_verbose("Clearing metadata area %s.", display_lvname(lvl->lv)); - /* - * Rather than wiping lv->size, we can simply - * wipe the first sector to remove the superblock of any previous - * RAID devices. It is much quicker. - */ - if (!wipe_lv(lvl->lv, (struct wipe_params) { .do_zero = 1, .zero_sectors = 1 })) { - log_error("Failed to zero %s.", display_lvname(lvl->lv)); - r = 0; - goto out; - } + for (s = 0; s < seg->area_count; s++) { + if ((seg_lv(seg, s)->status & LVM_WRITE) && /* Split off track changes raid1 leg */ + lv_is_visible(seg_lv(seg, s))) + return 1; + if (seg->meta_areas && lv_is_visible(seg_metalv(seg, s))) + return 1; } -out: - /* TODO: deactivation is only needed with clustered locking - * in normal case we should keep device active - */ - sz = 0; - dm_list_iterate_items(lvl, lv_list) - if ((i > sz) && !was_active[sz++] && - !deactivate_lv(vg->cmd, lvl->lv)) { - log_error("Failed to deactivate %s.", display_lvname(lvl->lv)); - r = 0; /* continue deactivating */ - } - return r; + return 0; } /* raid0* <-> raid10_near area reorder helper: swap 2 LV segment areas @a1 and @a2 */ diff --git a/tools/lvchange.c b/tools/lvchange.c index 537e582..975e24f 100644 --- a/tools/lvchange.c +++ b/tools/lvchange.c @@ -311,7 +311,6 @@ static int _lvchange_resync(struct cmd_context *cmd, struct logical_volume *lv) int monitored; struct lv_segment *seg = first_seg(lv); struct dm_list device_list; - struct lv_list *lvl; dm_list_init(&device_list); @@ -389,6 +388,7 @@ static int _lvchange_resync(struct cmd_context *cmd, struct logical_volume *lv) * Now we handle mirrors with log devices */ lv->status &= ~LV_NOTSYNCED; + lv->status |= LV_ACTIVATION_SKIP; /* Separate mirror log or metadata devices so we can clear them */ if (!_detach_metadata_devices(seg, &device_list)) { @@ -407,32 +407,8 @@ static int _lvchange_resync(struct cmd_context *cmd, struct logical_volume *lv) /* No backup for intermediate metadata, so just unlock memory */ memlock_unlock(lv->vg->cmd); - dm_list_iterate_items(lvl, &device_list) { - if (!activate_lv(cmd, lvl->lv)) { - log_error("Unable to activate %s for %s clearing.", - display_lvname(lvl->lv), (seg_is_raid(seg)) ? - "metadata area" : "mirror log"); - return 0; - } - - if (!wipe_lv(lvl->lv, (struct wipe_params) - { .do_zero = 1, .zero_sectors = lvl->lv->size })) { - log_error("Unable to reset sync status for %s.", - display_lvname(lv)); - if (!deactivate_lv(cmd, lvl->lv)) - log_error("Failed to deactivate log LV after " - "wiping failed"); - return 0; - } - - if (!deactivate_lv(cmd, lvl->lv)) { - log_error("Unable to deactivate %s LV %s " - "after wiping for resync.", - (seg_is_raid(seg)) ? "metadata" : "log", - display_lvname(lvl->lv)); - return 0; - } - } + if (!activate_and_wipe_lvlist(&device_list, 0)) + return 0; /* Wait until devices are away */ if (!sync_local_dev_names(lv->vg->cmd)) { @@ -448,6 +424,8 @@ static int _lvchange_resync(struct cmd_context *cmd, struct logical_volume *lv) return 0; } + lv->status &= ~LV_ACTIVATION_SKIP; + if (!_vg_write_commit(lv, NULL)) return 0; diff --git a/tools/lvconvert.c b/tools/lvconvert.c index dbc9fea..8329bc6 100644 --- a/tools/lvconvert.c +++ b/tools/lvconvert.c @@ -1992,19 +1992,9 @@ static int _lvconvert_snapshot(struct cmd_context *cmd, if (!zero || !(lv->status & LVM_WRITE)) log_warn("WARNING: %s not zeroed.", snap_name); - else { - lv->status |= LV_TEMPORARY; - if (!activate_lv(cmd, lv) || - !wipe_lv(lv, (struct wipe_params) { .do_zero = 1 })) { - log_error("Aborting. Failed to wipe snapshot exception store."); - return 0; - } - lv->status &= ~LV_TEMPORARY; - /* Deactivates cleared metadata LV */ - if (!deactivate_lv(lv->vg->cmd, lv)) { - log_error("Failed to deactivate zeroed snapshot exception store."); - return 0; - } + else if (!activate_and_wipe_lv(lv, 0)) { + log_error("Aborting. Failed to wipe snapshot exception store."); + return 0; } if (!archive(lv->vg)) @@ -3160,12 +3150,12 @@ static int _lvconvert_to_pool(struct cmd_context *cmd, goto_bad; if (zero_metadata) { - metadata_lv->status |= LV_TEMPORARY; + metadata_lv->status |= LV_ACTIVATION_SKIP; if (!activate_lv(cmd, metadata_lv)) { log_error("Aborting. Failed to activate metadata lv."); goto bad; } - metadata_lv->status &= ~LV_TEMPORARY; + metadata_lv->status &= ~LV_ACTIVATION_SKIP; if (!wipe_lv(metadata_lv, (struct wipe_params) { .do_zero = 1 })) { log_error("Aborting. Failed to wipe metadata lv.");

1 0

master - WHATS_NEW
by Heinz Mauelshagen 10 Dec '18

10 Dec '18

Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=b2d1facd960e192b38ae4… Commit: b2d1facd960e192b38ae4a9866f3bea9646e6cd5 Parent: edb72cb70cb8abd0acb92f07937715be2816d04a Author: Heinz Mauelshagen <heinzm(a)redhat.com> AuthorDate: Mon Dec 10 14:48:13 2018 +0100 Committer: Heinz Mauelshagen <heinzm(a)redhat.com> CommitterDate: Mon Dec 10 14:49:33 2018 +0100 WHATS_NEW --- WHATS_NEW | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index 68cdcea..ffc0d6a 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,5 +1,6 @@ Version 2.03.02 - =================================== + Prohibit mirrored 'mirror' log via lvcreate and lvconvert Use sync io if async io_setup fails, or use_aio=0 is set in config. Version 2.03.01 - 31st October 2018

1 0

master - lvcreate/lvconvert: prohibit creation of/conversion to mirrored mirror logs
by Heinz Mauelshagen 08 Dec '18

08 Dec '18

Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=edb72cb70cb8abd0acb92… Commit: edb72cb70cb8abd0acb92f07937715be2816d04a Parent: a4b8377488f82c496b14ee6d644fc681968acaca Author: Heinz Mauelshagen <heinzm(a)redhat.com> AuthorDate: Fri Dec 7 19:56:49 2018 +0100 Committer: Heinz Mauelshagen <heinzm(a)redhat.com> CommitterDate: Sat Dec 8 02:52:50 2018 +0100 lvcreate/lvconvert: prohibit creation of/conversion to mirrored mirror logs In RHEL7 we marked mirrored mirror logs as deprecated and added a related message. This patch prohibits creating new 'mirror' LVs with that log type or converting existing LVs to have one. Existing LVs with mirrored mirror log can be activated and converted to disk/core logs. Avoid double deprecation message when running lvconvert. Resolves: rhbz1643562 --- lib/metadata/mirror.c | 3 ++- tools/lvconvert.c | 5 ----- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/lib/metadata/mirror.c b/lib/metadata/mirror.c index ba3f33d..8b20979 100644 --- a/lib/metadata/mirror.c +++ b/lib/metadata/mirror.c @@ -1908,7 +1908,8 @@ int add_mirror_log(struct cmd_context *cmd, struct logical_volume *lv, } if (log_count > 1) { - log_warn("WARNING: Log type \"mirrored\" is DEPRECATED and will be removed in the future. Use RAID1 LV or disk log instead."); + log_err("Log type \"mirrored\" is DEPRECATED. Use RAID1 LV or disk log instead."); + return 0; } if (!(parallel_areas = build_parallel_areas_from_lv(lv, 0, 0))) diff --git a/tools/lvconvert.c b/tools/lvconvert.c index 65a4ed3..dbc9fea 100644 --- a/tools/lvconvert.c +++ b/tools/lvconvert.c @@ -1262,11 +1262,6 @@ static int _lvconvert_mirrors(struct cmd_context *cmd, (old_log_count == new_log_count)) return 1; - if ((old_log_count != new_log_count) && - (new_log_count == MIRROR_LOG_MIRRORED)) { - log_warn("WARNING: Log type \"mirrored\" is DEPRECATED and will be removed in the future. Use RAID1 LV or disk log instead."); - } - if (!_lvconvert_mirrors_aux(cmd, lv, lp, NULL, new_mimage_count, new_log_count, lp->pvh)) return_0;

1 0

master - lvmlockd: fix missing LV lock for lvconvert repair
by David Teigland 07 Dec '18

07 Dec '18

Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=a4b8377488f82c496b14e… Commit: a4b8377488f82c496b14ee6d644fc681968acaca Parent: 21784e94d6f4308931fa497679f1a4e5e1966578 Author: David Teigland <teigland(a)redhat.com> AuthorDate: Fri Dec 7 13:11:31 2018 -0600 Committer: David Teigland <teigland(a)redhat.com> CommitterDate: Fri Dec 7 13:11:31 2018 -0600 lvmlockd: fix missing LV lock for lvconvert repair Add missing lvmlockd LV lock for lvconvert repair on mirror and thin/cache pools. --- tools/lvconvert.c | 16 ++++++++++++++++ 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/tools/lvconvert.c b/tools/lvconvert.c index a29a66b..65a4ed3 100644 --- a/tools/lvconvert.c +++ b/tools/lvconvert.c @@ -3534,6 +3534,18 @@ static int _lvconvert_repair_pvs_mirror(struct cmd_context *cmd, struct logical_ int ret; /* + * We want to allow cmirror active on multiple nodes to be repaired, + * but normal mirror to only be repaired if active exclusively here. + * If the LV is active it already has the necessary lock, but if not + * active, then require ex since we cannot know the active state on + * other hosts. + */ + if (!lv_is_active(lv)) { + if (!lockd_lv(cmd, lv, "ex", 0)) + return_0; + } + + /* * FIXME: temporary use of lp because _lvconvert_mirrors_repair() * and _aux() still use lp fields everywhere. * Migrate them away from using lp (for the most part just use @@ -3663,6 +3675,10 @@ static int _lvconvert_repair_cachepool_thinpool(struct cmd_context *cmd, struct int poolmetadataspare = arg_int_value(cmd, poolmetadataspare_ARG, DEFAULT_POOL_METADATA_SPARE); struct dm_list *use_pvh; + /* ensure it's not active elsewhere. */ + if (!lockd_lv(cmd, lv, "ex", 0)) + return_0; + if (cmd->position_argc > 1) { /* First pos arg is required LV, remaining are optional PVs. */ if (!(use_pvh = create_pv_list(cmd->mem, lv->vg, cmd->position_argc - 1, cmd->position_argv + 1, 0)))

1 0

v2_02_183 annotated tag has been created
by Marian Csontos 07 Dec '18

07 Dec '18

Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=f21d9eb8527258479805f… Commit: f21d9eb8527258479805f5a8132dd9e6e3e572d8 Parent: 0000000000000000000000000000000000000000 Author: Marian Csontos <mcsontos(a)redhat.com> AuthorDate: 2018-12-07 14:13 +0000 Committer: Marian Csontos <mcsontos(a)redhat.com> CommitterDate: 2018-12-07 14:13 +0000 annotated tag: v2_02_183 has been created at f21d9eb8527258479805f5a8132dd9e6e3e572d8 (tag) tagging f5ea02ffee3269a77e295d2b186ca81481f592d7 (commit) replaces v2_02_182 Release 2.02.183 Bug fix release. Use sync IO when AIO fails due to system reaching the aio request limit. Add global/use_aio configuration option to disable aio. MD RAID verion 0.90 devices are filtered out preventing them to be seen as duplicate PVs. Fix regression in dmstats resulting in no output. -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAABAgAGBQJcCoAQAAoJELkRJDHlCQOf6FUP+gNHcULrPp8Tv47Hk6LOOY6x G6YEkwEmks070B3FPPLGJ49YuGbuiiGhbBdBKajaOiubTStuEc2cFO597kCV8SlX TwNF9NEGCUy3XyhqAXStZb3nOEZo0pO7ZZsnwlkfEOWb/R+2fcGA+c6SwLUoOwbx NZYBRCCdaTcVdJhD0qNjz1SrZ5GREQjXDwfSN+uqAw+QwWWWsqZd+hrlM86mEIer cls2EfhNkEV7UBqi6CS7d9uzQNpLuul//jOmggH1q39rVw7ZxCLv6wPQOVR9Z+xm T7Do3w2kmy0wsmcdxRcOJ8POeh7nSJApU3owa2BjBEetRwyxfDhdNcsjG52XHCIx l9PyG6VNvg92FeDVkkOxGLQP9bd76+u4JiRzyzMjYczuCEbbV4Yhwud3MoHA1/fU E0kD7FWImPIRudS5GR0XRTfZB/hT4LaL0k2RDtMYr2IIfEsixgWEPwMD0P9RrYnS vTKnAm62oO2TLvkIFq+D5vguEjg9JLz5xV3NV3yZGCe6YiShaz0C0LScmAOCIpq7 YQ2xPBGBKKuYDsA1TEgUXzaPX6qtZQhQ2CC25OR4PL/z2d7eotVlcjNSChZgKOLt g8j7CKKqgp8R5F5SKPHbGHVtNqkWUYmUQbRSgtFwl0wCTIoFILt0NnbDO9XEVjD3 f7uAulpVklECGeS76zKr =4F0E -----END PGP SIGNATURE----- Bryn M. Reeves (2): libdm-stats: move no regions warning after dm_stats_list() dmsetup: fix stats report command output David Teigland (9): devices: reuse bcache fd when getting block size update WHATS_NEW io: use sync io if aio fails bcache: sync io fixes WHATS_NEW: sync io pvscan lvmetad: use full md filter when md 1.0 devices are present lvmetad: only disable if repair will do something lvmetad: fix disabling in previous commit pvscan lvmetad: use udev info to improve md component detection Marian Csontos (3): post-release build: make generate pre-release Peter Rajnoha (1): scan: md metadata version 0.90 is at the end of disk Zdenek Kabelac (1): libdm: do not add params for resume and remove

1 0

master - udev: 69-dm-lvm-metad.rules is still needed
by Marian Csontos 06 Dec '18

06 Dec '18

Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=21784e94d6f4308931fa4… Commit: 21784e94d6f4308931fa497679f1a4e5e1966578 Parent: 92b0d014aab12eee3fe1f757a5fb9ecfe9f4b58c Author: Marian Csontos <mcsontos(a)redhat.com> AuthorDate: Thu Dec 6 10:50:42 2018 +0100 Committer: Marian Csontos <mcsontos(a)redhat.com> CommitterDate: Thu Dec 6 10:50:42 2018 +0100 udev: 69-dm-lvm-metad.rules is still needed --- spec/packages.inc | 1 + udev/Makefile.in | 5 +---- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/spec/packages.inc b/spec/packages.inc index 0be4f2e..81ef539 100644 --- a/spec/packages.inc +++ b/spec/packages.inc @@ -150,6 +150,7 @@ fi %endif %if %{enable_udev} %{_udevdir}/11-dm-lvm.rules + %{_udevdir}/69-dm-lvm-metad.rules %endif %dir %{_sysconfdir}/lvm %ghost %{_sysconfdir}/lvm/cache/.cache diff --git a/udev/Makefile.in b/udev/Makefile.in index 81c2c7b..e32cba9 100644 --- a/udev/Makefile.in +++ b/udev/Makefile.in @@ -18,10 +18,7 @@ top_builddir = @top_builddir@ include $(top_builddir)/make.tmpl DM_RULES=10-dm.rules 13-dm-disk.rules 95-dm-notify.rules -LVM_RULES=11-dm-lvm.rules -ifeq ("@BUILD_LVMETAD@", "yes") -LVM_RULES+=69-dm-lvm-metad.rules -endif +LVM_RULES=11-dm-lvm.rules 69-dm-lvm-metad.rules DM_DIR=$(shell $(GREP) "\#define DM_DIR" $(top_srcdir)/libdm/misc/dm-ioctl.h | $(AWK) '{print $$3}')

1 0

master - build: Upse PYTHON_CONFIG env.variable when set
by Marian Csontos 06 Dec '18

06 Dec '18

Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=92b0d014aab12eee3fe1f… Commit: 92b0d014aab12eee3fe1f757a5fb9ecfe9f4b58c Parent: 73687b7b75c4f12b6f21ba829898b4d13422db1c Author: Marian Csontos <mcsontos(a)redhat.com> AuthorDate: Fri Nov 2 10:06:51 2018 +0100 Committer: Marian Csontos <mcsontos(a)redhat.com> CommitterDate: Thu Dec 6 09:02:47 2018 +0100 build: Upse PYTHON_CONFIG env.variable when set This adds up to Commit 6462e8dffc7ed53973ce796cec18fecc0fb89d77. --- configure | 1 - configure.ac | 1 - 2 files changed, 0 insertions(+), 2 deletions(-) diff --git a/configure b/configure index efd438c..66acdb2 100755 --- a/configure +++ b/configure @@ -11664,7 +11664,6 @@ fi ################################################################################ if test "$BUILD_LVMDBUSD" = yes; then - unset PYTHON_CONFIG unset am_cv_pathless_PYTHON ac_cv_path_PYTHON am_cv_python_platform unset am_cv_python_pythondir am_cv_python_version am_cv_python_pyexecdir unset ac_cv_path_PYTHON_CONFIG ac_cv_path_ac_pt_PYTHON_CONFIG diff --git a/configure.ac b/configure.ac index 3578f7b..2b29210 100644 --- a/configure.ac +++ b/configure.ac @@ -1172,7 +1172,6 @@ AS_IF([test "$NOTIFYDBUS_SUPPORT" = yes && test "BUILD_LVMDBUSD" = yes], dnl -- Enable Python dbus library if test "$BUILD_LVMDBUSD" = yes; then - unset PYTHON_CONFIG unset am_cv_pathless_PYTHON ac_cv_path_PYTHON am_cv_python_platform unset am_cv_python_pythondir am_cv_python_version am_cv_python_pyexecdir unset ac_cv_path_PYTHON_CONFIG ac_cv_path_ac_pt_PYTHON_CONFIG

1 0

← Newer
1
2
3
4
5
6
7
8
9
10
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

lvm2-commits December 2018