diff options
author | Christoph Lameter <cl@linux.com> | 2010-12-08 11:42:23 -0500 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2010-12-17 09:18:05 -0500 |
commit | 3ea9f6833c8f865a221b59ce37d7650dcf3b3e17 (patch) | |
tree | 1f304f3ef65c84c9a6446d2ad7fa3ce2e7e248ad | |
parent | b2e4ae69757cdfef4c612a04f097c1e20489a565 (diff) |
connector: Use this_cpu operations
The patch was originally in the use cpuops patchset but it needs an
inc_return and is therefore dependent on an extension of the cpu ops.
Fixed up and verified that it compiles.
get_seq can benefit from this_cpu_operations. Address calculation is
avoided and the increment is done using an xadd.
Cc: Scott James Remnant <scott@ubuntu.com>
Cc: Mike Frysinger <vapier@gentoo.org>
Acked-by: H. Peter Anvin <hpa@zytor.com>
Signed-off-by: Christoph Lameter <cl@linux.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
-rw-r--r-- | drivers/connector/cn_proc.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/connector/cn_proc.c b/drivers/connector/cn_proc.c index a7f046b0096c..2b46a7efa0ac 100644 --- a/drivers/connector/cn_proc.c +++ b/drivers/connector/cn_proc.c | |||
@@ -43,9 +43,10 @@ static DEFINE_PER_CPU(__u32, proc_event_counts) = { 0 }; | |||
43 | 43 | ||
44 | static inline void get_seq(__u32 *ts, int *cpu) | 44 | static inline void get_seq(__u32 *ts, int *cpu) |
45 | { | 45 | { |
46 | *ts = get_cpu_var(proc_event_counts)++; | 46 | preempt_disable(); |
47 | *ts = __this_cpu_inc_return(proc_event_counts) -1; | ||
47 | *cpu = smp_processor_id(); | 48 | *cpu = smp_processor_id(); |
48 | put_cpu_var(proc_event_counts); | 49 | preempt_enable(); |
49 | } | 50 | } |
50 | 51 | ||
51 | void proc_fork_connector(struct task_struct *task) | 52 | void proc_fork_connector(struct task_struct *task) |