提交 f2e06c58 编写于 作者: N NeilBrown

md/bitmap: remove confusing code from filemap_get_page.

file_page_index(store, 0) is *always* 0.
This is because the bitmap sb, at 256 bytes, is *always* less than
one page.
So subtracting it has no effect and the code should be removed.
Reported-by: NGoldwyn Rodrigues <rgoldwyn@suse.de>
Signed-off-by: NNeilBrown <neilb@suse.de>
上级 cf170f3f
...@@ -669,17 +669,13 @@ static inline unsigned long file_page_offset(struct bitmap_storage *store, ...@@ -669,17 +669,13 @@ static inline unsigned long file_page_offset(struct bitmap_storage *store,
/* /*
* return a pointer to the page in the filemap that contains the given bit * return a pointer to the page in the filemap that contains the given bit
* *
* this lookup is complicated by the fact that the bitmap sb might be exactly
* 1 page (e.g., x86) or less than 1 page -- so the bitmap might start on page
* 0 or page 1
*/ */
static inline struct page *filemap_get_page(struct bitmap_storage *store, static inline struct page *filemap_get_page(struct bitmap_storage *store,
unsigned long chunk) unsigned long chunk)
{ {
if (file_page_index(store, chunk) >= store->file_pages) if (file_page_index(store, chunk) >= store->file_pages)
return NULL; return NULL;
return store->filemap[file_page_index(store, chunk) return store->filemap[file_page_index(store, chunk)];
- file_page_index(store, 0)];
} }
static int bitmap_storage_alloc(struct bitmap_storage *store, static int bitmap_storage_alloc(struct bitmap_storage *store,
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册