diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2010-01-09 14:04:47 -0500 |
---|---|---|
committer | Frederic Weisbecker <fweisbec@gmail.com> | 2010-01-16 06:27:42 -0500 |
commit | 889ff0150661512d79484219612b7e2e024b6c07 (patch) | |
tree | 6343ad7386aa3aefa7fe3f8ab9389f6abec1d141 /include | |
parent | 881516eb828a3f7276c378bcef96b7788fc99016 (diff) |
perf/core: Split context's event group list into pinned and non-pinned lists
Split-up struct perf_event_context::group_list into pinned_groups
and flexible_groups (non-pinned).
This first appears to be useless as it duplicates various loops around
the group list handlings.
But it scales better in the fast-path in perf_sched_in(). We don't
anymore iterate twice through the entire list to separate pinned and
non-pinned scheduling. Instead we interate through two distinct lists.
The another desired effect is that it makes easier to define distinct
scheduling rules on both.
Changes in v2:
- Respectively rename pinned_grp_list and
volatile_grp_list into pinned_groups and flexible_groups as per
Ingo suggestion.
- Various cleanups
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Acked-by: Peter Zijlstra <peterz@infradead.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Arnaldo Carvalho de Melo <acme@infradead.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/perf_event.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h index 9a1d276db754..cdbc2aa64a0b 100644 --- a/include/linux/perf_event.h +++ b/include/linux/perf_event.h | |||
@@ -683,7 +683,8 @@ struct perf_event_context { | |||
683 | */ | 683 | */ |
684 | struct mutex mutex; | 684 | struct mutex mutex; |
685 | 685 | ||
686 | struct list_head group_list; | 686 | struct list_head pinned_groups; |
687 | struct list_head flexible_groups; | ||
687 | struct list_head event_list; | 688 | struct list_head event_list; |
688 | int nr_events; | 689 | int nr_events; |
689 | int nr_active; | 690 | int nr_active; |