[libvirt] [PATCH v2 00/10] perf: Add software perf events

[Daniel P. Berrange]

On Mon, Mar 06, 2017 at 02:55:18PM -0500, John Ferlan wrote:
> 
> 
> On 02/23/2017 09:55 AM, Nitesh Konkar wrote:
> > This patch series adds software perf events.
> > The perl and go patches shall follow shortly.
> > 
> > Nitesh Konkar (10):
> >   perf: add cpu_clock software perf event support
> >   perf: add task_clock software perf event support
> >   perf: add page_faults software perf event support
> >   perf: add context_switches software perf event support
> >   perf: add cpu_migrations software perf event support
> >   perf: add page_faults_min software perf event support
> >   perf: add page_faults_maj software perf event support
> >   perf: add alignment_faults software perf event support
> >   perf: add emulation_faults software perf event support
> >   news: Update the news.xml about perf events added
> > 
> >  docs/formatdomain.html.in                   | 74 +++++++++++++++++++++++
> >  docs/news.xml                               | 11 ++++
> >  docs/schemas/domaincommon.rng               |  9 +++
> >  include/libvirt/libvirt-domain.h            | 92 +++++++++++++++++++++++++++++
> >  src/libvirt-domain.c                        | 25 ++++++++
> >  src/qemu/qemu_driver.c                      |  9 +++
> >  src/util/virperf.c                          | 33 ++++++++++-
> >  src/util/virperf.h                          |  9 +++
> >  tests/genericxml2xmlindata/generic-perf.xml |  9 +++
> >  tools/virsh.pod                             | 27 +++++++++
> >  10 files changed, 297 insertions(+), 1 deletion(-)
> > 
> 
> ACK series - altered a couple of inconsistencies w/ periods in virsh.pod
> and altered the news.xml to move the note into the 3.2 release.
> 
> Going to wait to push this though until you provide an acceptable GO patch.

BTW, while I very much appreciate if contributors provide a Go patch to
go along with C API additions, you don't need to consider it a blocker
for merging libvirt APIs. I'll simply write the Go patch myself if the
contributor doesn't/can't provide one prior to merge. Likewise for Perl.

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://entangle-photo.org       -o-    http://search.cpan.org/~danberr/ :|