On Fri, Mar 15, 2013 at 9:23 PM, Kirill A. Shutemov <kirill.shutemov@xxxxxxxxxxxxxxx> wrote: > Hillf Danton wrote: >> On Fri, Mar 15, 2013 at 1:50 AM, Kirill A. Shutemov >> <kirill.shutemov@xxxxxxxxxxxxxxx> wrote: >> > + page_cache_get(page); >> > + spin_lock_irq(&mapping->tree_lock); >> > + page->mapping = mapping; >> > + page->index = offset; >> > + error = radix_tree_insert(&mapping->page_tree, offset, page); >> > + if (unlikely(error)) >> > + goto err; >> > + if (PageTransHuge(page)) { >> > + int i; >> > + for (i = 1; i < HPAGE_CACHE_NR; i++) { >> struct page *tail = page + i; to easy reader >> >> > + page_cache_get(page + i); >> s/page_cache_get/get_page_foll/ ? > > Why? > see follow_trans_huge_pmd() please. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html