aboutsummaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorNeilBrown <neilb@suse.de>2008-10-12 20:55:12 -0400
committerNeilBrown <neilb@suse.de>2008-10-12 20:55:12 -0400
commit4bbf3771ca40d0aaec8316d0e7476b16010288e5 (patch)
treef7cba1f72612c5d2132f549881c46ff572e01933 /drivers
parentd710e13812600037a723a673dc5c96a071de98d3 (diff)
md: Relax minimum size restrictions on chunk_size.
Currently, the 'chunk_size' of an array must be at-least PAGE_SIZE. This makes moving an array to a machine with a larger PAGE_SIZE, or changing the kernel to use a larger PAGE_SIZE, can stop an array from working. For RAID10 and RAID4/5/6, this is non-trivial to fix as the resync process works on whole pages at a time, and assumes them to be wholly within a stripe. For other raid personalities, this restriction is not needed at all and can be dropped. So remove the test on chunk_size from common can, and add it in just the places where it is needed: raid10 and raid4/5/6. Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/md/md.c7
-rw-r--r--drivers/md/raid10.c5
-rw-r--r--drivers/md/raid5.c7
3 files changed, 11 insertions, 8 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c
index a29187d1fcf5..be2014f6e37b 100644
--- a/drivers/md/md.c
+++ b/drivers/md/md.c
@@ -3520,17 +3520,12 @@ static int do_md_run(mddev_t * mddev)
3520 return -EINVAL; 3520 return -EINVAL;
3521 } 3521 }
3522 /* 3522 /*
3523 * chunk-size has to be a power of 2 and multiples of PAGE_SIZE 3523 * chunk-size has to be a power of 2
3524 */ 3524 */
3525 if ( (1 << ffz(~chunk_size)) != chunk_size) { 3525 if ( (1 << ffz(~chunk_size)) != chunk_size) {
3526 printk(KERN_ERR "chunk_size of %d not valid\n", chunk_size); 3526 printk(KERN_ERR "chunk_size of %d not valid\n", chunk_size);
3527 return -EINVAL; 3527 return -EINVAL;
3528 } 3528 }
3529 if (chunk_size < PAGE_SIZE) {
3530 printk(KERN_ERR "too small chunk_size: %d < %ld\n",
3531 chunk_size, PAGE_SIZE);
3532 return -EINVAL;
3533 }
3534 3529
3535 /* devices must have minimum size of one chunk */ 3530 /* devices must have minimum size of one chunk */
3536 rdev_for_each(rdev, tmp, mddev) { 3531 rdev_for_each(rdev, tmp, mddev) {
diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c
index 8bdc9bfc2887..e3794799f308 100644
--- a/drivers/md/raid10.c
+++ b/drivers/md/raid10.c
@@ -2028,8 +2028,9 @@ static int run(mddev_t *mddev)
2028 int nc, fc, fo; 2028 int nc, fc, fo;
2029 sector_t stride, size; 2029 sector_t stride, size;
2030 2030
2031 if (mddev->chunk_size == 0) { 2031 if (mddev->chunk_size < PAGE_SIZE) {
2032 printk(KERN_ERR "md/raid10: non-zero chunk size required.\n"); 2032 printk(KERN_ERR "md/raid10: chunk size must be "
2033 "at least PAGE_SIZE(%ld).\n", PAGE_SIZE);
2033 return -EINVAL; 2034 return -EINVAL;
2034 } 2035 }
2035 2036
diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c
index d72be4b89e64..a36a7435edf5 100644
--- a/drivers/md/raid5.c
+++ b/drivers/md/raid5.c
@@ -4007,6 +4007,13 @@ static int run(mddev_t *mddev)
4007 return -EIO; 4007 return -EIO;
4008 } 4008 }
4009 4009
4010 if (mddev->chunk_size < PAGE_SIZE) {
4011 printk(KERN_ERR "md/raid5: chunk_size must be at least "
4012 "PAGE_SIZE but %d < %ld\n",
4013 mddev->chunk_size, PAGE_SIZE);
4014 return -EINVAL;
4015 }
4016
4010 if (mddev->reshape_position != MaxSector) { 4017 if (mddev->reshape_position != MaxSector) {
4011 /* Check that we can continue the reshape. 4018 /* Check that we can continue the reshape.
4012 * Currently only disks can change, it must 4019 * Currently only disks can change, it must