Re: [Intel-gfx] [PATCH] drm/i915: set ctx->initialized only after RCS

2014-01-09 Thread Daniel Vetter
On Thu, Jan 09, 2014 at 03:17:25PM -0800, Ben Widawsky wrote: > On Tue, Jan 07, 2014 at 10:47:18AM +, Chris Wilson wrote: > > On Tue, Jan 07, 2014 at 08:10:33AM +0100, Daniel Vetter wrote: > > > On Sat, Dec 28, 2013 at 01:31:49PM -0800, Ben Widawsky wrote: > > > > The initialized flag is used t

Re: [Intel-gfx] [PATCH] drm/i915: set ctx->initialized only after RCS

2014-01-09 Thread Ben Widawsky
On Tue, Jan 07, 2014 at 10:47:18AM +, Chris Wilson wrote: > On Tue, Jan 07, 2014 at 08:10:33AM +0100, Daniel Vetter wrote: > > On Sat, Dec 28, 2013 at 01:31:49PM -0800, Ben Widawsky wrote: > > > The initialized flag is used to specify a context has been initialized > > > and it's context is saf

Re: [Intel-gfx] [PATCH] drm/i915: set ctx->initialized only after RCS

2014-01-07 Thread Chris Wilson
On Tue, Jan 07, 2014 at 08:10:33AM +0100, Daniel Vetter wrote: > On Sat, Dec 28, 2013 at 01:31:49PM -0800, Ben Widawsky wrote: > > The initialized flag is used to specify a context has been initialized > > and it's context is safe to load, ie. the 3d state is setup properly. > > With full PPGTT, we

Re: [Intel-gfx] [PATCH] drm/i915: set ctx->initialized only after RCS

2014-01-06 Thread Daniel Vetter
On Sat, Dec 28, 2013 at 01:31:49PM -0800, Ben Widawsky wrote: > The initialized flag is used to specify a context has been initialized > and it's context is safe to load, ie. the 3d state is setup properly. > With full PPGTT, we emit the address space loads during context switch > and this currentl

Re: [Intel-gfx] [PATCH] drm/i915: set ctx->initialized only after RCS

2014-01-01 Thread Ben Widawsky
On Tue, Dec 31, 2013 at 11:26:17AM +, Chris Wilson wrote: > On Mon, Dec 30, 2013 at 01:34:46PM -0800, Ben Widawsky wrote: > > On Sun, Dec 29, 2013 at 09:59:26AM +, Chris Wilson wrote: > > > On Sat, Dec 28, 2013 at 01:31:49PM -0800, Ben Widawsky wrote: > > > > The initialized flag is used to

Re: [Intel-gfx] [PATCH] drm/i915: set ctx->initialized only after RCS

2013-12-31 Thread Chris Wilson
On Mon, Dec 30, 2013 at 01:34:46PM -0800, Ben Widawsky wrote: > On Sun, Dec 29, 2013 at 09:59:26AM +, Chris Wilson wrote: > > On Sat, Dec 28, 2013 at 01:31:49PM -0800, Ben Widawsky wrote: > > > The initialized flag is used to specify a context has been initialized > > > and it's context is safe

Re: [Intel-gfx] [PATCH] drm/i915: set ctx->initialized only after RCS

2013-12-30 Thread Ben Widawsky
On Sun, Dec 29, 2013 at 09:59:26AM +, Chris Wilson wrote: > On Sat, Dec 28, 2013 at 01:31:49PM -0800, Ben Widawsky wrote: > > The initialized flag is used to specify a context has been initialized > > and it's context is safe to load, ie. the 3d state is setup properly. > > With full PPGTT, we

Re: [Intel-gfx] [PATCH] drm/i915: set ctx->initialized only after RCS

2013-12-29 Thread Chris Wilson
On Sat, Dec 28, 2013 at 01:31:49PM -0800, Ben Widawsky wrote: > The initialized flag is used to specify a context has been initialized > and it's context is safe to load, ie. the 3d state is setup properly. > With full PPGTT, we emit the address space loads during context switch > and this currentl

[Intel-gfx] [PATCH] drm/i915: set ctx->initialized only after RCS

2013-12-28 Thread Ben Widawsky
The initialized flag is used to specify a context has been initialized and it's context is safe to load, ie. the 3d state is setup properly. With full PPGTT, we emit the address space loads during context switch and this currently marks a context as initialized. With full PPGTT patches, if a client