lkml.org 
[lkml]   [2010]   [Dec]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 6/7] Good bye remove_from_page_cache
    Date
    Now delete_from_page_cache replaces remove_from_page_cache.
    So we remove remove_from_page_cache so fs or something out of
    mainline will notice it when compile time and can fix it.

    Cc: Christoph Hellwig <hch@infradead.org>
    Cc: Hugh Dickins <hughd@google.com>
    Signed-off-by: Minchan Kim <minchan.kim@gmail.com>
    ---
    include/linux/pagemap.h | 1 -
    mm/filemap.c | 27 +++++++++++----------------
    2 files changed, 11 insertions(+), 17 deletions(-)

    diff --git a/include/linux/pagemap.h b/include/linux/pagemap.h
    index 7a1cb49..7bf6587 100644
    --- a/include/linux/pagemap.h
    +++ b/include/linux/pagemap.h
    @@ -455,7 +455,6 @@ int add_to_page_cache_locked(struct page *page, struct address_space *mapping,
    pgoff_t index, gfp_t gfp_mask);
    int add_to_page_cache_lru(struct page *page, struct address_space *mapping,
    pgoff_t index, gfp_t gfp_mask);
    -extern void remove_from_page_cache(struct page *page);
    extern void __remove_from_page_cache(struct page *page);
    extern void delete_from_page_cache(struct page *page);

    diff --git a/mm/filemap.c b/mm/filemap.c
    index 1ca7475..a4c43f7 100644
    --- a/mm/filemap.c
    +++ b/mm/filemap.c
    @@ -148,7 +148,17 @@ void __remove_from_page_cache(struct page *page)
    }
    }

    -void remove_from_page_cache(struct page *page)
    +/**
    + * delete_from_page_cache - delete page from page cache
    + *
    + * @page: the page which the kernel is trying to remove from page cache
    + *
    + * This must be called only on pages that have
    + * been verified to be in the page cache and locked.
    + * It will never put the page into the free list,
    + * the caller has a reference on the page.
    + */
    +void delete_from_page_cache(struct page *page)
    {
    struct address_space *mapping = page->mapping;
    void (*freepage)(struct page *);
    @@ -163,22 +173,7 @@ void remove_from_page_cache(struct page *page)

    if (freepage)
    freepage(page);
    -}
    -EXPORT_SYMBOL(remove_from_page_cache);

    -/**
    - * delete_from_page_cache - delete page from page cache
    - *
    - * @page: the page which the kernel is trying to remove from page cache
    - *
    - * This must be called only on pages that have
    - * been verified to be in the page cache and locked.
    - * It will never put the page into the free list,
    - * the caller has a reference on the page.
    - */
    -void delete_from_page_cache(struct page *page)
    -{
    - remove_from_page_cache(page);
    page_cache_release(page);
    }
    EXPORT_SYMBOL(delete_from_page_cache);
    --
    1.7.0.4


    \
     
     \ /
      Last update: 2010-12-22 16:37    [W:3.970 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site