On 07/11/2013 11:52 AM, Ben Widawsky wrote: > The algorithm/information was originally written by Chad, though I > changed the control flow, and I think his original code had a couple of > bugs, though I didn't look very hard before rewriting. That could have > also been different interpretations of the spec. > > I've tested this on two platforms, and it seems to perform how I want. > > With this patch is a small tool for igt to query the size. This can be > used as a reference for DRI clients wishing to query the information. > > v2: Update name of the SDM location (Bryan) > Dissent: Use a new param instead of reusing HAS_LLC param (Chris, Chad) > Fix unicode multiply symbol (Ben) > > CC: Chad Versace <chad.versace at linux.intel.com> > CC: Bryan Bell <bryan.j.bell at intel.com> > Signed-off-by: Ben Widawsky <ben at bwidawsk.net> > --- > drivers/gpu/drm/i915/i915_dma.c | 3 +++ > drivers/gpu/drm/i915/i915_drv.h | 2 ++ > drivers/gpu/drm/i915/i915_gem.c | 55 +++++++++++++++++++++++++++++++++++++++++ > include/uapi/drm/i915_drm.h | 1 + > 4 files changed, 61 insertions(+) As long as this is tested and returns the same as L3 in /proc/cpuinfo, this patch is Reviewed-by: Chad Versace <chad.versace at linux.intel.com>