 1d037ca164
			
		
	
	
	1d037ca164
	
	
	
		
			
			perf defines both __used and __unused variables to use for marking
unused variables. The variable __used is defined to
__attribute__((__unused__)), which contradicts the kernel definition to
__attribute__((__used__)) for new gcc versions. On Android, __used is
also defined in system headers and this leads to warnings like: warning:
'__used__' attribute ignored
__unused is not defined in the kernel and is not a standard definition.
If __unused is included everywhere instead of __used, this leads to
conflicts with glibc headers, since glibc has a variables with this name
in its headers.
The best approach is to use __maybe_unused, the definition used in the
kernel for __attribute__((unused)). In this way there is only one
definition in perf sources (instead of 2 definitions that point to the
same thing: __used and __unused) and it works on both Linux and Android.
This patch simply replaces all instances of __used and __unused with
__maybe_unused.
Signed-off-by: Irina Tirdea <irina.tirdea@intel.com>
Acked-by: Pekka Enberg <penberg@kernel.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Namhyung Kim <namhyung.kim@lge.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Steven Rostedt <rostedt@goodmis.org>
Link: http://lkml.kernel.org/r/1347315303-29906-7-git-send-email-irina.tirdea@intel.com
[ committer note: fixed up conflict with a116e05 in builtin-sched.c ]
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
		
	
			
		
			
				
	
	
		
			89 lines
		
	
	
	
		
			2.3 KiB
			
		
	
	
	
		
			C
		
	
	
	
	
	
			
		
		
	
	
			89 lines
		
	
	
	
		
			2.3 KiB
			
		
	
	
	
		
			C
		
	
	
	
	
	
| /*
 | |
|  * build-id.c
 | |
|  *
 | |
|  * build-id support
 | |
|  *
 | |
|  * Copyright (C) 2009, 2010 Red Hat Inc.
 | |
|  * Copyright (C) 2009, 2010 Arnaldo Carvalho de Melo <acme@redhat.com>
 | |
|  */
 | |
| #include "util.h"
 | |
| #include <stdio.h>
 | |
| #include "build-id.h"
 | |
| #include "event.h"
 | |
| #include "symbol.h"
 | |
| #include <linux/kernel.h>
 | |
| #include "debug.h"
 | |
| #include "session.h"
 | |
| #include "tool.h"
 | |
| 
 | |
| static int build_id__mark_dso_hit(struct perf_tool *tool __maybe_unused,
 | |
| 				  union perf_event *event,
 | |
| 				  struct perf_sample *sample __maybe_unused,
 | |
| 				  struct perf_evsel *evsel __maybe_unused,
 | |
| 				  struct machine *machine)
 | |
| {
 | |
| 	struct addr_location al;
 | |
| 	u8 cpumode = event->header.misc & PERF_RECORD_MISC_CPUMODE_MASK;
 | |
| 	struct thread *thread = machine__findnew_thread(machine, event->ip.pid);
 | |
| 
 | |
| 	if (thread == NULL) {
 | |
| 		pr_err("problem processing %d event, skipping it.\n",
 | |
| 			event->header.type);
 | |
| 		return -1;
 | |
| 	}
 | |
| 
 | |
| 	thread__find_addr_map(thread, machine, cpumode, MAP__FUNCTION,
 | |
| 			      event->ip.ip, &al);
 | |
| 
 | |
| 	if (al.map != NULL)
 | |
| 		al.map->dso->hit = 1;
 | |
| 
 | |
| 	return 0;
 | |
| }
 | |
| 
 | |
| static int perf_event__exit_del_thread(struct perf_tool *tool __maybe_unused,
 | |
| 				       union perf_event *event,
 | |
| 				       struct perf_sample *sample
 | |
| 				       __maybe_unused,
 | |
| 				       struct machine *machine)
 | |
| {
 | |
| 	struct thread *thread = machine__findnew_thread(machine, event->fork.tid);
 | |
| 
 | |
| 	dump_printf("(%d:%d):(%d:%d)\n", event->fork.pid, event->fork.tid,
 | |
| 		    event->fork.ppid, event->fork.ptid);
 | |
| 
 | |
| 	if (thread) {
 | |
| 		rb_erase(&thread->rb_node, &machine->threads);
 | |
| 		machine->last_match = NULL;
 | |
| 		thread__delete(thread);
 | |
| 	}
 | |
| 
 | |
| 	return 0;
 | |
| }
 | |
| 
 | |
| struct perf_tool build_id__mark_dso_hit_ops = {
 | |
| 	.sample	= build_id__mark_dso_hit,
 | |
| 	.mmap	= perf_event__process_mmap,
 | |
| 	.fork	= perf_event__process_task,
 | |
| 	.exit	= perf_event__exit_del_thread,
 | |
| 	.attr		 = perf_event__process_attr,
 | |
| 	.build_id	 = perf_event__process_build_id,
 | |
| };
 | |
| 
 | |
| char *dso__build_id_filename(struct dso *self, char *bf, size_t size)
 | |
| {
 | |
| 	char build_id_hex[BUILD_ID_SIZE * 2 + 1];
 | |
| 
 | |
| 	if (!self->has_build_id)
 | |
| 		return NULL;
 | |
| 
 | |
| 	build_id__sprintf(self->build_id, sizeof(self->build_id), build_id_hex);
 | |
| 	if (bf == NULL) {
 | |
| 		if (asprintf(&bf, "%s/.build-id/%.2s/%s", buildid_dir,
 | |
| 			     build_id_hex, build_id_hex + 2) < 0)
 | |
| 			return NULL;
 | |
| 	} else
 | |
| 		snprintf(bf, size, "%s/.build-id/%.2s/%s", buildid_dir,
 | |
| 			 build_id_hex, build_id_hex + 2);
 | |
| 	return bf;
 | |
| }
 |