summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Rubin <mrubin@google.com>2010-08-20 02:31:26 -0700
committerSage Weil <sage@newdream.net>2010-08-22 15:16:51 -0700
commit679ceace848e9fd570678396ffe1ef034e00e82d (patch)
tree670768527852f134cfe3e20d425534a16edd968a
parenteb6bb1c5bdc6e455a9d16cb845cc65afc9b0a617 (diff)
downloadlinux-3.10-679ceace848e9fd570678396ffe1ef034e00e82d.tar.gz
linux-3.10-679ceace848e9fd570678396ffe1ef034e00e82d.tar.bz2
linux-3.10-679ceace848e9fd570678396ffe1ef034e00e82d.zip
mm: exporting account_page_dirty
This allows code outside of the mm core to safely manipulate page state and not worry about the other accounting. Not using these routines means that some code will lose track of the accounting and we get bugs. This has happened once already. Signed-off-by: Michael Rubin <mrubin@google.com> Signed-off-by: Sage Weil <sage@newdream.net>
-rw-r--r--fs/ceph/addr.c8
-rw-r--r--mm/page-writeback.c1
2 files changed, 2 insertions, 7 deletions
diff --git a/fs/ceph/addr.c b/fs/ceph/addr.c
index 5598a0d0229..420d46974ec 100644
--- a/fs/ceph/addr.c
+++ b/fs/ceph/addr.c
@@ -105,13 +105,7 @@ static int ceph_set_page_dirty(struct page *page)
spin_lock_irq(&mapping->tree_lock);
if (page->mapping) { /* Race with truncate? */
WARN_ON_ONCE(!PageUptodate(page));
-
- if (mapping_cap_account_dirty(mapping)) {
- __inc_zone_page_state(page, NR_FILE_DIRTY);
- __inc_bdi_stat(mapping->backing_dev_info,
- BDI_RECLAIMABLE);
- task_io_account_write(PAGE_CACHE_SIZE);
- }
+ account_page_dirtied(page, page->mapping);
radix_tree_tag_set(&mapping->page_tree,
page_index(page), PAGECACHE_TAG_DIRTY);
diff --git a/mm/page-writeback.c b/mm/page-writeback.c
index 37498ef6154..849d0ccbe91 100644
--- a/mm/page-writeback.c
+++ b/mm/page-writeback.c
@@ -1096,6 +1096,7 @@ void account_page_dirtied(struct page *page, struct address_space *mapping)
task_io_account_write(PAGE_CACHE_SIZE);
}
}
+EXPORT_SYMBOL(account_page_dirtied);
/*
* For address_spaces which do not use buffers. Just tag the page as dirty in