[lvm-devel] [PATCH 4/4] Move activation of background polling processes to a common function.

Alasdair G Kergon agk at redhat.com
Tue Sep 29 20:44:20 UTC 2009


On Tue, Sep 29, 2009 at 11:22:57AM -0400, Mike Snitzer wrote:
> Move activation of background polling processes to a common function.
> Avoids code duplication and fixes an associated where vgchange would
> only restart pvmove and _not_ lvconvert (lvchange would restart
> both).
 
Applied, but I've retained the more-efficient version for pvmove for
vgchange: when you know you are looping through all the LVs in the VG
you can skip the normal (LOCKED) LVs which are being moved and trigger
from the PVMOVE ones that perform the pvmove.  As lvchange might only
be processing a single LV, you can't do that.

Alasdair




More information about the lvm-devel mailing list