On Tue, 14 Feb 2012 22:09:09 +0100, Ben Widawsky <b...@bwidawsk.net> wrote:
> This is needed for an upcoming workaround.
>
> Signed-off-by: Ben Widawsky <b...@bwidawsk.net>
> ---
> drivers/gpu/drm/i915/intel_ringbuffer.c | 4 ++++
> drivers/gpu/drm/i915/intel_ringbuffer.h | 5 +++++
> 2 files changed, 9 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c
> b/drivers/gpu/drm/i915/intel_ringbuffer.c
> index 4956f1b..411a0e5 100644
> --- a/drivers/gpu/drm/i915/intel_ringbuffer.c
> +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c
> @@ -412,6 +412,10 @@ static int init_render_ring(struct intel_ring_buffer
> *ring)
> return ret;
> }
>
> + if (INTEL_INFO(dev)->gen == 6)
> + ring->itlb_before_ctx_switch =
> + !!(I915_READ(GFX_MODE) & GFX_TLB_INVALIDATE_ALWAYS);
> +
> if (INTEL_INFO(dev)->gen >= 6) {
> I915_WRITE(INSTPM,
> INSTPM_FORCE_ORDERING << 16 | INSTPM_FORCE_ORDERING);
> diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.h
> b/drivers/gpu/drm/i915/intel_ringbuffer.h
> index c8b9cc0..fad4251 100644
> --- a/drivers/gpu/drm/i915/intel_ringbuffer.h
> +++ b/drivers/gpu/drm/i915/intel_ringbuffer.h
> @@ -110,6 +110,11 @@ struct intel_ring_buffer {
> wait_queue_head_t irq_queue;
> drm_local_map_t map;
>
> + /**
> + * Do an explicit TLB flush before MI_SET_CONTEXT
> + */
> + bool itlb_before_ctx_switch;
> +
> void *private;
> };I really wanted to see the consumer of this when trying to review this code. They weren't logically separate -- this is to implement a quoted piece of spec in the later commit. (What I was trying to figure out was whether this was needed for gen7. It looks like it is not required).
pgpNVqwVafIiT.pgp
Description: PGP signature
_______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
