fs/buffer.c: remove block_write_full_page_endio()

The last in-tree caller of block_write_full_page_endio() was removed in
January 2013.  It's time to remove the EXPORT_SYMBOL, which leaves
block_write_full_page() as the only caller of
block_write_full_page_endio(), so inline block_write_full_page_endio()
into block_write_full_page().

Signed-off-by: Matthew Wilcox <matthew.r.wilcox@intel.com>
Cc: Hugh Dickins <hughd@google.com>
Cc: Dave Chinner <david@fromorbit.com>
Cc: Dheeraj Reddy <dheeraj.reddy@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

diff --git a/fs/buffer.c b/fs/buffer.c
index 9ddb9fc7d923fa31299a8aba228f61973d3b429f..7b5bb90bd0700be1a2c50b0caed0c5ec4c9cf37f 100644 (file)
--- a/fs/buffer.c
+++ b/fs/buffer.c
@@ -2879,10 +2879,9 @@ EXPORT_SYMBOL(block_truncate_page);
 
 /*
  * The generic ->writepage function for buffer-backed address_spaces
- * this form passes in the end_io handler used to finish the IO.
  */
-int block_write_full_page_endio(struct page *page, get_block_t *get_block,
-                       struct writeback_control *wbc, bh_end_io_t *handler)
+int block_write_full_page(struct page *page, get_block_t *get_block,
+                       struct writeback_control *wbc)
 {
        struct inode * const inode = page->mapping->host;
        loff_t i_size = i_size_read(inode);
@@ -2892,7 +2891,7 @@ int block_write_full_page_endio(struct page *page, get_block_t *get_block,
        /* Is the page fully inside i_size? */
        if (page->index < end_index)
                return __block_write_full_page(inode, page, get_block, wbc,
-                                              handler);
+                                              end_buffer_async_write);
 
        /* Is the page fully outside i_size? (truncate in progress) */
        offset = i_size & (PAGE_CACHE_SIZE-1);
@@ -2915,18 +2914,8 @@ int block_write_full_page_endio(struct page *page, get_block_t *get_block,
         * writes to that region are not written out to the file."
         */
        zero_user_segment(page, offset, PAGE_CACHE_SIZE);
-       return __block_write_full_page(inode, page, get_block, wbc, handler);
-}
-EXPORT_SYMBOL(block_write_full_page_endio);
-
-/*
- * The generic ->writepage function for buffer-backed address_spaces
- */
-int block_write_full_page(struct page *page, get_block_t *get_block,
-                       struct writeback_control *wbc)
-{
-       return block_write_full_page_endio(page, get_block, wbc,
-                                          end_buffer_async_write);
+       return __block_write_full_page(inode, page, get_block, wbc,
+                                                       end_buffer_async_write);
 }
 EXPORT_SYMBOL(block_write_full_page);
 

diff --git a/fs/ext4/page-io.c b/fs/ext4/page-io.c
index c18d95b5054081c75e0c7a2fab975976838f9b02..1a64e7a52b841be24ddd3daa56cb47939f07656c 100644 (file)
--- a/fs/ext4/page-io.c
+++ b/fs/ext4/page-io.c
@@ -429,7 +429,7 @@ int ext4_bio_write_page(struct ext4_io_submit *io,
                block_start = bh_offset(bh);
                if (block_start >= len) {
                        /*
-                        * Comments copied from block_write_full_page_endio:
+                        * Comments copied from block_write_full_page:
                         *
                         * The page straddles i_size.  It must be zeroed out on
                         * each and every writepage invocation because it may

diff --git a/fs/ocfs2/file.c b/fs/ocfs2/file.c
index 8970dcf74de53e71539e8fad7042a55f66a2c713..8eb6e5732d3b73b115abea0681cdd95d9195becf 100644 (file)
--- a/fs/ocfs2/file.c
+++ b/fs/ocfs2/file.c
@@ -828,7 +828,7 @@ static int ocfs2_write_zero_page(struct inode *inode, u64 abs_from,
                /*
                 * fs-writeback will release the dirty pages without page lock
                 * whose offset are over inode size, the release happens at
-                * block_write_full_page_endio().
+                * block_write_full_page().
                 */
                i_size_write(inode, abs_to);
                inode->i_blocks = ocfs2_inode_sector_count(inode);

diff --git a/include/linux/buffer_head.h b/include/linux/buffer_head.h
index c40302f909ce00910260e2818dd712d658257a77..e05c7ecdb24d13f037ab5ed222a8858d783e5a00 100644 (file)
--- a/include/linux/buffer_head.h
+++ b/include/linux/buffer_head.h
@@ -207,8 +207,6 @@ void block_invalidatepage(struct page *page, unsigned int offset,
                          unsigned int length);
 int block_write_full_page(struct page *page, get_block_t *get_block,
                                struct writeback_control *wbc);
-int block_write_full_page_endio(struct page *page, get_block_t *get_block,
-                       struct writeback_control *wbc, bh_end_io_t *handler);
 int block_read_full_page(struct page*, get_block_t*);
 int block_is_partially_uptodate(struct page *page, unsigned long from,
                                unsigned long count);