[Intel-gfx] [PATCH 1/4] drm/i915 Update Gen8 golden context batch buffer

Damien Lespiau damien.lespiau at intel.com
Thu Oct 23 18:40:47 CEST 2014


On Thu, Oct 23, 2014 at 08:34:27AM -0700, armin.c.reese at intel.com wrote:
> From: Armin Reese <armin.c.reese at intel.com>
> 
> The file drivers/gpu/drm/i915/intel_renderstate_gen8.c is
> updated to the version created by IGT null_state_gen
> 
> Signed-off-by: Armin Reese <armin.c.reese at intel.com>

For the first 3 patches:

Reviewed-by: Damien Lespiau <damien.lespiau at intel.com>

For the last patch, I'm really not convinced about reusing
i915_context_status(), I'd re-use i915_dump_lrc() instead. The reason is
that _status() seems to be about having a short summary of all context
images around, while dump_lrc() dumps content (only one page for now,
but that could be changed).

Also why the drm_clflush_pages()? CPU shouldn't access those pages past
the initial init.

-- 
Damien




More information about the Intel-gfx mailing list