diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2010-06-03 15:45:26 -0700 | 
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-06-03 15:45:26 -0700 | 
| commit | f150dba6d4a1e275b62ca76572c2786c71b91e85 (patch) | |
| tree | 1fb8ed0a64f69431e8a3304a1b346a19979028b9 /kernel/trace/trace_event_perf.c | |
| parent | 636667a545b2d16797f27002a65d688c195c9b60 (diff) | |
| parent | c6df8d5ab87a246942d138321e1721edbb69f6e1 (diff) | |
Merge branch 'perf-fixes-for-linus-2' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip
* 'perf-fixes-for-linus-2' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip:
  perf: Fix crash in swevents
  perf buildid-list: Fix --with-hits event processing
  perf scripts python: Give field dict to unhandled callback
  perf hist: fix objdump output parsing
  perf-record: Check correct pid when forking
  perf: Do the comm inheritance per thread in event__process_task
  perf: Use event__process_task from perf sched
  perf: Process comm events by tid
  blktrace: Fix new kernel-doc warnings
  perf_events: Fix unincremented buffer base on partial copy
  perf_events: Fix event scheduling issues introduced by transactional API
  perf_events, trace: Fix perf_trace_destroy(), mutex went missing
  perf_events, trace: Fix probe unregister race
  perf_events: Fix races in group composition
  perf_events: Fix races and clean up perf_event and perf_mmap_data interaction
Diffstat (limited to 'kernel/trace/trace_event_perf.c')
| -rw-r--r-- | kernel/trace/trace_event_perf.c | 15 | 
1 files changed, 12 insertions, 3 deletions
diff --git a/kernel/trace/trace_event_perf.c b/kernel/trace/trace_event_perf.c index cb6f365016e..e6f65887842 100644 --- a/kernel/trace/trace_event_perf.c +++ b/kernel/trace/trace_event_perf.c @@ -116,7 +116,7 @@ int perf_trace_enable(struct perf_event *p_event)  	if (WARN_ON_ONCE(!list))  		return -EINVAL; -	list = per_cpu_ptr(list, smp_processor_id()); +	list = this_cpu_ptr(list);  	hlist_add_head_rcu(&p_event->hlist_entry, list);  	return 0; @@ -132,8 +132,9 @@ void perf_trace_destroy(struct perf_event *p_event)  	struct ftrace_event_call *tp_event = p_event->tp_event;  	int i; +	mutex_lock(&event_mutex);  	if (--tp_event->perf_refcount > 0) -		return; +		goto out;  	if (tp_event->class->reg)  		tp_event->class->reg(tp_event, TRACE_REG_PERF_UNREGISTER); @@ -142,6 +143,12 @@ void perf_trace_destroy(struct perf_event *p_event)  					    tp_event->class->perf_probe,  					    tp_event); +	/* +	 * Ensure our callback won't be called anymore. See +	 * tracepoint_probe_unregister() and __DO_TRACE(). +	 */ +	synchronize_sched(); +  	free_percpu(tp_event->perf_events);  	tp_event->perf_events = NULL; @@ -151,6 +158,8 @@ void perf_trace_destroy(struct perf_event *p_event)  			perf_trace_buf[i] = NULL;  		}  	} +out: +	mutex_unlock(&event_mutex);  }  __kprobes void *perf_trace_buf_prepare(int size, unsigned short type, @@ -169,7 +178,7 @@ __kprobes void *perf_trace_buf_prepare(int size, unsigned short type,  	if (*rctxp < 0)  		return NULL; -	raw_data = per_cpu_ptr(perf_trace_buf[*rctxp], smp_processor_id()); +	raw_data = this_cpu_ptr(perf_trace_buf[*rctxp]);  	/* zero the dead bytes from align to not leak stack to user */  	memset(&raw_data[size - sizeof(u64)], 0, sizeof(u64));  | 
