]> git.baikalelectronics.ru Git - kernel.git/commit
md/bitmap: md_bitmap_get_counter returns wrong blocks
authorZhao Heming <heming.zhao@suse.com>
Mon, 5 Oct 2020 16:00:24 +0000 (00:00 +0800)
committerSong Liu <songliubraving@fb.com>
Fri, 9 Oct 2020 05:31:29 +0000 (22:31 -0700)
commitab5150f324f5dabefd9bfc36d1897d34da1ea2ea
treeb46c1b95bd92c91fb6c106e25bbfcc87b2502cca
parent4bf6cc4d0b048f6efe3be3fa809f27011ad9f4dc
md/bitmap: md_bitmap_get_counter returns wrong blocks

md_bitmap_get_counter() has code:

```
    if (bitmap->bp[page].hijacked ||
        bitmap->bp[page].map == NULL)
        csize = ((sector_t)1) << (bitmap->chunkshift +
                      PAGE_COUNTER_SHIFT - 1);
```

The minus 1 is wrong, this branch should report 2048 bits of space.
With "-1" action, this only report 1024 bit of space.

This bug code returns wrong blocks, but it doesn't inflence bitmap logic:
1. Most callers focus this function return value (the counter of offset),
   not the parameter blocks.
2. The bug is only triggered when hijacked is true or map is NULL.
   the hijacked true condition is very rare.
   the "map == null" only true when array is creating or resizing.
3. Even the caller gets wrong blocks, current code makes caller just to
   call md_bitmap_get_counter() one more time.

Signed-off-by: Zhao Heming <heming.zhao@suse.com>
Signed-off-by: Song Liu <songliubraving@fb.com>
drivers/md/md-bitmap.c