]> sourceware.org Git - lvm2.git/commit
lvmetad: two phase vg_update
authorDavid Teigland <teigland@redhat.com>
Wed, 8 Jun 2016 19:42:03 +0000 (14:42 -0500)
committerAlasdair G Kergon <agk@redhat.com>
Tue, 28 Jun 2016 01:30:31 +0000 (02:30 +0100)
commita7c45ddc59449fa8b1823bcab31e3fbb64fed97c
tree7ff3a663fadb54a88922228ac1c0e3eb345d976f
parentcc3e7c7c31de8d69d9f1eb610181584482ee90a3
lvmetad: two phase vg_update

Previously, a command sent lvmetad new VG metadata in vg_commit().
In vg_commit(), devices are suspended, so any memory allocation
done by the command while sending to lvmetad, or by lvmetad while
updating its cache could deadlock if memory reclaim was triggered.

Now lvmetad is updated in unlock_vg(), after devices are resumed.
The new method for updating VG metadata in lvmetad is in two phases:

1. In vg_write(), before devices are suspended, the command sends
   lvmetad a short message ("set_vg_info") telling it what the new
   VG seqno will be.  lvmetad sees that the seqno is newer than
   the seqno of its cached VG, so it sets the INVALID flag for the
   cached VG.  If sending the message to lvmetad fails, the command
   fails before the metadata is committed and the change is not made.
   If sending the message succeeds, vg_commit() is called.

2. In unlock_vg(), after devices are resumed, the command sends
   lvmetad the standard vg_update message with the new metadata.
   lvmetad sees that the seqno in the new metadata matches the
   seqno it saved from set_vg_info, and knows it has the latest
   copy, so it clears the INVALID flag for the cached VG.

If a command fails between 1 and 2 (after committing the VG on disk,
but before sending lvmetad the new metadata), the cached VG retains
the INVALID flag in lvmetad.  A subsequent command will read the
cached VG from lvmetad, see the INVALID flag, ignore the cached
copy, read the VG from disk instead, update the lvmetad copy
with the latest copy from disk, (this clears the INVALID flag
in lvmetad), and use the correct VG metadata for the command.

(This INVALID mechanism already existed for use by lvmlockd.)
26 files changed:
daemons/lvmetad/lvmetad-core.c
lib/cache/lvmetad.c
lib/cache/lvmetad.h
lib/format_text/archiver.c
lib/locking/locking.h
lib/metadata/metadata.c
lib/metadata/pv_manip.c
lib/metadata/replicator_manip.c
lib/metadata/vg.c
lib/metadata/vg.h
liblvm/lvm_pv.c
liblvm/lvm_vg.c
tools/pvchange.c
tools/pvcreate.c
tools/pvdisplay.c
tools/pvremove.c
tools/pvscan.c
tools/reporter.c
tools/toollib.c
tools/vgcfgrestore.c
tools/vgcreate.c
tools/vgextend.c
tools/vgimportclone.c
tools/vgrename.c
tools/vgscan.c
tools/vgsplit.c
This page took 0.038941 seconds and 5 git commands to generate.