diff options
author | Peter Rajnoha <prajnoha@redhat.com> | 2016-04-08 14:18:00 +0200 |
---|---|---|
committer | Peter Rajnoha <prajnoha@redhat.com> | 2016-04-08 14:21:47 +0200 |
commit | ed0c779bd120e7c36cb7f100df3c6edc98e8cc52 (patch) | |
tree | 9c82bdc570197de97d0464f00f20649734ed673a /tools | |
parent | 50866034a2e612250c45116674765da08820fb27 (diff) | |
download | lvm2-ed0c779bd120e7c36cb7f100df3c6edc98e8cc52.tar.gz |
report: remove superfluous SEGSSTATUS report type
We don't have any report field of this type yet. Return this patch into
the play if we really need that. Currenly we always report status
(result of "status" dm ioctl) for an LV as a whole where we choose
segment which represents the LV, not calling status for each possible
segment it contains - we don't need this now so I'm removing it to
not make the code more complex uselessly.
Diffstat (limited to 'tools')
-rw-r--r-- | tools/reporter.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/tools/reporter.c b/tools/reporter.c index 220e307ce..a4cae6032 100644 --- a/tools/reporter.c +++ b/tools/reporter.c @@ -409,10 +409,10 @@ static int _get_final_report_type(int args_are_pvs, *lv_info_needed = (report_type & (LVSINFO | LVSINFOSTATUS)) ? 1 : 0; /* Do we need to acquire LV device status in addition? */ - *lv_segment_status_needed = (report_type & (SEGSSTATUS | LVSSTATUS | LVSINFOSTATUS)) ? 1 : 0; + *lv_segment_status_needed = (report_type & (LVSSTATUS | LVSINFOSTATUS)) ? 1 : 0; /* Ensure options selected are compatible */ - if (report_type & (SEGS | SEGSSTATUS)) + if (report_type & SEGS) report_type |= LVS; if (report_type & PVSEGS) report_type |= PVS; @@ -429,7 +429,7 @@ static int _get_final_report_type(int args_are_pvs, else if ((report_type & PVS) || ((report_type & LABEL) && (report_type & VGS))) report_type = PVS; - else if (report_type & (SEGS | SEGSSTATUS)) + else if (report_type & SEGS) report_type = SEGS; else if (report_type & (LVS | LVSINFO | LVSSTATUS | LVSINFOSTATUS)) report_type = LVS; @@ -907,8 +907,6 @@ static int _report(struct cmd_context *cmd, int argc, char **argv, r = process_each_vg(cmd, argc, argv, NULL, 0, handle, &_pvs_in_vg); break; - case SEGSSTATUS: - /* fall through */ case SEGS: r = process_each_lv(cmd, argc, argv, 0, handle, lv_info_needed && !lv_segment_status_needed ? &_lvsegs_with_info_single : |