]> git.ipfire.org Git - thirdparty/dracut.git/commit
lvm: remove all quirk arguments
authorHarald Hoyer <harald@redhat.com>
Tue, 10 Nov 2015 16:55:50 +0000 (17:55 +0100)
committerHarald Hoyer <harald@redhat.com>
Wed, 11 Nov 2015 10:48:32 +0000 (11:48 +0100)
commit53a738465371db3cb7df533a3ae2b5648dfd13e4
tree8b937630984064a911a24350fdfbdfd40865e7d7
parentc97209fe69dbbacb1f87acf5ec8a0e558e553ddc
lvm: remove all quirk arguments

https://bugzilla.redhat.com/show_bug.cgi?id=742564

Once lvm2 starts using /run (bug 742554), it should be no longer
necessary to disable file-based locking in the vgchange call in
fedora-storage-init.
Removing '--sysinit' will make it safe to call LVM operations
concurrently from other units.

The --sysinit is a compound option consisting of:

  -> --ignorelockingfailure - not needed anymore, the /run/lock/lvm is
available rw soon in boot process

  -> --ignoremonitoring - not needed since /run is available, this would
require the dm-event.service to be run before
fedora-storage-init.service (and new lvm2-activation.service when
deployed). But that's a one line change - I'll have a look whether it
plays well with other services and if yes, I'll commit the change. N.B.:
This has a consequence that all volumes activated on vgchange -ay will
be monitored at the same time they're activated (which is a plus I
think). The lvm2-monitor will just grab all the other volumes not
activated at the time of the boot's vgchange -ay call. But that's not an
issue (for already monitored volumes, calling vgchange --monitor y will
just be a NOOP).

 -> --poll n - not needed, we can run the polldaemon as the /run is
available and rw (in case there's unfinished merge or mirror sync from
previous system run, the poll-daemon will be triggered at boot now).
Actually, the polldaemon should be triggered as a service like dmeventd
is, not forked off from the LVM command itself, like from vgchange in
this case - we still need to change this - there's a bug open for this
request already (bug #814857). However, we don't have this feature ready
yet so I need to check whether this is OK with the early boot process
with the current state.
modules.d/90lvm/lvm_scan.sh