<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 30, 2015 at 9:30 AM, Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk" target="_blank">chris@chris-wilson.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Tue, Sep 29, 2015 at 03:39:03PM +0100, Robert Bragg wrote:<br>
> Updating Mesa and GPU Top to experiment with this was straightforward<br>
> given the similarity to the perf interface.  The main difference is that<br>
> it only supports forwarding metrics via read()s instead of an mmaped<br>
> circular buffer. As mentioned above, I think that suits this well, and<br>
> requires no additional copying of data. I think the userspace code has<br>
> ended up being a little simpler too.<br>
<br>
</span>Did you try updating the existing perf based overlay?<br></blockquote><div><br></div><div>I don't recall the overlay attempting to read OA counters, but potentially it could be quite nice to add support - sorry I hadn't considered that so far.<br><br></div><div>I don't believe being perf based or not will affect the effort to do this though. The perf based driver doesn't handle OA counter normalization in the kernel so userspace needs to be able to handle that - which is probably the bigger effort.<br><br></div><div>Something to note here about your early pmu driver, is that it was notably for counters that were explicitly sampled from the cpu using a hrtimer via mmio. I think they were a better fit for the existing perf design than the OA unit, primarily because they were explicitly read from the cpu and each counter was very independent.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class=""><br>
> Overall the driver currently isn't much more code than with perf (~200<br>
> lines).<br>
><br>
> Personally my gut feeling a.t.m, is that we should aim to move forward<br>
> independent from perf.<br>
><br>
> I'd really appreciate some feedback from others on this though.<br>
><br>
> Daniel and Chris; although I think it made sense at the outset to try<br>
> and use perf, in light of the above would you be open to a non-perf<br>
> based driver for the OA unit?<br>
<br>
</span>No. I strongly dislike that they will be multiple incompatibile perf<br>
interfaces and strongly like the coupling with other profiling that<br>
comes with perf - i.e. we very much want to simultaneously sample CPU<br>
and GPU workloads along with other devices, that information is much<br>
more useful to me for the purposes of scheduling work and maximising<br>
concurrency than optimising shaders.<br></blockquote><div><br>In this case I don't think there's inherently any more compatibility that comes from using perf or not - no existing userspace will Just Work™ with the perf based OA driver.<br><br></div><div>I think some of the cases you're referring to may be ok to expose via the existing perf infrastructure, but I'm currently enabling the OA unit which poses some unique difficulties I've tried to explain.<br><br></div><div>A guiding differentiator may be whether or not the counter is orthogonal (in terms of configuration and normalization) and explicitly readable from the cpu, as to whether the existing perf pmu infrastructure is a good fit.<br><br></div><div>'i915 perf' shows my lack of imagination naming this and maybe another  name could imply a more limited scope. I.e. on a case by case basis, when looking to expose a new counters we can still evaluate whether it makes sense to expose via the existing perf infrastructure or this.<br><br></div><div>- Robert<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class=""><font color="#888888">-Chris<br>
<br>
--<br>
Chris Wilson, Intel Open Source Technology Centre<br>
--<br>
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in<br>
the body of a message to <a href="mailto:majordomo@vger.kernel.org">majordomo@vger.kernel.org</a><br>
More majordomo info at  <a href="http://vger.kernel.org/majordomo-info.html" rel="noreferrer" target="_blank">http://vger.kernel.org/majordomo-info.html</a><br>
Please read the FAQ at  <a href="http://www.tux.org/lkml/" rel="noreferrer" target="_blank">http://www.tux.org/lkml/</a><br>
</font></span></blockquote></div><br></div></div>