[lvm-devel] [Red Hat - Possible Forgery] Re: [PATCH] Pass parsed metadata to activation code.
Petr Rockai
prockai at redhat.com
Wed Mar 20 11:11:44 UTC 2013
Hi there,
Zdenek Kabelac <zkabelac at redhat.com> writes:
> I've went into very quick overview - would it be possible to split some
> patches into 'rename/clean old code' 'add new functionality' ?
I don't see any places where "cleaning old code" is being done. Can you
point out which bits you mean?
> Also - how does the new solution 'scales' in terms of having thousands of LVs
> in the metadata - I've noticed couple new linear scans 'dm_list_iterate()'. The
> trick with lv_ondisk() code is not yet very clean to me - will need to find
> some time for this - it seems to be adding new complexity in the code.
lv_ondisk is trivial, it just finds the same LV you are using in the
"ondisk" version of its metadata; the "ondisk" version of VG is just
that: the struct volume_group that came pristine from vg_read, or after
you vg_commit, the version that has been committed.
For iteration, yes, it does a linear scan, but not a "new" one. You
either call lv_ondisk OR you call lv_from_lvid. Both do a single linear
scan through the list of LVs. In fact, for suspend, the old code does 2
such scans while the new code only does one. So the net result is quite
the opposite, there's one iteration less and no new ones.
Petr
--
id' Ash = Ash; id' Dust = Dust; id' _ = undefined
More information about the lvm-devel
mailing list