1
0
mirror of git://sourceware.org/git/lvm2.git synced 2025-02-24 17:57:48 +03:00
David Teigland 6b3e402298 toollib: add comment about missing device
Add a comment in _process_pvs_in_vg() to document the
place where there have been problems with processing
PVs twice.

For a while we had a hacky workaround here where we'd
skip processing a PV if its device wasn't found in
all_devices (and !is_missing_pv since we want to
process PVs with missing devices.).  That workaround
was removed in commit 5cd4d46f because it was no
longer needed.

The workaround had originally been needed to prevent
a device from being processed twice when the PV had
no MDAs -- it would be processed once in its real VG
and then the workaround would prevent it from being
processed a second time in the orphan VG.

Wrongly appearing as an orphan likely happened because
lvmcache would consider the no-MDA PV an orphan unless
the real VG holding that PV was also in lvmcache.
This issue is also mentioned in pvchange where holding
the global lock allows VGs to remain in lvmcache so
PVs with 0 mdas are not considered orphans.

The workaround in _process_pvs_in_vg() was originally
intended for reporting commands, not for pvchange.
But, it was accidentally helping pvchange also because
the method described by the pvchange global lock
comment had been subverted by commit 80f4b4b8.
Commit 80f4b4b8 was found to be unnecessary, and was
reverted in commit e710bac0.  This restored the
intended global lock lvmcache effect to pvchange, and
it no longer relied on the workaround in toollib.
2016-01-18 16:09:22 -06:00
..
2015-12-14 14:26:47 -06:00
2015-11-09 17:04:10 +01:00
2015-12-01 09:36:52 -06:00
2008-01-30 14:00:02 +00:00
2015-07-06 17:30:18 +01:00
2008-01-30 14:00:02 +00:00
2015-12-09 11:59:49 -06:00
2015-12-11 20:15:51 +01:00
2015-07-02 15:42:26 -05:00
2016-01-18 14:48:30 -06:00
2015-07-02 15:42:26 -05:00
2015-12-14 14:26:47 -06:00
2014-01-30 13:09:15 +00:00
2015-07-06 17:30:18 +01:00
2016-01-18 14:48:30 -06:00
2016-01-13 16:40:02 -06:00
2015-12-03 18:17:45 +01:00
2015-07-02 15:42:26 -05:00
2015-12-14 14:26:47 -06:00
2015-07-02 15:42:26 -05:00