lkml.org 
[lkml]   [2014]   [Sep]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subject[RFC 00/14] perf pollfd v3
Date
From: Arnaldo Carvalho de Melo <acme@redhat.com>

Hi,

Main different to what was discussed in v2[1]:

Poll all event file descriptors, not just the ones mmaped.

This is a bug that was present before, noticed by Adrian Hunter while
reviewing this patchkit, i.e. we were polling just the mmaped ones, because
then we would look at all the ring buffers when just one of the polled
descriptors was ready to read.

But this could lead to delays in polling when one of the mmaped fds was
closed and we would then not be notified when another, associated to it by
means of PERF_EVENT_IOC_SET_OUTPUT, had events available.

Also refcount the mmaps associated by means of PERF_EVENT_IOC_SET_OUTPUT,
so that we can properly unmap a ring buffer to which all its file descriptors
had been closed, for longer running apps.

It is all available on my tree:

git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git perf/fdarray.v2

I kept the v3 for the series as I initially called it pollfd, but since the
class that grew out of it was named fdarray, thus the fdarray.v2 branch name.

Please let me know if there are still any other problems, now back to
fixing up my last perf/core pull req wrt those external proggies problems.

- Arnaldo

[1] http://lkml.kernel.org/r/1409781604-16778-1-git-send-email-acme@kernel.org

Arnaldo Carvalho de Melo (14):
perf evlist: Introduce perf_evlist__filter_pollfd method
perf tests: Add test for perf_evlist__filter_pollfd()
perf evlist: Monitor POLLERR and POLLHUP events too
perf evlist: We need to poll all event file descriptors
perf record: Filter out POLLHUP'ed file descriptors
perf trace: Filter out POLLHUP'ed file descriptors
perf evlist: Allow growing pollfd on add method
perf tests: Add pollfd growing test
perf kvm stat live: Use perf_evlist__add_pollfd() instead of local equivalent
perf evlist: Introduce poll method for common code idiom
tools lib api: Adopt fdarray class from perf's evlist
perf evlist: Refcount mmaps
tools lib fd array: Allow associating an integer cookie with each entry
perf evlist: Unmap ring buffer when fd is nuked

tools/lib/api/Makefile | 7 +-
tools/lib/api/fd/array.c | 128 ++++++++++++++++++
tools/lib/api/fd/array.h | 34 +++++
tools/perf/Makefile.perf | 1 +
tools/perf/builtin-kvm.c | 24 ++--
tools/perf/builtin-record.c | 5 +-
tools/perf/builtin-top.c | 4 +-
tools/perf/builtin-trace.c | 3 +-
tools/perf/tests/builtin-test.c | 8 ++
tools/perf/tests/evlist.c | 217 ++++++++++++++++++++++++++++++
tools/perf/tests/open-syscall-tp-fields.c | 2 +-
tools/perf/tests/perf-record.c | 2 +-
tools/perf/tests/task-exit.c | 2 +-
tools/perf/tests/tests.h | 2 +
tools/perf/util/evlist.c | 77 +++++++++--
tools/perf/util/evlist.h | 16 ++-
tools/perf/util/python.c | 6 +-
17 files changed, 499 insertions(+), 39 deletions(-)
create mode 100644 tools/lib/api/fd/array.c
create mode 100644 tools/lib/api/fd/array.h
create mode 100644 tools/perf/tests/evlist.c

--
1.9.3



\
 
 \ /
  Last update: 2014-09-10 16:41    [W:0.180 / U:0.336 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site