Also... On Mon, 4 Feb 2013 16:06:24 -0800 Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote: > > +put_page: > > + /* Undo the effects of former get_user_pages(), we won't pin anything */ > > + for (i = 0; i < ret; i++) > > + put_page(pages[i]); We can use release_pages() here. release_pages() is designed to be more efficient when we're putting the final reference to (most of) the pages. It probably has little if any benefit when putting still-in-use pages, as we're doing here. But please consider... -- 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