[PATCH] md: clean up 'page' related names in md
Substitute: page_cache_get -> get_page page_cache_release -> put_page PAGE_CACHE_SHIFT -> PAGE_SHIFT PAGE_CACHE_SIZE -> PAGE_SIZE PAGE_CACHE_MASK -> PAGE_MASK __free_page -> put_page because we aren't using the page cache, we are just using pages. Signed-off-by: Neil Brown <neilb@suse.de> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
This commit is contained in:
parent
d7603b7e3a
commit
2d1f3b5d1b
|
@ -341,7 +341,7 @@ static int write_page(struct bitmap *bitmap, struct page *page, int wait)
|
||||||
/* add to list to be waited for by daemon */
|
/* add to list to be waited for by daemon */
|
||||||
struct page_list *item = mempool_alloc(bitmap->write_pool, GFP_NOIO);
|
struct page_list *item = mempool_alloc(bitmap->write_pool, GFP_NOIO);
|
||||||
item->page = page;
|
item->page = page;
|
||||||
page_cache_get(page);
|
get_page(page);
|
||||||
spin_lock(&bitmap->write_lock);
|
spin_lock(&bitmap->write_lock);
|
||||||
list_add(&item->list, &bitmap->complete_pages);
|
list_add(&item->list, &bitmap->complete_pages);
|
||||||
spin_unlock(&bitmap->write_lock);
|
spin_unlock(&bitmap->write_lock);
|
||||||
|
@ -357,10 +357,10 @@ static struct page *read_page(struct file *file, unsigned long index,
|
||||||
struct inode *inode = file->f_mapping->host;
|
struct inode *inode = file->f_mapping->host;
|
||||||
struct page *page = NULL;
|
struct page *page = NULL;
|
||||||
loff_t isize = i_size_read(inode);
|
loff_t isize = i_size_read(inode);
|
||||||
unsigned long end_index = isize >> PAGE_CACHE_SHIFT;
|
unsigned long end_index = isize >> PAGE_SHIFT;
|
||||||
|
|
||||||
PRINTK("read bitmap file (%dB @ %Lu)\n", (int)PAGE_CACHE_SIZE,
|
PRINTK("read bitmap file (%dB @ %Lu)\n", (int)PAGE_SIZE,
|
||||||
(unsigned long long)index << PAGE_CACHE_SHIFT);
|
(unsigned long long)index << PAGE_SHIFT);
|
||||||
|
|
||||||
page = read_cache_page(inode->i_mapping, index,
|
page = read_cache_page(inode->i_mapping, index,
|
||||||
(filler_t *)inode->i_mapping->a_ops->readpage, file);
|
(filler_t *)inode->i_mapping->a_ops->readpage, file);
|
||||||
|
@ -368,7 +368,7 @@ static struct page *read_page(struct file *file, unsigned long index,
|
||||||
goto out;
|
goto out;
|
||||||
wait_on_page_locked(page);
|
wait_on_page_locked(page);
|
||||||
if (!PageUptodate(page) || PageError(page)) {
|
if (!PageUptodate(page) || PageError(page)) {
|
||||||
page_cache_release(page);
|
put_page(page);
|
||||||
page = ERR_PTR(-EIO);
|
page = ERR_PTR(-EIO);
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
@ -376,14 +376,14 @@ static struct page *read_page(struct file *file, unsigned long index,
|
||||||
if (index > end_index) /* we have read beyond EOF */
|
if (index > end_index) /* we have read beyond EOF */
|
||||||
*bytes_read = 0;
|
*bytes_read = 0;
|
||||||
else if (index == end_index) /* possible short read */
|
else if (index == end_index) /* possible short read */
|
||||||
*bytes_read = isize & ~PAGE_CACHE_MASK;
|
*bytes_read = isize & ~PAGE_MASK;
|
||||||
else
|
else
|
||||||
*bytes_read = PAGE_CACHE_SIZE; /* got a full page */
|
*bytes_read = PAGE_SIZE; /* got a full page */
|
||||||
out:
|
out:
|
||||||
if (IS_ERR(page))
|
if (IS_ERR(page))
|
||||||
printk(KERN_ALERT "md: bitmap read error: (%dB @ %Lu): %ld\n",
|
printk(KERN_ALERT "md: bitmap read error: (%dB @ %Lu): %ld\n",
|
||||||
(int)PAGE_CACHE_SIZE,
|
(int)PAGE_SIZE,
|
||||||
(unsigned long long)index << PAGE_CACHE_SHIFT,
|
(unsigned long long)index << PAGE_SHIFT,
|
||||||
PTR_ERR(page));
|
PTR_ERR(page));
|
||||||
return page;
|
return page;
|
||||||
}
|
}
|
||||||
|
@ -558,7 +558,7 @@ static void bitmap_mask_state(struct bitmap *bitmap, enum bitmap_state bits,
|
||||||
spin_unlock_irqrestore(&bitmap->lock, flags);
|
spin_unlock_irqrestore(&bitmap->lock, flags);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
page_cache_get(bitmap->sb_page);
|
get_page(bitmap->sb_page);
|
||||||
spin_unlock_irqrestore(&bitmap->lock, flags);
|
spin_unlock_irqrestore(&bitmap->lock, flags);
|
||||||
sb = (bitmap_super_t *)kmap(bitmap->sb_page);
|
sb = (bitmap_super_t *)kmap(bitmap->sb_page);
|
||||||
switch (op) {
|
switch (op) {
|
||||||
|
@ -569,7 +569,7 @@ static void bitmap_mask_state(struct bitmap *bitmap, enum bitmap_state bits,
|
||||||
default: BUG();
|
default: BUG();
|
||||||
}
|
}
|
||||||
kunmap(bitmap->sb_page);
|
kunmap(bitmap->sb_page);
|
||||||
page_cache_release(bitmap->sb_page);
|
put_page(bitmap->sb_page);
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
@ -622,12 +622,12 @@ static void bitmap_file_unmap(struct bitmap *bitmap)
|
||||||
|
|
||||||
while (pages--)
|
while (pages--)
|
||||||
if (map[pages]->index != 0) /* 0 is sb_page, release it below */
|
if (map[pages]->index != 0) /* 0 is sb_page, release it below */
|
||||||
page_cache_release(map[pages]);
|
put_page(map[pages]);
|
||||||
kfree(map);
|
kfree(map);
|
||||||
kfree(attr);
|
kfree(attr);
|
||||||
|
|
||||||
if (sb_page)
|
if (sb_page)
|
||||||
page_cache_release(sb_page);
|
put_page(sb_page);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void bitmap_stop_daemon(struct bitmap *bitmap);
|
static void bitmap_stop_daemon(struct bitmap *bitmap);
|
||||||
|
@ -654,7 +654,7 @@ static void drain_write_queues(struct bitmap *bitmap)
|
||||||
|
|
||||||
while ((item = dequeue_page(bitmap))) {
|
while ((item = dequeue_page(bitmap))) {
|
||||||
/* don't bother to wait */
|
/* don't bother to wait */
|
||||||
page_cache_release(item->page);
|
put_page(item->page);
|
||||||
mempool_free(item, bitmap->write_pool);
|
mempool_free(item, bitmap->write_pool);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -763,7 +763,7 @@ static void bitmap_file_set_bit(struct bitmap *bitmap, sector_t block)
|
||||||
|
|
||||||
/* make sure the page stays cached until it gets written out */
|
/* make sure the page stays cached until it gets written out */
|
||||||
if (! (get_page_attr(bitmap, page) & BITMAP_PAGE_DIRTY))
|
if (! (get_page_attr(bitmap, page) & BITMAP_PAGE_DIRTY))
|
||||||
page_cache_get(page);
|
get_page(page);
|
||||||
|
|
||||||
/* set the bit */
|
/* set the bit */
|
||||||
kaddr = kmap_atomic(page, KM_USER0);
|
kaddr = kmap_atomic(page, KM_USER0);
|
||||||
|
@ -938,7 +938,7 @@ static int bitmap_init_from_disk(struct bitmap *bitmap, sector_t start)
|
||||||
if (ret) {
|
if (ret) {
|
||||||
kunmap(page);
|
kunmap(page);
|
||||||
/* release, page not in filemap yet */
|
/* release, page not in filemap yet */
|
||||||
page_cache_release(page);
|
put_page(page);
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -1043,7 +1043,7 @@ int bitmap_daemon_work(struct bitmap *bitmap)
|
||||||
/* skip this page unless it's marked as needing cleaning */
|
/* skip this page unless it's marked as needing cleaning */
|
||||||
if (!((attr=get_page_attr(bitmap, page)) & BITMAP_PAGE_CLEAN)) {
|
if (!((attr=get_page_attr(bitmap, page)) & BITMAP_PAGE_CLEAN)) {
|
||||||
if (attr & BITMAP_PAGE_NEEDWRITE) {
|
if (attr & BITMAP_PAGE_NEEDWRITE) {
|
||||||
page_cache_get(page);
|
get_page(page);
|
||||||
clear_page_attr(bitmap, page, BITMAP_PAGE_NEEDWRITE);
|
clear_page_attr(bitmap, page, BITMAP_PAGE_NEEDWRITE);
|
||||||
}
|
}
|
||||||
spin_unlock_irqrestore(&bitmap->lock, flags);
|
spin_unlock_irqrestore(&bitmap->lock, flags);
|
||||||
|
@ -1057,13 +1057,13 @@ int bitmap_daemon_work(struct bitmap *bitmap)
|
||||||
default:
|
default:
|
||||||
bitmap_file_kick(bitmap);
|
bitmap_file_kick(bitmap);
|
||||||
}
|
}
|
||||||
page_cache_release(page);
|
put_page(page);
|
||||||
}
|
}
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* grab the new page, sync and release the old */
|
/* grab the new page, sync and release the old */
|
||||||
page_cache_get(page);
|
get_page(page);
|
||||||
if (lastpage != NULL) {
|
if (lastpage != NULL) {
|
||||||
if (get_page_attr(bitmap, lastpage) & BITMAP_PAGE_NEEDWRITE) {
|
if (get_page_attr(bitmap, lastpage) & BITMAP_PAGE_NEEDWRITE) {
|
||||||
clear_page_attr(bitmap, lastpage, BITMAP_PAGE_NEEDWRITE);
|
clear_page_attr(bitmap, lastpage, BITMAP_PAGE_NEEDWRITE);
|
||||||
|
@ -1078,7 +1078,7 @@ int bitmap_daemon_work(struct bitmap *bitmap)
|
||||||
spin_unlock_irqrestore(&bitmap->lock, flags);
|
spin_unlock_irqrestore(&bitmap->lock, flags);
|
||||||
}
|
}
|
||||||
kunmap(lastpage);
|
kunmap(lastpage);
|
||||||
page_cache_release(lastpage);
|
put_page(lastpage);
|
||||||
if (err)
|
if (err)
|
||||||
bitmap_file_kick(bitmap);
|
bitmap_file_kick(bitmap);
|
||||||
} else
|
} else
|
||||||
|
@ -1133,7 +1133,7 @@ int bitmap_daemon_work(struct bitmap *bitmap)
|
||||||
spin_unlock_irqrestore(&bitmap->lock, flags);
|
spin_unlock_irqrestore(&bitmap->lock, flags);
|
||||||
}
|
}
|
||||||
|
|
||||||
page_cache_release(lastpage);
|
put_page(lastpage);
|
||||||
}
|
}
|
||||||
|
|
||||||
return err;
|
return err;
|
||||||
|
@ -1184,7 +1184,7 @@ static void bitmap_writeback_daemon(mddev_t *mddev)
|
||||||
PRINTK("finished page writeback: %p\n", page);
|
PRINTK("finished page writeback: %p\n", page);
|
||||||
|
|
||||||
err = PageError(page);
|
err = PageError(page);
|
||||||
page_cache_release(page);
|
put_page(page);
|
||||||
if (err) {
|
if (err) {
|
||||||
printk(KERN_WARNING "%s: bitmap file writeback "
|
printk(KERN_WARNING "%s: bitmap file writeback "
|
||||||
"failed (page %lu): %d\n",
|
"failed (page %lu): %d\n",
|
||||||
|
|
|
@ -339,7 +339,7 @@ static int alloc_disk_sb(mdk_rdev_t * rdev)
|
||||||
static void free_disk_sb(mdk_rdev_t * rdev)
|
static void free_disk_sb(mdk_rdev_t * rdev)
|
||||||
{
|
{
|
||||||
if (rdev->sb_page) {
|
if (rdev->sb_page) {
|
||||||
page_cache_release(rdev->sb_page);
|
put_page(rdev->sb_page);
|
||||||
rdev->sb_loaded = 0;
|
rdev->sb_loaded = 0;
|
||||||
rdev->sb_page = NULL;
|
rdev->sb_page = NULL;
|
||||||
rdev->sb_offset = 0;
|
rdev->sb_offset = 0;
|
||||||
|
|
|
@ -361,7 +361,7 @@ static int raid0_run (mddev_t *mddev)
|
||||||
* chunksize should be used in that case.
|
* chunksize should be used in that case.
|
||||||
*/
|
*/
|
||||||
{
|
{
|
||||||
int stripe = mddev->raid_disks * mddev->chunk_size / PAGE_CACHE_SIZE;
|
int stripe = mddev->raid_disks * mddev->chunk_size / PAGE_SIZE;
|
||||||
if (mddev->queue->backing_dev_info.ra_pages < 2* stripe)
|
if (mddev->queue->backing_dev_info.ra_pages < 2* stripe)
|
||||||
mddev->queue->backing_dev_info.ra_pages = 2* stripe;
|
mddev->queue->backing_dev_info.ra_pages = 2* stripe;
|
||||||
}
|
}
|
||||||
|
|
|
@ -139,7 +139,7 @@ static void * r1buf_pool_alloc(gfp_t gfp_flags, void *data)
|
||||||
out_free_pages:
|
out_free_pages:
|
||||||
for (i=0; i < RESYNC_PAGES ; i++)
|
for (i=0; i < RESYNC_PAGES ; i++)
|
||||||
for (j=0 ; j < pi->raid_disks; j++)
|
for (j=0 ; j < pi->raid_disks; j++)
|
||||||
__free_page(r1_bio->bios[j]->bi_io_vec[i].bv_page);
|
put_page(r1_bio->bios[j]->bi_io_vec[i].bv_page);
|
||||||
j = -1;
|
j = -1;
|
||||||
out_free_bio:
|
out_free_bio:
|
||||||
while ( ++j < pi->raid_disks )
|
while ( ++j < pi->raid_disks )
|
||||||
|
@ -159,7 +159,7 @@ static void r1buf_pool_free(void *__r1_bio, void *data)
|
||||||
if (j == 0 ||
|
if (j == 0 ||
|
||||||
r1bio->bios[j]->bi_io_vec[i].bv_page !=
|
r1bio->bios[j]->bi_io_vec[i].bv_page !=
|
||||||
r1bio->bios[0]->bi_io_vec[i].bv_page)
|
r1bio->bios[0]->bi_io_vec[i].bv_page)
|
||||||
__free_page(r1bio->bios[j]->bi_io_vec[i].bv_page);
|
put_page(r1bio->bios[j]->bi_io_vec[i].bv_page);
|
||||||
}
|
}
|
||||||
for (i=0 ; i < pi->raid_disks; i++)
|
for (i=0 ; i < pi->raid_disks; i++)
|
||||||
bio_put(r1bio->bios[i]);
|
bio_put(r1bio->bios[i]);
|
||||||
|
@ -384,7 +384,7 @@ static int raid1_end_write_request(struct bio *bio, unsigned int bytes_done, int
|
||||||
/* free extra copy of the data pages */
|
/* free extra copy of the data pages */
|
||||||
int i = bio->bi_vcnt;
|
int i = bio->bi_vcnt;
|
||||||
while (i--)
|
while (i--)
|
||||||
__free_page(bio->bi_io_vec[i].bv_page);
|
put_page(bio->bi_io_vec[i].bv_page);
|
||||||
}
|
}
|
||||||
/* clear the bitmap if all writes complete successfully */
|
/* clear the bitmap if all writes complete successfully */
|
||||||
bitmap_endwrite(r1_bio->mddev->bitmap, r1_bio->sector,
|
bitmap_endwrite(r1_bio->mddev->bitmap, r1_bio->sector,
|
||||||
|
@ -733,7 +733,7 @@ static struct page **alloc_behind_pages(struct bio *bio)
|
||||||
do_sync_io:
|
do_sync_io:
|
||||||
if (pages)
|
if (pages)
|
||||||
for (i = 0; i < bio->bi_vcnt && pages[i]; i++)
|
for (i = 0; i < bio->bi_vcnt && pages[i]; i++)
|
||||||
__free_page(pages[i]);
|
put_page(pages[i]);
|
||||||
kfree(pages);
|
kfree(pages);
|
||||||
PRINTK("%dB behind alloc failed, doing sync I/O\n", bio->bi_size);
|
PRINTK("%dB behind alloc failed, doing sync I/O\n", bio->bi_size);
|
||||||
return NULL;
|
return NULL;
|
||||||
|
@ -1893,7 +1893,7 @@ out_free_conf:
|
||||||
if (conf->r1bio_pool)
|
if (conf->r1bio_pool)
|
||||||
mempool_destroy(conf->r1bio_pool);
|
mempool_destroy(conf->r1bio_pool);
|
||||||
kfree(conf->mirrors);
|
kfree(conf->mirrors);
|
||||||
__free_page(conf->tmppage);
|
put_page(conf->tmppage);
|
||||||
kfree(conf->poolinfo);
|
kfree(conf->poolinfo);
|
||||||
kfree(conf);
|
kfree(conf);
|
||||||
mddev->private = NULL;
|
mddev->private = NULL;
|
||||||
|
|
|
@ -134,10 +134,10 @@ static void * r10buf_pool_alloc(gfp_t gfp_flags, void *data)
|
||||||
|
|
||||||
out_free_pages:
|
out_free_pages:
|
||||||
for ( ; i > 0 ; i--)
|
for ( ; i > 0 ; i--)
|
||||||
__free_page(bio->bi_io_vec[i-1].bv_page);
|
put_page(bio->bi_io_vec[i-1].bv_page);
|
||||||
while (j--)
|
while (j--)
|
||||||
for (i = 0; i < RESYNC_PAGES ; i++)
|
for (i = 0; i < RESYNC_PAGES ; i++)
|
||||||
__free_page(r10_bio->devs[j].bio->bi_io_vec[i].bv_page);
|
put_page(r10_bio->devs[j].bio->bi_io_vec[i].bv_page);
|
||||||
j = -1;
|
j = -1;
|
||||||
out_free_bio:
|
out_free_bio:
|
||||||
while ( ++j < nalloc )
|
while ( ++j < nalloc )
|
||||||
|
@ -157,7 +157,7 @@ static void r10buf_pool_free(void *__r10_bio, void *data)
|
||||||
struct bio *bio = r10bio->devs[j].bio;
|
struct bio *bio = r10bio->devs[j].bio;
|
||||||
if (bio) {
|
if (bio) {
|
||||||
for (i = 0; i < RESYNC_PAGES; i++) {
|
for (i = 0; i < RESYNC_PAGES; i++) {
|
||||||
__free_page(bio->bi_io_vec[i].bv_page);
|
put_page(bio->bi_io_vec[i].bv_page);
|
||||||
bio->bi_io_vec[i].bv_page = NULL;
|
bio->bi_io_vec[i].bv_page = NULL;
|
||||||
}
|
}
|
||||||
bio_put(bio);
|
bio_put(bio);
|
||||||
|
@ -2015,7 +2015,7 @@ static int run(mddev_t *mddev)
|
||||||
* maybe...
|
* maybe...
|
||||||
*/
|
*/
|
||||||
{
|
{
|
||||||
int stripe = conf->raid_disks * mddev->chunk_size / PAGE_CACHE_SIZE;
|
int stripe = conf->raid_disks * mddev->chunk_size / PAGE_SIZE;
|
||||||
stripe /= conf->near_copies;
|
stripe /= conf->near_copies;
|
||||||
if (mddev->queue->backing_dev_info.ra_pages < 2* stripe)
|
if (mddev->queue->backing_dev_info.ra_pages < 2* stripe)
|
||||||
mddev->queue->backing_dev_info.ra_pages = 2* stripe;
|
mddev->queue->backing_dev_info.ra_pages = 2* stripe;
|
||||||
|
|
|
@ -167,7 +167,7 @@ static void shrink_buffers(struct stripe_head *sh, int num)
|
||||||
if (!p)
|
if (!p)
|
||||||
continue;
|
continue;
|
||||||
sh->dev[i].page = NULL;
|
sh->dev[i].page = NULL;
|
||||||
page_cache_release(p);
|
put_page(p);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1956,7 +1956,7 @@ static int run(mddev_t *mddev)
|
||||||
*/
|
*/
|
||||||
{
|
{
|
||||||
int stripe = (mddev->raid_disks-1) * mddev->chunk_size
|
int stripe = (mddev->raid_disks-1) * mddev->chunk_size
|
||||||
/ PAGE_CACHE_SIZE;
|
/ PAGE_SIZE;
|
||||||
if (mddev->queue->backing_dev_info.ra_pages < 2 * stripe)
|
if (mddev->queue->backing_dev_info.ra_pages < 2 * stripe)
|
||||||
mddev->queue->backing_dev_info.ra_pages = 2 * stripe;
|
mddev->queue->backing_dev_info.ra_pages = 2 * stripe;
|
||||||
}
|
}
|
||||||
|
|
|
@ -186,7 +186,7 @@ static void shrink_buffers(struct stripe_head *sh, int num)
|
||||||
if (!p)
|
if (!p)
|
||||||
continue;
|
continue;
|
||||||
sh->dev[i].page = NULL;
|
sh->dev[i].page = NULL;
|
||||||
page_cache_release(p);
|
put_page(p);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -2069,7 +2069,7 @@ static int run(mddev_t *mddev)
|
||||||
*/
|
*/
|
||||||
{
|
{
|
||||||
int stripe = (mddev->raid_disks-2) * mddev->chunk_size
|
int stripe = (mddev->raid_disks-2) * mddev->chunk_size
|
||||||
/ PAGE_CACHE_SIZE;
|
/ PAGE_SIZE;
|
||||||
if (mddev->queue->backing_dev_info.ra_pages < 2 * stripe)
|
if (mddev->queue->backing_dev_info.ra_pages < 2 * stripe)
|
||||||
mddev->queue->backing_dev_info.ra_pages = 2 * stripe;
|
mddev->queue->backing_dev_info.ra_pages = 2 * stripe;
|
||||||
}
|
}
|
||||||
|
@ -2084,7 +2084,7 @@ abort:
|
||||||
if (conf) {
|
if (conf) {
|
||||||
print_raid6_conf(conf);
|
print_raid6_conf(conf);
|
||||||
if (conf->spare_page)
|
if (conf->spare_page)
|
||||||
page_cache_release(conf->spare_page);
|
put_page(conf->spare_page);
|
||||||
if (conf->stripe_hashtbl)
|
if (conf->stripe_hashtbl)
|
||||||
free_pages((unsigned long) conf->stripe_hashtbl,
|
free_pages((unsigned long) conf->stripe_hashtbl,
|
||||||
HASH_PAGES_ORDER);
|
HASH_PAGES_ORDER);
|
||||||
|
|
Loading…
Reference in New Issue