Re: [PATCH] drm/i915: Only grab correct forcewake for the engine with execlists

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 06, 2016 at 03:49:55PM +0100, Tvrtko Ursulin wrote:
> +static const i915_reg_t gen9_shadowed_regs[] = {
> +	RING_TAIL(RENDER_RING_BASE),
> +	RING_TAIL(GEN6_BSD_RING_BASE),
> +	RING_TAIL(VEBOX_RING_BASE),
> +	RING_TAIL(BLT_RING_BASE),

It occurs to me that we will never use these in anger and so can drop
from the search table.

> +	FORCEWAKE_BLITTER_GEN9,
> +	FORCEWAKE_RENDER_GEN9,
> +	FORCEWAKE_MEDIA_GEN9,

These always use raw accessors (internal to intel_uncore.c).

> +	GEN6_RPNSWREQ,

Always written in conjunction with other registers notable by their
absence from this table

> +	GEN6_RC_VIDEO_FREQ,

Used once per load/resume/reset, whoop-a-do.

> +	/* TODO: Other registers are not yet used */

I hope there are more magic registers that we actually use.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux