]> git.baikalelectronics.ru Git - kernel.git/commit
perf_event: Optimize perf_output_lock()
authorPeter Zijlstra <a.p.zijlstra@chello.nl>
Mon, 16 Nov 2009 11:45:14 +0000 (12:45 +0100)
committerIngo Molnar <mingo@elte.hu>
Mon, 16 Nov 2009 12:27:45 +0000 (13:27 +0100)
commitbc8a2e7598e717f1a1c9e936ed06d936b4c22e0b
treeb722805dbebd9584ffb981e2be84385ee0e22c67
parent43eb9d815c78308bd14a876006a87a29a55168f8
perf_event: Optimize perf_output_lock()

The purpose of perf_output_{un,}lock() is to:

 1) avoid publishing incomplete data
    [ possible when publishing a head that is ahead of an entry
      that is still being written ]

 2) guarantee fwd progress
    [ a simple refcount on pending writers doesn't need to drop to
      0, making it so would end up implementing something like forced
      quiecent states of RCU ]

To satisfy the above without undue complexity it serializes
between CPUs, this means that a pending writer can only be the
same cpu in a nested context, and since (under normal operation)
a cpu always makes progress we're good -- if the head is only
published when the bottom  most writer completes.

Now we don't need to disable IRQs in order to serialize between
CPUs, disabling preemption ought to be sufficient, esp since we
already deal with nesting due to NMIs.

This avoids potentially expensive (and needless) local IRQ
disable/enable ops.

Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
LKML-Reference: <1258373161.26714.254.camel@laptop>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
include/linux/perf_event.h
kernel/perf_event.c