md/md-llbitmap: grow the page cache in place for reshape

Use the page-control helpers to grow llbitmap's cached pages in place
for resize and later reshape preparation, instead of rebuilding the
whole cache.

Tested-by: Mykola Marzhan <mykola@meshstor.io>
Link: https://patch.msgid.link/20260802195038.164272-16-yukuai@kernel.org
Signed-off-by: Yu Kuai <yukuai@fygo.io>
This commit is contained in:
Yu Kuai 2026-08-03 03:50:24 +08:00
parent e9f0d66b57
commit 1cbc6ea5fa

View File

@ -416,6 +416,19 @@ static char state_machine[BitStateCount][BitmapActionCount] = {
};
static void __llbitmap_flush(struct mddev *mddev);
static void llbitmap_flush(struct mddev *mddev);
static void llbitmap_update_sb(void *data);
static void llbitmap_calculate_chunks(struct mddev *mddev, sector_t blocks,
unsigned long *chunksize,
unsigned long *chunks)
{
*chunks = DIV_ROUND_UP_SECTOR_T(blocks, *chunksize);
while (*chunks > mddev->bitmap_info.space << SECTOR_SHIFT) {
*chunksize = *chunksize << 1;
*chunks = DIV_ROUND_UP_SECTOR_T(blocks, *chunksize);
}
}
static enum llbitmap_state llbitmap_read(struct llbitmap *llbitmap, loff_t pos)
{
@ -655,6 +668,48 @@ static unsigned int llbitmap_reserved_pages(struct llbitmap *llbitmap)
PAGE_SIZE);
}
static int llbitmap_expand_pages(struct llbitmap *llbitmap,
unsigned long chunks)
{
struct llbitmap_page_ctl **pctl;
unsigned int old_nr_pages = llbitmap->nr_pages;
unsigned int nr_pages = llbitmap_used_pages(llbitmap, chunks);
unsigned int i;
int ret;
if (nr_pages <= old_nr_pages)
return 0;
pctl = kcalloc(nr_pages, sizeof(*pctl), GFP_NOIO);
if (!pctl)
return -ENOMEM;
if (llbitmap->pctl)
memcpy(pctl, llbitmap->pctl,
array_size(old_nr_pages, sizeof(*pctl)));
for (i = old_nr_pages; i < nr_pages; i++) {
pctl[i] = llbitmap_alloc_page_ctl(llbitmap, i);
if (IS_ERR(pctl[i]))
goto err_alloc_ptr;
}
kfree(llbitmap->pctl);
llbitmap->pctl = pctl;
llbitmap->nr_pages = nr_pages;
return 0;
err_alloc_ptr:
ret = PTR_ERR(pctl[i]);
while (i-- > old_nr_pages) {
__free_page(pctl[i]->page);
percpu_ref_exit(&pctl[i]->active);
kfree(pctl[i]);
}
kfree(pctl);
return ret;
}
static int llbitmap_alloc_pages(struct llbitmap *llbitmap)
{
unsigned int used_pages = llbitmap_used_pages(llbitmap, llbitmap->chunks);
@ -730,6 +785,34 @@ static bool llbitmap_zero_all_disks(struct llbitmap *llbitmap)
return true;
}
static void llbitmap_mark_range(struct llbitmap *llbitmap,
unsigned long start,
unsigned long end,
enum llbitmap_state state)
{
while (start <= end) {
llbitmap_write(llbitmap, state, start);
start++;
}
}
static int llbitmap_prepare_resize(struct llbitmap *llbitmap,
unsigned long old_chunks,
unsigned long new_chunks,
unsigned long cache_chunks)
{
int ret;
llbitmap_flush(llbitmap->mddev);
ret = llbitmap_expand_pages(llbitmap, cache_chunks);
if (ret)
return ret;
if (new_chunks > old_chunks)
llbitmap_mark_range(llbitmap, old_chunks, new_chunks - 1,
BitUnwritten);
return 0;
}
static void llbitmap_init_state(struct llbitmap *llbitmap)
{
struct mddev *mddev = llbitmap->mddev;
@ -1032,10 +1115,10 @@ static int llbitmap_read_sb(struct llbitmap *llbitmap)
goto out_put_page;
}
if (chunksize < DIV_ROUND_UP_SECTOR_T(mddev->resync_max_sectors,
if (chunksize < DIV_ROUND_UP_SECTOR_T(sync_size,
mddev->bitmap_info.space << SECTOR_SHIFT)) {
pr_err("md/llbitmap: %s: chunksize too small %lu < %llu / %lu",
mdname(mddev), chunksize, mddev->resync_max_sectors,
mdname(mddev), chunksize, sync_size,
mddev->bitmap_info.space);
goto out_put_page;
}
@ -1184,24 +1267,62 @@ static int llbitmap_create(struct mddev *mddev)
static int llbitmap_resize(struct mddev *mddev, sector_t blocks, int chunksize)
{
struct llbitmap *llbitmap = mddev->bitmap;
sector_t old_blocks = llbitmap->sync_size;
unsigned long old_chunks = llbitmap->chunks;
unsigned long chunks;
unsigned long cache_chunks;
int ret = 0;
unsigned long bitmap_chunksize;
bool reshape;
bool quiesced = false;
if (chunksize == 0)
chunksize = llbitmap->chunksize;
/* If there is enough space, leave the chunksize unchanged. */
chunks = DIV_ROUND_UP_SECTOR_T(blocks, chunksize);
while (chunks > mddev->bitmap_info.space << SECTOR_SHIFT) {
chunksize = chunksize << 1;
chunks = DIV_ROUND_UP_SECTOR_T(blocks, chunksize);
bitmap_chunksize = chunksize;
llbitmap_calculate_chunks(mddev, blocks, &bitmap_chunksize, &chunks);
reshape = mddev->delta_disks || mddev->new_level != mddev->level ||
mddev->new_layout != mddev->layout ||
mddev->new_chunk_sectors != mddev->chunk_sectors;
if (!reshape && bitmap_chunksize != llbitmap->chunksize)
return -EOPNOTSUPP;
if (blocks == old_blocks && chunks == llbitmap->chunks)
return 0;
if (mddev->pers->quiesce) {
mddev->pers->quiesce(mddev, 1);
quiesced = true;
}
llbitmap->chunkshift = ffz(~chunksize);
llbitmap->chunksize = chunksize;
llbitmap->chunks = chunks;
llbitmap->sync_size = blocks;
mutex_lock(&mddev->bitmap_info.mutex);
cache_chunks = reshape ? max(old_chunks, chunks) : chunks;
ret = llbitmap_prepare_resize(llbitmap, old_chunks, chunks, cache_chunks);
if (ret)
goto out;
if (reshape) {
llbitmap->chunks = max(old_chunks, chunks);
} else {
if (blocks < old_blocks && chunks < old_chunks)
llbitmap_mark_range(llbitmap, chunks, old_chunks - 1,
BitUnwritten);
mddev->bitmap_info.chunksize = bitmap_chunksize;
llbitmap->chunks = chunks;
llbitmap->sync_size = blocks;
llbitmap_update_sb(llbitmap);
}
__llbitmap_flush(mddev);
mutex_unlock(&mddev->bitmap_info.mutex);
if (quiesced)
mddev->pers->quiesce(mddev, 0);
return 0;
out:
mutex_unlock(&mddev->bitmap_info.mutex);
if (quiesced)
mddev->pers->quiesce(mddev, 0);
return ret;
}
static int llbitmap_load(struct mddev *mddev)