* Christoph Hellwig <[EMAIL PROTECTED]> wrote: > On Sat, Jan 06, 2007 at 07:31:52PM +0000, Christoph Hellwig wrote: > > On Sun, Jan 07, 2007 at 03:18:45AM +1100, Rusty Russell wrote: > > > PS. drm_memory.h has a "drm_follow_page": this forces us to uninline > > > various page tables ops. Can this use follow_page() somehow, or do we > > > need an "__follow_page" export for this case? > > > > Not if avoidable. And it seems avoidable as drm really should be using > > vmalloc_to_page. Untested patch below: > > Even better we can actualy avid most of the page table walks > completely.
agreed. I think there's an important side-observation here as well: having inlined functions uninlined and exported puts them under a lot more scrutiny. Hence individual exports instead of the global paravirt_ops export is a big plus. Ingo - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/