I'm not convinced this patch catches everything yet. E.g. vg_remove_check has a: dm_list_del(&pvl->list); and I don't spot the matching pv->vg change. If it's there, it's well-hidden. Every place in the code where vg->pvs changes should have a corresponding pv->vg change next to it. Alasdair