On Fri, Apr 13, 2012 at 03:37:49PM +0100, Chris Wilson wrote: > On Fri, 13 Apr 2012 16:26:31 +0200, Daniel Vetter <daniel at ffwll.ch> wrote: > > On Fri, Apr 13, 2012 at 03:21:42PM +0100, Chris Wilson wrote: > > > And share with i915_gem_tiling.c. Or is that overkill? > > > > The function we might want here already exists as > > i915_gem_object_needs_bit17_swizzle but I've figured that's overkill. If > > you do tiled pwrites, you're doing it wrong. No point jumping through > > hoops for it. > > Fixes regression from d9e86c0ee60f323e890484628f351bf50fa9a15d. > Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk> Queued for -next with a note about the regression added, thanks for the review. -Daniel -- Daniel Vetter Mail: daniel at ffwll.ch Mobile: +41 (0)79 365 57 48