On Thu, 1 Mar 2012 00:14:53 +0100 Daniel Vetter <daniel at ffwll.ch> wrote: > I'll redo this patch by adding _multipage versions of these 2 functions > for i915. OK, but I hope "for i915" doesn't mean "private to"! Put 'em in pagemap.h, for maintenance reasons and because they are generic. Making them inline is a bit sad, but that's OK for now - they have few callsites.