]> git.neil.brown.name Git - mdadm.git/commitdiff
super-0.90: don't write bitmap larger than 60K
authorNeilBrown <neilb@suse.de>
Wed, 7 Jul 2010 11:09:23 +0000 (21:09 +1000)
committerNeilBrown <neilb@suse.de>
Wed, 7 Jul 2010 11:31:33 +0000 (21:31 +1000)
The 4K superblock can be as close as 64K from the end
of the device.  As the bitmap (with header) lives after
the superblock (with 0.90 metadata) there could be as
little as 60K of space.
So limit the bitmaps to 59.5K, and only write 60K including
the header.

The bug fixed here means that bitmaps cannot be created
on devices which are exact multiples of 64K in size

Signed-off-by: NeilBrown <neilb@suse.de>
super0.c

index 83600cbf34e2a91a8a9b92c4802ff6cb0fce93d6..b8dfdb44eb3723aaf4549c840a2a8c6f85f397ca 100644 (file)
--- a/super0.c
+++ b/super0.c
@@ -962,7 +962,7 @@ static int add_internal_bitmap0(struct supertype *st, int *chunkp,
         * size is in sectors,  chunk is in bytes !!!
         */
        unsigned long long bits;
-       unsigned long long max_bits = 60*1024*8;
+       unsigned long long max_bits = (60*1024 - sizeof(bitmap_super_t))*8;
        unsigned long long min_chunk;
        int chunk = *chunkp;
        mdp_super_t *sb = st->sb;
@@ -1048,7 +1048,7 @@ static int write_bitmap0(struct supertype *st, int fd)
 
        memset(buf, 0xff, 4096);
        memcpy(buf,  ((char*)sb)+MD_SB_BYTES, sizeof(bitmap_super_t));
-       towrite = 64*1024;
+       towrite = 60*1024;
        while (towrite > 0) {
                n = towrite;
                if (n > 4096)