diff options
author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2010-06-11 11:32:03 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-09-09 14:46:29 -0400 |
commit | 24cd7f54a0d47e1d5b3de29e2456bfbd2d8447b7 (patch) | |
tree | a37d3a4cb101e3f67635a1920f447c9e9e8d8ab2 /include | |
parent | 9ed6060d286b1eb55974d09080f442f809408c42 (diff) |
perf: Reduce perf_disable() usage
Since the current perf_disable() usage is only an optimization,
remove it for now. This eases the removal of the __weak
hw_perf_enable() interface.
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: paulus <paulus@samba.org>
Cc: stephane eranian <eranian@googlemail.com>
Cc: Robert Richter <robert.richter@amd.com>
Cc: Will Deacon <will.deacon@arm.com>
Cc: Paul Mundt <lethal@linux-sh.org>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Cyrill Gorcunov <gorcunov@gmail.com>
Cc: Lin Ming <ming.m.lin@intel.com>
Cc: Yanmin <yanmin_zhang@linux.intel.com>
Cc: Deng-Cheng Zhu <dengcheng.zhu@gmail.com>
Cc: David Miller <davem@davemloft.net>
Cc: Michael Cree <mcree@orcon.net.nz>
LKML-Reference: <new-submission>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/perf_event.h | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h index ab72f56eb372..243286a8ded7 100644 --- a/include/linux/perf_event.h +++ b/include/linux/perf_event.h | |||
@@ -564,26 +564,26 @@ struct pmu { | |||
564 | struct list_head entry; | 564 | struct list_head entry; |
565 | 565 | ||
566 | /* | 566 | /* |
567 | * Should return -ENOENT when the @event doesn't match this pmu | 567 | * Should return -ENOENT when the @event doesn't match this PMU. |
568 | */ | 568 | */ |
569 | int (*event_init) (struct perf_event *event); | 569 | int (*event_init) (struct perf_event *event); |
570 | 570 | ||
571 | int (*enable) (struct perf_event *event); | 571 | int (*enable) (struct perf_event *event); |
572 | void (*disable) (struct perf_event *event); | 572 | void (*disable) (struct perf_event *event); |
573 | int (*start) (struct perf_event *event); | 573 | int (*start) (struct perf_event *event); |
574 | void (*stop) (struct perf_event *event); | 574 | void (*stop) (struct perf_event *event); |
575 | void (*read) (struct perf_event *event); | 575 | void (*read) (struct perf_event *event); |
576 | void (*unthrottle) (struct perf_event *event); | 576 | void (*unthrottle) (struct perf_event *event); |
577 | 577 | ||
578 | /* | 578 | /* |
579 | * Group events scheduling is treated as a transaction, add group | 579 | * Group events scheduling is treated as a transaction, add |
580 | * events as a whole and perform one schedulability test. If the test | 580 | * group events as a whole and perform one schedulability test. |
581 | * fails, roll back the whole group | 581 | * If the test fails, roll back the whole group |
582 | */ | 582 | */ |
583 | 583 | ||
584 | /* | 584 | /* |
585 | * Start the transaction, after this ->enable() doesn't need | 585 | * Start the transaction, after this ->enable() doesn't need to |
586 | * to do schedulability tests. | 586 | * do schedulability tests. |
587 | */ | 587 | */ |
588 | void (*start_txn) (struct pmu *pmu); | 588 | void (*start_txn) (struct pmu *pmu); |
589 | /* | 589 | /* |
@@ -594,8 +594,8 @@ struct pmu { | |||
594 | */ | 594 | */ |
595 | int (*commit_txn) (struct pmu *pmu); | 595 | int (*commit_txn) (struct pmu *pmu); |
596 | /* | 596 | /* |
597 | * Will cancel the transaction, assumes ->disable() is called for | 597 | * Will cancel the transaction, assumes ->disable() is called |
598 | * each successfull ->enable() during the transaction. | 598 | * for each successfull ->enable() during the transaction. |
599 | */ | 599 | */ |
600 | void (*cancel_txn) (struct pmu *pmu); | 600 | void (*cancel_txn) (struct pmu *pmu); |
601 | }; | 601 | }; |