[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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



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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]