diff options
author | Dave Wysochanski <dwysocha@redhat.com> | 2009-09-15 01:38:59 +0000 |
---|---|---|
committer | Dave Wysochanski <dwysocha@redhat.com> | 2009-09-15 01:38:59 +0000 |
commit | 266214db845d24f010d9c40e4cf5c4ac606d9d21 (patch) | |
tree | e9abdd9ba4f848ab72fabd9984c4f46f80886546 /tools/vgimport.c | |
parent | 90c8088760d5fd46ad9f5edb58482a491be53f26 (diff) | |
download | lvm2-266214db845d24f010d9c40e4cf5c4ac606d9d21.tar.gz |
Fix process_each_vg / _process_one_vg when vg_read() returns FAILED_LOCKING.
Remove the checks for vg_read_error() in most of the tools callback
functions and instead make the check in _process_one_vg() more general.
In all but vgcfgbackup, we do not want to proceed if we get any error
from vg_read(). In vgcfgbackup's case, we may proceed if the backup
is to proceed with inconsistent VGs. This is a special case though,
and we mark it with the READ_ALLOW_INCONSISTENT flag passed to
process_each_vg (and subsequently to _process_one_vg).
NOTE: More cleanup is needed in the vg_read_error() path cases.
This patch is a start.
Diffstat (limited to 'tools/vgimport.c')
-rw-r--r-- | tools/vgimport.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/tools/vgimport.c b/tools/vgimport.c index 7eeb5db3f..ee8e40a3d 100644 --- a/tools/vgimport.c +++ b/tools/vgimport.c @@ -23,9 +23,6 @@ static int vgimport_single(struct cmd_context *cmd __attribute((unused)), struct pv_list *pvl; struct physical_volume *pv; - if (vg_read_error(vg)) - goto_bad; - if (!vg_is_exported(vg)) { log_error("Volume group \"%s\" is not exported", vg_name); goto bad; |