shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2024-12-21 13:34:40 +03:00
Code Releases Activity

metadata: Fix handling of orphan PV linking & re-linking.

Browse Source
This commit is contained in:
Petr Rockai 2013-02-19 03:13:59 +01:00
parent 651d5093ed
commit ba6d6f0028
1 changed files with 28 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
lib/metadata/metadata.c
View File

@ -269,6 +269,12 @@ int add_pv_to_vg(struct volume_group *vg, const char *pv_name,
vg->extent_count += pv->pe_count;
vg->free_count += pv->pe_count;
dm_list_iterate_items(pvl, &fid->fmt->orphan_vg->pvs)
if (pv == pvl->pv) { /* unlink from orphan */
dm_list_del(&pvl->list);
break;
}
if (pv->status & UNLABELLED_PV) {
if (!(pvc = dm_pool_zalloc(mem, sizeof(*pvc)))) {
log_error("pv_to_create allocation for '%s' failed", pv_name);
@ -2810,8 +2816,10 @@ static struct volume_group *_vg_read_orphans(struct cmd_context *cmd,
struct lvmcache_vginfo *vginfo;
struct volume_group *vg = NULL;
struct _vg_read_orphan_baton baton;
struct pv_list *pvl;
struct pv_list *pvl, *pvl_;
struct pv_list head;
dm_list_init(&head.list);
lvmcache_label_scan(cmd, 0);
lvmcache_seed_infos_from_lvmetad(cmd);
@ -2822,14 +2830,31 @@ static struct volume_group *_vg_read_orphans(struct cmd_context *cmd,
return_NULL;
vg = fmt->orphan_vg;
dm_list_iterate_items(pvl, &vg->pvs)
pv_set_fid(pvl->pv, NULL);
restart:
dm_list_iterate_items(pvl, &vg->pvs) {
if (pvl->pv->status & UNLABELLED_PV ) {
dm_list_del(&pvl->list);
dm_list_add(&head.list, &pvl->list);
goto restart;
} else
pv_set_fid(pvl->pv, NULL);
}
dm_list_init(&vg->pvs);
vg->pv_count = 0;
vg->extent_count = 0;
vg->free_count = 0;
baton.warnings = warnings;
baton.vg = vg;
while (!dm_list_empty(&head.list)) {
pvl = (struct pv_list *) dm_list_first(&head.list);
dm_list_del(&pvl->list);
add_pvl_to_vgs(vg, pvl);
vg->extent_count += pvl->pv->pe_count;
vg->free_count += pvl->pv->pe_count;
}
if (!lvmcache_foreach_pv(vginfo, _vg_read_orphan_pv, &baton))
return_NULL;