[lvm-devel] [PATCH 3/4] Use allog_vg for _free_vg in _pvsegs_sub_single

Alasdair G Kergon agk at redhat.com
Wed Mar 9 03:01:51 UTC 2011


On Tue, Mar 08, 2011 at 11:30:16PM +0100, Zdenek Kabelac wrote:
> The major purpose if the patch is to keep also reference count setting in one
> place (needed in my vg sharing patch) - also whenever someone adds any new
> member to VG structure - only one place needs to be updated for proper
> initialization.  So I'd still prefer to have this code running through
> standard allocation procedure.
 
But the new function is not (yet?) initialising the whole structure.

The 'reporter' use is only as a read-only dummy structure to allow us
to share some code paths.  It should not need a vgmem (or reference
counting), I'd have thought.

Alasdair




More information about the lvm-devel mailing list