<div dir="ltr">Thanks<div><br></div><div>Please just ignore this one for now. It will be removed on next round.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 3, 2014 at 5:38 PM, Ben Widawsky <span dir="ltr"><<a href="mailto:benjamin.widawsky@intel.com" target="_blank">benjamin.widawsky@intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thu, Jul 03, 2014 at 05:33:05PM -0400, Rodrigo Vivi wrote:<br>
> From: Ben Widawsky <<a href="mailto:benjamin.widawsky@intel.com">benjamin.widawsky@intel.com</a>><br>
><br>
> The PDPs seem to get screwed up otherwise, specifically PDP0. I am not<br>
> really clear why this is required, it just works with full PPGTT.<br>
><br>
> v2: Only do it for gen8, to limit regression potential<br>
><br>
> v3: Fix the bugzilla links<br>
><br>
> Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=78891" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=78891</a><br>
> Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=78935" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=78935</a><br>
> Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=78936" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=78936</a><br>
> Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=78937" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=78937</a><br>
> Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=78938" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=78938</a><br>
><br>
> Signed-off-by: Ben Widawsky <<a href="mailto:ben@bwidawsk.net">ben@bwidawsk.net</a>><br>
> Signed-off-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>><br>
> ---<br>
>  drivers/gpu/drm/i915/i915_gem_context.c | 3 +++<br>
>  1 file changed, 3 insertions(+)<br>
><br>
> diff --git a/drivers/gpu/drm/i915/i915_gem_context.c b/drivers/gpu/drm/i915/i915_gem_context.c<br>
> index 633e318..61b60b6 100644<br>
> --- a/drivers/gpu/drm/i915/i915_gem_context.c<br>
> +++ b/drivers/gpu/drm/i915/i915_gem_context.c<br>
> @@ -573,6 +573,9 @@ mi_set_context(struct intel_engine_cs *ring,<br>
>       else<br>
>               intel_ring_emit(ring, MI_NOOP);<br>
><br>
> +     if (INTEL_INFO(ring->dev)->gen == 8)<br>
> +             hw_flags |= MI_FORCE_RESTORE;<br>
> +<br>
>       intel_ring_emit(ring, MI_NOOP);<br>
>       intel_ring_emit(ring, MI_SET_CONTEXT);<br>
>       intel_ring_emit(ring,<br>
<br>
</div></div>Ville had a good point on this patch wrt to note setting both<br>
MI_FORCE_RESTORE, and MI_RESTORE_INHIBIT (though it seems to cause no<br>
problems).<br>
<br>
I think also with some of the do_switch() cleanups recently submitted,<br>
this one may no longer be necessary - not sure.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Ben Widawsky, Intel Open Source Technology Center<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Rodrigo Vivi</div><div>Blog: <a href="http://blog.vivi.eng.br" target="_blank">http://blog.vivi.eng.br</a></div><div> </div>
</div>