Re: [PATCH 7/7] drm/i915: expose energy counter on SNB and IVB

[Ben Widawsky <ben@bwidawsk.net>]

On Wed, Apr 10, 2013 at 03:27:05PM +0100, Chris Wilson wrote:
> On Fri, Sep 07, 2012 at 10:51:44AM -0700, Ben Widawsky wrote:
> > On Fri, 7 Sep 2012 12:03:37 +0200
> > Daniel Vetter <daniel@ffwll.ch> wrote:
> > 
> > > On Thu, Sep 06, 2012 at 01:54:10PM -0700, Ben Widawsky wrote:
> > > > From: Jesse Barnes <jbarnes@virtuousgeek.org>
> > > > 
> > > > On SNB and IVB, there's an MSR (also exposed through MCHBAR) we can use
> > > > to read out the amount of energy used over time.  Expose this in sysfs
> > > > to make it easy to do power comparisons with different configurations.
> > > > 
> > > > If the platform supports it, the file will show up under the
> > > > drm/card0/power subdirectory of the PCI device in sysfs as gt_energy_uJ.
> > > > The value in the file is a running total of energy (in microjoules)
> > > > consumed by the graphics device.
> > > > 
> > > > v2: move to sysfs (Ben, Daniel)
> > > >     expose a simple value (Chris)
> > > >     drop unrelated hunk (Ben)
> > > > 
> > > > Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
> > > > 
> > > > v3: by Ben
> > > > Tied it into existing rc6  sysfs entries and named that a more generic
> > > > "power attrs." Fixed rebase conflicts.
> > > > 
> > > > Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
> 
> A reminder that we never finished this patch.
> -Chris
> 
> -- 
> Chris Wilson, Intel Open Source Technology Centre

This will be supported in the RAPL driver, so I think we don't need it.
The latest version of that can be found:
https://lkml.org/lkml/2013/4/9/812

-- 
Ben Widawsky, Intel Open Source Technology Center