diff options
author | NeilBrown <neilb@suse.de> | 2006-01-06 03:20:39 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-01-06 11:34:07 -0500 |
commit | 7dd5d34c6c2da04e8cd0732e08b33cd3359e4bae (patch) | |
tree | a2272fd63abc351e8f07ad771b0abd077cc91145 /drivers/md/bitmap.c | |
parent | 097426f689f179747f3cd6b4749eb2a6b605702d (diff) |
[PATCH] md: remove inappropriate limits in md/bitmap configuration.
The kernel should not be imposing these policy limits: The time between
bitmap updates should certainly be allowed to be more than 15 seconds, and
if someone wants a bitmap chunk size in excess of 4MB, the kernel isn't the
place to stop them.
Signed-off-by: Neil Brown <neilb@suse.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/md/bitmap.c')
-rw-r--r-- | drivers/md/bitmap.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/md/bitmap.c b/drivers/md/bitmap.c index 519b1bf7a09a..abe415f0c039 100644 --- a/drivers/md/bitmap.c +++ b/drivers/md/bitmap.c | |||
@@ -485,12 +485,12 @@ static int bitmap_read_sb(struct bitmap *bitmap) | |||
485 | else if (le32_to_cpu(sb->version) < BITMAP_MAJOR_LO || | 485 | else if (le32_to_cpu(sb->version) < BITMAP_MAJOR_LO || |
486 | le32_to_cpu(sb->version) > BITMAP_MAJOR_HI) | 486 | le32_to_cpu(sb->version) > BITMAP_MAJOR_HI) |
487 | reason = "unrecognized superblock version"; | 487 | reason = "unrecognized superblock version"; |
488 | else if (chunksize < 512 || chunksize > (1024 * 1024 * 4)) | 488 | else if (chunksize < PAGE_SIZE) |
489 | reason = "bitmap chunksize out of range (512B - 4MB)"; | 489 | reason = "bitmap chunksize too small"; |
490 | else if ((1 << ffz(~chunksize)) != chunksize) | 490 | else if ((1 << ffz(~chunksize)) != chunksize) |
491 | reason = "bitmap chunksize not a power of 2"; | 491 | reason = "bitmap chunksize not a power of 2"; |
492 | else if (daemon_sleep < 1 || daemon_sleep > 15) | 492 | else if (daemon_sleep < 1 || daemon_sleep > MAX_SCHEDULE_TIMEOUT / HZ) |
493 | reason = "daemon sleep period out of range (1-15s)"; | 493 | reason = "daemon sleep period out of range"; |
494 | else if (write_behind > COUNTER_MAX) | 494 | else if (write_behind > COUNTER_MAX) |
495 | reason = "write-behind limit out of range (0 - 16383)"; | 495 | reason = "write-behind limit out of range (0 - 16383)"; |
496 | if (reason) { | 496 | if (reason) { |