[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