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/vgdisplay.c | |
parent | 90c8088760d5fd46ad9f5edb58482a491be53f26 (diff) | |
download | lvm2-266214db845d24f010d9c40e4cf5c4ac606d9d21.tar.gz lvm2-266214db845d24f010d9c40e4cf5c4ac606d9d21.tar.xz lvm2-266214db845d24f010d9c40e4cf5c4ac606d9d21.zip |
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/vgdisplay.c')
-rw-r--r-- | tools/vgdisplay.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/tools/vgdisplay.c b/tools/vgdisplay.c index 3876d351..06965bf5 100644 --- a/tools/vgdisplay.c +++ b/tools/vgdisplay.c @@ -20,11 +20,6 @@ static int vgdisplay_single(struct cmd_context *cmd, const char *vg_name, void *handle __attribute((unused))) { /* FIXME Do the active check here if activevolumegroups_ARG ? */ - if (vg_read_error(vg)) { - stack; - return ECMD_FAILED; - } - vg_check_status(vg, EXPORTED_VG); if (arg_count(cmd, colon_ARG)) { |