<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 25, 2017 at 9:55 PM, John Ferlan <span dir="ltr"><<a href="mailto:jferlan@redhat.com" target="_blank">jferlan@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
On 01/18/2017 11:58 PM, Nitesh Konkar wrote:<br>
> This patch series adds support and documentation for<br>
> a generalized hardware cache event called CACHE_L1D<br>
> perf event. This perf event is split into cache_l1dra,<br>
> cache_l1drm, cache_l1dwa, cache_l1dwm, cache_l1dpa and<br>
> cache_l1dpm perf events depending on the config value<br>
> set.<br>
><br>
> Nitesh Konkar (7):<br>
>   perf: add cache_l1dra perf event support<br>
>   perf: add cache_l1drm perf event support<br>
>   perf: add cache_l1dwa perf event support<br>
>   perf: add cache_l1dwm perf event support<br>
>   perf: add cache_l1dpa perf event support<br>
>   perf: add cache_l1dpm perf event support<br>
>   virsh: Fix manpage typo<br>
><br>
>  docs/<a href="http://formatdomain.html.in" rel="noreferrer" target="_blank">formatdomain.html.in</a>                   | 42 ++++++++++++++++++<br>
>  docs/news.xml                               |  6 ++-<br>
>  docs/schemas/domaincommon.rng               |  6 +++<br>
>  include/libvirt/libvirt-<wbr>domain.h            | 66 +++++++++++++++++++++++++++++<br>
>  src/libvirt-domain.c                        | 18 ++++++++<br>
>  src/qemu/qemu_driver.c                      |  6 +++<br>
>  src/remote/remote_protocol.x                |  2 +-<br>
>  src/util/virperf.c                          | 34 ++++++++++++++-<br>
>  src/util/virperf.h                          |  6 +++<br>
>  tests/genericxml2xmlindata/<wbr>generic-perf.xml |  6 +++<br>
>  tools/virsh.pod                             | 26 ++++++++++--<br>
>  11 files changed, 210 insertions(+), 8 deletions(-)<br>
><br>
<br>
</div></div>Is there any possibility for combining the first 6 patches in some<br>
manner? They're all just some "feature" within the level 1 cache<br>
statistics.  I'm thinking of syntax such as:<br>
<br>
virsh perf $dom --enable cache_1d=$LIST where $LIST is a comma separated<br>
list and has the possibility for ra, rm, wa, wm, pa, and pm.<br>
<br>
I didn't put much thought to it API wise and whether there's any<br>
other/similar example. It's just something that it would seem would be<br>
useful rather than the repetitiveness of these patches.<br></blockquote><div>I agree. Will work on that. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Also, lots of extra work within the *perl and *go bindings that at this<br>
point to add all the new definitions. I think you should supply the<br>
patches for those rather than relying on myself and/or Daniel to do<br>
that. Just follow the countless previous examples!<br></blockquote><div>Sure. I shall provide it the next time.  <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
NB: A couple of patches require an extra space between the "*" and<br>
description within include/libvirt/libvirt-<wbr>domain.h.<br></blockquote><div>Noted.  <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'll handle patch 7 separately...<br></blockquote><div>Sure.<br></div><div>I shall also send a patch series to add perf software events, just like we <br></div><div>have generic hardware events now.  Meanwhile will work on the above<br></div><div>suggestion for hardware cache events. <br></div><div>Thanks,<br></div><div>Nitesh.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
<br>
John<br>
</font></span></blockquote></div><br></div></div>