Hi Neil,

On Mon, 2 Mar 2015 17:03:45 +1100 NeilBrown <neilb@suse.de> wrote:
>
> I think
> +               bm_blocks = DIV_ROUND_UP(bm_blocks, 4096);
> 
> needs DIV_ROUND_UP_SECTOR_T()

I tried that and it was not sufficient.

> The first patch you identified adds that line.  The second relocates it.

The second also changes this:

bm_blocks = sector_div(bitmap->mddev->resync_max_sectors, (chunksize >> 9));

(added by the first) to this:

bm_blocks = bitmap->mddev->resync_max_sectors / (bitmap->mddev->bitmap_info.chunksize >> 9);

where bitmap->mddev->resync_max_sectors is a sector_t ...

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au