diff options
author | Milan Broz <mbroz@redhat.com> | 2009-06-06 22:00:20 +0000 |
---|---|---|
committer | Milan Broz <mbroz@redhat.com> | 2009-06-06 22:00:20 +0000 |
commit | 12b1f31cbbaf55544efae206ee997ea1fd2692f6 (patch) | |
tree | 85e6830717d549d9244d5425c6d480033ada5d42 /tools/lvchange.c | |
parent | 2d37634918af238923a62fcd2dd42abad9e41c6f (diff) | |
download | lvm2-12b1f31cbbaf55544efae206ee997ea1fd2692f6.tar.gz lvm2-12b1f31cbbaf55544efae206ee997ea1fd2692f6.tar.xz lvm2-12b1f31cbbaf55544efae206ee997ea1fd2692f6.zip |
Round readahead more inteligently and print warning.
Round readahead at least to one page up.
Print warning instead of verbose message.
Diffstat (limited to 'tools/lvchange.c')
-rw-r--r-- | tools/lvchange.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/tools/lvchange.c b/tools/lvchange.c index 10795e3b..5abb6e54 100644 --- a/tools/lvchange.c +++ b/tools/lvchange.c @@ -370,9 +370,12 @@ static int lvchange_readahead(struct cmd_context *cmd, if (read_ahead != DM_READ_AHEAD_AUTO && read_ahead != DM_READ_AHEAD_NONE && read_ahead % pagesize) { - read_ahead = (read_ahead / pagesize) * pagesize; - log_verbose("Rounding down readahead to %u sectors, a multiple " - "of page size %u.", read_ahead, pagesize); + if (read_ahead < pagesize) + read_ahead = pagesize; + else + read_ahead = (read_ahead / pagesize) * pagesize; + log_warn("WARNING: Overriding readahead to %u sectors, a multiple " + "of %uK page size.", read_ahead, pagesize >> 1); } if (lv->read_ahead == read_ahead) { |