Re: [Intel-gfx] [PATCH 2/2] drm/i915: Flush request queue when waiting for ring space

2014-05-07 Thread Volkin, Bradley D
On Mon, May 05, 2014 at 01:07:33AM -0700, Chris Wilson wrote: > During the review of > > commit 1f70999f9052f5a1b0ce1a55aff3808f2ec9fe42 > Author: Chris Wilson > Date: Mon Jan 27 22:43:07 2014 + > > drm/i915: Prevent recursion by retiring requests when the ring is full > > Ville raise

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Flush request queue when waiting for ring space

2014-05-05 Thread Chris Wilson
On Mon, May 05, 2014 at 11:49:12AM -0700, Ben Widawsky wrote: > On Mon, May 05, 2014 at 09:07:33AM +0100, Chris Wilson wrote: > > During the review of > > > > commit 1f70999f9052f5a1b0ce1a55aff3808f2ec9fe42 > > Author: Chris Wilson > > Date: Mon Jan 27 22:43:07 2014 + > > > > drm/i915:

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Flush request queue when waiting for ring space

2014-05-05 Thread Ben Widawsky
On Mon, May 05, 2014 at 09:07:33AM +0100, Chris Wilson wrote: > During the review of > > commit 1f70999f9052f5a1b0ce1a55aff3808f2ec9fe42 > Author: Chris Wilson > Date: Mon Jan 27 22:43:07 2014 + > > drm/i915: Prevent recursion by retiring requests when the ring is full > > Ville raise

[Intel-gfx] [PATCH 2/2] drm/i915: Flush request queue when waiting for ring space

2014-05-05 Thread Chris Wilson
During the review of commit 1f70999f9052f5a1b0ce1a55aff3808f2ec9fe42 Author: Chris Wilson Date: Mon Jan 27 22:43:07 2014 + drm/i915: Prevent recursion by retiring requests when the ring is full Ville raised the point that our interaction with request->tail was likely to foul up other