On 03/08/17 18:07, Chris Wilson wrote:
Quoting Lionel Landwerlin (2017-08-03 17:58:07)We were reserving fewer dwords in the ring than necessary. Indeed we're always writing all registers once, so discard the actual number of registers given by the user and just program the whitelisted ones once. Fixes: 19f81df2859e ("drm/i915/perf: Add OA unit support for Gen 8+") Reported-by: Matthew Auld <matthew.william.auld@xxxxxxxxx> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx> Reviewed-by: Matthew Auld <matthew.auld@xxxxxxxxx> Cc: <stable@xxxxxxxxxxxxxxx> # v4.12+For reference, if you enable CONFIG_DRM_I915_DEBUG_GEM it should catch these mistakes (by bugging out, you are using netconsole, right?) -Chris
Thanks, I haven't actually tried it. Will do.