Re: [Intel-gfx] [PATCH 10/10] drm/i915: Use MI_BATCH_BUFFER_START on 830/845

2015-12-15 Thread Ville Syrjälä
On Tue, Dec 15, 2015 at 11:22:29AM +, Chris Wilson wrote: > On Tue, Dec 15, 2015 at 01:05:56PM +0200, Ville Syrjälä wrote: > > On Tue, Dec 15, 2015 at 10:24:13AM +, Chris Wilson wrote: > > > The other question, does this obsolete our work around? Can I be that > > > optimisitic? > > > > Th

Re: [Intel-gfx] [PATCH 10/10] drm/i915: Use MI_BATCH_BUFFER_START on 830/845

2015-12-15 Thread Chris Wilson
On Tue, Dec 15, 2015 at 01:05:56PM +0200, Ville Syrjälä wrote: > On Tue, Dec 15, 2015 at 10:24:13AM +, Chris Wilson wrote: > > The other question, does this obsolete our work around? Can I be that > > optimisitic? > > The CS TLB one? I think I tried it at some point, and things till > failed.

Re: [Intel-gfx] [PATCH 10/10] drm/i915: Use MI_BATCH_BUFFER_START on 830/845

2015-12-15 Thread Ville Syrjälä
On Tue, Dec 15, 2015 at 10:24:13AM +, Chris Wilson wrote: > On Tue, Dec 15, 2015 at 10:09:30AM +, Chris Wilson wrote: > > On Mon, Dec 14, 2015 at 07:25:13PM +0200, Ville Syrjälä wrote: > > > On Mon, Dec 14, 2015 at 04:58:54PM +, Chris Wilson wrote: > > > > On Mon, Dec 14, 2015 at 06:23:

Re: [Intel-gfx] [PATCH 10/10] drm/i915: Use MI_BATCH_BUFFER_START on 830/845

2015-12-15 Thread Chris Wilson
On Tue, Dec 15, 2015 at 10:09:30AM +, Chris Wilson wrote: > On Mon, Dec 14, 2015 at 07:25:13PM +0200, Ville Syrjälä wrote: > > On Mon, Dec 14, 2015 at 04:58:54PM +, Chris Wilson wrote: > > > On Mon, Dec 14, 2015 at 06:23:49PM +0200, ville.syrj...@linux.intel.com > > > wrote: > > > > From:

Re: [Intel-gfx] [PATCH 10/10] drm/i915: Use MI_BATCH_BUFFER_START on 830/845

2015-12-15 Thread Chris Wilson
On Mon, Dec 14, 2015 at 07:25:13PM +0200, Ville Syrjälä wrote: > On Mon, Dec 14, 2015 at 04:58:54PM +, Chris Wilson wrote: > > On Mon, Dec 14, 2015 at 06:23:49PM +0200, ville.syrj...@linux.intel.com > > wrote: > > > From: Ville Syrjälä > > > > > > MI_BATCH_BUFFER is nasty since it requires t

Re: [Intel-gfx] [PATCH 10/10] drm/i915: Use MI_BATCH_BUFFER_START on 830/845

2015-12-14 Thread Ville Syrjälä
On Mon, Dec 14, 2015 at 04:58:54PM +, Chris Wilson wrote: > On Mon, Dec 14, 2015 at 06:23:49PM +0200, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > MI_BATCH_BUFFER is nasty since it requires that userspace pass in the > > correct batch length. > > > > Let's switch to u

Re: [Intel-gfx] [PATCH 10/10] drm/i915: Use MI_BATCH_BUFFER_START on 830/845

2015-12-14 Thread Chris Wilson
On Mon, Dec 14, 2015 at 06:23:49PM +0200, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > MI_BATCH_BUFFER is nasty since it requires that userspace pass in the > correct batch length. > > Let's switch to using MI_BATCH_BUFFER_START instead (like we do on > other platforms). Then w

[Intel-gfx] [PATCH 10/10] drm/i915: Use MI_BATCH_BUFFER_START on 830/845

2015-12-14 Thread ville . syrjala
From: Ville Syrjälä MI_BATCH_BUFFER is nasty since it requires that userspace pass in the correct batch length. Let's switch to using MI_BATCH_BUFFER_START instead (like we do on other platforms). Then we don't have to specify the batch length at all, and the CS will instead execute until it see