diff options
author | NeilBrown <neilb@suse.de> | 2012-07-09 17:19:48 +1000 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2012-07-09 17:19:48 +1000 |
commit | 265460ababa169a2a0926b5fba10724ba117e318 (patch) | |
tree | 06718d59a40d6f6e318e53150f841894bd4e343c /mdadm.c | |
parent | 171dccc81381725935af15a0a1d28de35988655d (diff) |
Examine: split 'verbose' out from 'brief'.
The value of 'verbose' is sometimes mixed into 'brief', particularly
for Examine.
This is messy and confusing. So keep them separate.
'brief' still gets assumed when 'scan' is set, unless we are very
verbose.
Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'mdadm.c')
-rw-r--r-- | mdadm.c | 8 |
1 files changed, 5 insertions, 3 deletions
@@ -1221,6 +1221,10 @@ int main(int argc, char *argv[]) ident.autof = c.autof; + if (c.scan && c.verbose < 2) + /* --scan implied --brief unless -vv */ + c.brief = 1; + rv = 0; switch(mode) { case MANAGE: @@ -1350,9 +1354,7 @@ int main(int argc, char *argv[]) pr_err("No devices listed in %s\n", configfile?configfile:DefaultConfFile); exit(1); } - if (c.brief && c.verbose > 0) - c.brief = 2; - rv = Examine(devlist, c.scan?(c.verbose>1?0:c.verbose):c.brief, + rv = Examine(devlist, c.brief, c.verbose, c.export, c.scan, c.SparcAdjust, ss, c.homehost); } else if (devmode == DetailPlatform) { |