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/vgremove.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/vgremove.c')
-rw-r--r-- | tools/vgremove.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/tools/vgremove.c b/tools/vgremove.c index c5b5dcfe9..bc29d9992 100644 --- a/tools/vgremove.c +++ b/tools/vgremove.c @@ -22,11 +22,6 @@ static int vgremove_single(struct cmd_context *cmd, const char *vg_name, unsigned lv_count; force_t force; - if (vg_read_error(vg)) { - stack; - return ECMD_FAILED; - } - if (!vg_check_status(vg, EXPORTED_VG)) { stack; return ECMD_FAILED; |