On Wed, Jan 27, 2016 at 01:41:09PM +0000, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > This got broken in: > > commit de1add360522c876c25ef2bbbbab1c94bdb509ab > Author: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > Date: Fri Jan 15 15:12:50 2016 +0000 > > drm/i915: Decouple execbuf uAPI from internal implementation > > BSD ring flags need to be shifted before they can be considered > indices into the ring array. > > Reported by Zhipeng Gong. > > v2: Simplify the code. (Chris Wilson) Testcase: igt/gem_exec_basic # bdw-gt3 > Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Cc: Zhipeng Gong <zhipeng.gong@xxxxxxxxx> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> There were a few tests that would have hit this, but nothing existed to only try and touch the ring so I added gem_exec_basic to exercise the ring selection. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
