public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Robert Richter <rric@kernel.org>
To: Borislav Petkov <bp@alien8.de>
Cc: Jean Pihet <jean.pihet@linaro.org>,
	Peter Zijlstra <peterz@infradead.org>,
	Ingo Molnar <mingo@kernel.org>,
	Arnaldo Carvalho de Melo <acme@infradead.org>,
	Jiri Olsa <jolsa@redhat.com>,
	linux-kernel@vger.kernel.org,
	Tomasz Nowicki <tomasz.nowicki@linaro.org>
Subject: Re: [PATCH v4 00/16] perf, persistent: Add persistent events
Date: Fri, 9 May 2014 11:17:53 +0200	[thread overview]
Message-ID: <20140509091753.GT32718@rric.localhost> (raw)
In-Reply-To: <20140508182344.GG12548@pd.tnic>

On 08.05.14 20:23:44, Borislav Petkov wrote:
> On Wed, May 07, 2014 at 06:44:07PM +0200, Robert Richter wrote:
> > With transparently I mean that the process even does not know that the
> > same event is already running by another process. The kernel detects
> > this and maps the request to that event and buffer. Of course the
> > event's buffer must be at least readonly to be shared for this.
> 
> Yes.
> 
> > This could be a mechanism to connect to persistent events. The kernel
> > detects by type and attr that the requested event is already running
> > persistent and maps to it.
> > 
> > But at the moment persistent events can only be shared using
> > 
> >  attr.type = PERF_TYPE_PERSISTENT
> >  attr.config = id
> > 
> > So the above is more an alternative to connect to persistent events
> > and the question is, which one to use. Presumable the easiest first,
> > which is the current implementation.
> 
> Well, there is no trivial way to share event buffers if they're not
> read-only AFAICT.
> 
> But in questions like this, we always have to step one step back and ask
> ourselves: what are the use cases for shared events and after we have
> enumerated them, to design the kernel side so that it supports them.
> 
> So, do we want anything else besides shared, read-only events?

I only talk about events that are sharable since they are read-only.

The question I am asking here is how to connect to an event that is
sharable. This could be done transparently.

-Robert

  reply	other threads:[~2014-05-09  9:18 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-07 15:04 [PATCH v4 00/16] perf, persistent: Add persistent events Jean Pihet
2014-04-07 15:04 ` [PATCH 01/16] perf, mmap: Factor out ring_buffer_detach_all() Jean Pihet
2014-04-07 15:04 ` [PATCH 02/16] perf, mmap: Factor out try_get_event()/put_event() Jean Pihet
2014-04-07 15:04 ` [PATCH 03/16] perf, mmap: Factor out perf_alloc/free_rb() Jean Pihet
2014-04-22 14:25   ` Peter Zijlstra
2014-04-07 15:04 ` [PATCH 04/16] perf, mmap: Factor out perf_get_fd() Jean Pihet
2014-04-22 14:27   ` Peter Zijlstra
2014-04-25 13:54     ` Robert Richter
2014-04-25 14:43       ` Peter Zijlstra
2014-04-25 14:52         ` Peter Zijlstra
2014-04-29 15:34           ` Robert Richter
2014-04-07 15:04 ` [PATCH 05/16] perf: Add persistent events Jean Pihet
2014-04-07 15:04 ` [PATCH 06/16] mce, x86: Enable " Jean Pihet
2014-04-07 15:04 ` [PATCH 07/16] perf, persistent: Implementing a persistent pmu Jean Pihet
2014-04-07 15:04 ` [PATCH 08/16] perf, persistent: Exposing persistent events using sysfs Jean Pihet
2014-04-07 15:04 ` [PATCH 09/16] perf, persistent: Use unique event ids Jean Pihet
2014-04-07 15:04 ` [PATCH 10/16] perf, persistent: Implement reference counter for events Jean Pihet
2014-04-07 15:04 ` [PATCH 11/16] perf, persistent: Dynamically resize list of sysfs entries Jean Pihet
2014-04-07 15:04 ` [PATCH 12/16] perf, persistent: ioctl functions to control persistency Jean Pihet
2014-04-07 15:04 ` [PATCH 13/16] perf tools: Rename flex conditions to avoid name conflicts Jean Pihet
2014-04-07 15:04 ` [PATCH 14/16] perf tools: Modify event parser to update event attribute by index Jean Pihet
2014-04-07 15:04 ` [PATCH 15/16] perf tools: Add attr<num> syntax to event parser Jean Pihet
2014-04-07 15:04 ` [PATCH 16/16] perf tools: Retry mapping buffers readonly on EACCES Jean Pihet
2014-04-17 12:44 ` [PATCH v4 00/16] perf, persistent: Add persistent events Jean Pihet
2014-04-17 12:50   ` Borislav Petkov
2014-04-17 13:17     ` Jean Pihet
2014-04-17 13:21       ` Borislav Petkov
2014-04-22  8:20         ` Jean Pihet
2014-04-22 10:07           ` Jean Pihet
2014-04-22 10:20             ` Borislav Petkov
2014-05-06 12:39 ` Robert Richter
2014-05-06 18:50   ` Borislav Petkov
2014-05-06 18:53   ` Borislav Petkov
2014-05-07 16:44     ` Robert Richter
2014-05-08 18:23       ` Borislav Petkov
2014-05-09  9:17         ` Robert Richter [this message]
2014-05-06 18:58   ` Borislav Petkov
2014-05-07 17:01     ` Robert Richter
2014-05-08 18:36       ` Borislav Petkov
2014-05-09  8:52         ` Robert Richter
2014-05-09 10:17       ` Borislav Petkov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140509091753.GT32718@rric.localhost \
    --to=rric@kernel.org \
    --cc=acme@infradead.org \
    --cc=bp@alien8.de \
    --cc=jean.pihet@linaro.org \
    --cc=jolsa@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@kernel.org \
    --cc=peterz@infradead.org \
    --cc=tomasz.nowicki@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox