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

Re: [lvm-devel] duplicate pv names


David Teigland <teigland redhat com> writes:
> My current patch set it having a problem with the test vgimportclone.sh,
> which has me questioning the desired pvs output with duplicate pv names.

I ran into similar problems with a different set of patches, that remove
pv_read. What I'll probably need to do is adding a list of devices to
PVs, so we can check whether something on the command line matches any
of the "duplicate" PVs.

I see you make quite heavy use of pv_read in your new version of
process_each_pv. You can have a look at:

(git clone) http://repos.mornfall.net/lvm2/mornfall.git

branch "merged" or for more limited view of the pv_read removal patches,
branch "lvmcache". (git checkout origin/merged).

It would be good if we could get rid of pv_read, as the code for
handling nomda PVs is wrong anyway, and we always need to scan. I think
the only remaining issue with the non-pv_read version is the duplicate
handling as outlined above.


id' Ash = Ash; id' Dust = Dust; id' _ = undefined

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