Re: [Intel-gfx] [PATCH 1/6] drm/i915: Reinitialize default context after reset

2014-10-07 Thread Mika Kuoppala
"Volkin, Bradley D" writes: Hi Bradley, > [snip] > > On Thu, Sep 18, 2014 at 07:58:30AM -0700, Mika Kuoppala wrote: >> @@ -577,7 +596,7 @@ static int do_switch(struct intel_engine_cs *ring, >> vma->bind_vma(vma, to->legacy_hw_ctx.rcs_state->cache_level, >> GLOBAL_BIND); >> } >

Re: [Intel-gfx] [PATCH 1/6] drm/i915: Reinitialize default context after reset

2014-09-19 Thread Volkin, Bradley D
[snip] On Thu, Sep 18, 2014 at 07:58:30AM -0700, Mika Kuoppala wrote: > @@ -577,7 +596,7 @@ static int do_switch(struct intel_engine_cs *ring, > vma->bind_vma(vma, to->legacy_hw_ctx.rcs_state->cache_level, > GLOBAL_BIND); > } > > - if (!to->legacy_hw_ctx.initialized || i

Re: [Intel-gfx] [PATCH 1/6] drm/i915: Reinitialize default context after reset

2014-09-19 Thread Daniel Vetter
On Thu, Sep 18, 2014 at 04:36:47PM +0100, Chris Wilson wrote: > On Thu, Sep 18, 2014 at 05:58:30PM +0300, Mika Kuoppala wrote: > > We don't know in what shape the default context was before reset. > > The reset also dropped our changes that were done in > > ring->init_context. > > > > Mark our def

Re: [Intel-gfx] [PATCH 1/6] drm/i915: Reinitialize default context after reset

2014-09-18 Thread Chris Wilson
On Thu, Sep 18, 2014 at 05:58:30PM +0300, Mika Kuoppala wrote: > We don't know in what shape the default context was before reset. > The reset also dropped our changes that were done in > ring->init_context. > > Mark our default context as uninitialized for it to be properly > setup up on reset re

[Intel-gfx] [PATCH 1/6] drm/i915: Reinitialize default context after reset

2014-09-18 Thread Mika Kuoppala
We don't know in what shape the default context was before reset. The reset also dropped our changes that were done in ring->init_context. Mark our default context as uninitialized for it to be properly setup up on reset recovery . Signed-off-by: Mika Kuoppala --- drivers/gpu/drm/i915/i915_debu