[lvm-devel] [PATCH 08/22] Replicator: extend volume_group with list of VGs
Zdenek Kabelac
zkabelac at redhat.com
Mon Apr 26 14:39:19 UTC 2010
Dne 26.4.2010 16:26, Alasdair G Kergon napsal(a):
> On Mon, Apr 12, 2010 at 05:21:36PM +0200, Zdenek Kabelac wrote:
>> Add linked list of opened VGs. List keeps the information
>> about needed and locked and opened VGs for replicator target.
>> Also add missing_vgs status information for quick check and
>> also for possible continuos process_each_lv() where we need
>> to detect whether failure has been caused by missing VG or
>> some other reason.
>
>> diff --git a/lib/metadata/metadata-exported.h b/lib/metadata/metadata-exported.h
>> index 6d40365..9c73dba 100644
>> --- a/lib/metadata/metadata-exported.h
>> +++ b/lib/metadata/metadata-exported.h
>> @@ -250,6 +250,10 @@ struct volume_group {
>> + /* List of wanted/locked and opened VGs */
>> + struct dm_list vgs;
>> + uint32_t missing_vgs;
>
> Can we think up more-specific names?
>
> vg->vgs isn't very informative.
>
In fact I've had there some different names - like remote_vgs, needed_vgs,
deps_vgs....
but I could not decide - and as we now already have pvs, lvs, vgs seems to
be natural extension of naming concept.
vgs - is list of vgnames and flags for openning, including the vg itself -
thus remote_vgs doesn't fit. Needed - seems plain ugly.
So I'm open for new names here - note there are some manipulation functions
which should also use this name.
Zdenek
More information about the lvm-devel
mailing list