Re: [PATCHv2, RFC 09/30] thp, mm: rewrite delete_from_page_cache() to support huge pages

2013-03-15 Thread Kirill A. Shutemov
Hillf Danton wrote: > On Fri, Mar 15, 2013 at 1:50 AM, Kirill A. Shutemov > wrote: > > + if (PageTransHuge(page)) { > > + int i; > > + > > + for (i = 0; i < HPAGE_CACHE_NR; i++) > > + radix_tree_delete(>page_tree, page->index > > + i); > >

Re: [PATCHv2, RFC 09/30] thp, mm: rewrite delete_from_page_cache() to support huge pages

2013-03-15 Thread Kirill A. Shutemov
Hillf Danton wrote: On Fri, Mar 15, 2013 at 1:50 AM, Kirill A. Shutemov kirill.shute...@linux.intel.com wrote: + if (PageTransHuge(page)) { + int i; + + for (i = 0; i HPAGE_CACHE_NR; i++) + radix_tree_delete(mapping-page_tree,

Re: [PATCHv2, RFC 09/30] thp, mm: rewrite delete_from_page_cache() to support huge pages

2013-03-14 Thread Hillf Danton
On Fri, Mar 15, 2013 at 1:50 AM, Kirill A. Shutemov wrote: > + if (PageTransHuge(page)) { > + int i; > + > + for (i = 0; i < HPAGE_CACHE_NR; i++) > + radix_tree_delete(>page_tree, page->index + > i); Move the below page_cache_release for

[PATCHv2, RFC 09/30] thp, mm: rewrite delete_from_page_cache() to support huge pages

2013-03-14 Thread Kirill A. Shutemov
From: "Kirill A. Shutemov" As with add_to_page_cache_locked() we handle HPAGE_CACHE_NR pages a time. Signed-off-by: Kirill A. Shutemov --- mm/filemap.c | 27 +-- 1 file changed, 21 insertions(+), 6 deletions(-) diff --git a/mm/filemap.c b/mm/filemap.c index

[PATCHv2, RFC 09/30] thp, mm: rewrite delete_from_page_cache() to support huge pages

2013-03-14 Thread Kirill A. Shutemov
From: Kirill A. Shutemov kirill.shute...@linux.intel.com As with add_to_page_cache_locked() we handle HPAGE_CACHE_NR pages a time. Signed-off-by: Kirill A. Shutemov kirill.shute...@linux.intel.com --- mm/filemap.c | 27 +-- 1 file changed, 21 insertions(+), 6

Re: [PATCHv2, RFC 09/30] thp, mm: rewrite delete_from_page_cache() to support huge pages

2013-03-14 Thread Hillf Danton
On Fri, Mar 15, 2013 at 1:50 AM, Kirill A. Shutemov kirill.shute...@linux.intel.com wrote: + if (PageTransHuge(page)) { + int i; + + for (i = 0; i HPAGE_CACHE_NR; i++) + radix_tree_delete(mapping-page_tree, page-index + i); Move the