The ->counts field was never freed in the current code. Add perf_evsel__free_counts() function to free it properly. Signed-off-by: Namhyung Kim <namhyung@kernel.org> Cc: Ingo Molnar <mingo@kernel.org> Cc: Paul Mackerras <paulus@samba.org> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Stephane Eranian <eranian@google.com> Link: http://lkml.kernel.org/r/1359078284-32080-1-git-send-email-namhyung@kernel.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
		
			
				
	
	
		
			109 lines
		
	
	
	
		
			2.7 KiB
			
		
	
	
	
		
			C
		
	
	
	
	
	
			
		
		
	
	
			109 lines
		
	
	
	
		
			2.7 KiB
			
		
	
	
	
		
			C
		
	
	
	
	
	
#include "evsel.h"
 | 
						|
#include "tests.h"
 | 
						|
#include "thread_map.h"
 | 
						|
#include "cpumap.h"
 | 
						|
#include "debug.h"
 | 
						|
 | 
						|
int test__open_syscall_event_on_all_cpus(void)
 | 
						|
{
 | 
						|
	int err = -1, fd, cpu;
 | 
						|
	struct cpu_map *cpus;
 | 
						|
	struct perf_evsel *evsel;
 | 
						|
	unsigned int nr_open_calls = 111, i;
 | 
						|
	cpu_set_t cpu_set;
 | 
						|
	struct thread_map *threads = thread_map__new(-1, getpid(), UINT_MAX);
 | 
						|
 | 
						|
	if (threads == NULL) {
 | 
						|
		pr_debug("thread_map__new\n");
 | 
						|
		return -1;
 | 
						|
	}
 | 
						|
 | 
						|
	cpus = cpu_map__new(NULL);
 | 
						|
	if (cpus == NULL) {
 | 
						|
		pr_debug("cpu_map__new\n");
 | 
						|
		goto out_thread_map_delete;
 | 
						|
	}
 | 
						|
 | 
						|
	CPU_ZERO(&cpu_set);
 | 
						|
 | 
						|
	evsel = perf_evsel__newtp("syscalls", "sys_enter_open", 0);
 | 
						|
	if (evsel == NULL) {
 | 
						|
		pr_debug("is debugfs mounted on /sys/kernel/debug?\n");
 | 
						|
		goto out_thread_map_delete;
 | 
						|
	}
 | 
						|
 | 
						|
	if (perf_evsel__open(evsel, cpus, threads) < 0) {
 | 
						|
		pr_debug("failed to open counter: %s, "
 | 
						|
			 "tweak /proc/sys/kernel/perf_event_paranoid?\n",
 | 
						|
			 strerror(errno));
 | 
						|
		goto out_evsel_delete;
 | 
						|
	}
 | 
						|
 | 
						|
	for (cpu = 0; cpu < cpus->nr; ++cpu) {
 | 
						|
		unsigned int ncalls = nr_open_calls + cpu;
 | 
						|
		/*
 | 
						|
		 * XXX eventually lift this restriction in a way that
 | 
						|
		 * keeps perf building on older glibc installations
 | 
						|
		 * without CPU_ALLOC. 1024 cpus in 2010 still seems
 | 
						|
		 * a reasonable upper limit tho :-)
 | 
						|
		 */
 | 
						|
		if (cpus->map[cpu] >= CPU_SETSIZE) {
 | 
						|
			pr_debug("Ignoring CPU %d\n", cpus->map[cpu]);
 | 
						|
			continue;
 | 
						|
		}
 | 
						|
 | 
						|
		CPU_SET(cpus->map[cpu], &cpu_set);
 | 
						|
		if (sched_setaffinity(0, sizeof(cpu_set), &cpu_set) < 0) {
 | 
						|
			pr_debug("sched_setaffinity() failed on CPU %d: %s ",
 | 
						|
				 cpus->map[cpu],
 | 
						|
				 strerror(errno));
 | 
						|
			goto out_close_fd;
 | 
						|
		}
 | 
						|
		for (i = 0; i < ncalls; ++i) {
 | 
						|
			fd = open("/etc/passwd", O_RDONLY);
 | 
						|
			close(fd);
 | 
						|
		}
 | 
						|
		CPU_CLR(cpus->map[cpu], &cpu_set);
 | 
						|
	}
 | 
						|
 | 
						|
	/*
 | 
						|
	 * Here we need to explicitely preallocate the counts, as if
 | 
						|
	 * we use the auto allocation it will allocate just for 1 cpu,
 | 
						|
	 * as we start by cpu 0.
 | 
						|
	 */
 | 
						|
	if (perf_evsel__alloc_counts(evsel, cpus->nr) < 0) {
 | 
						|
		pr_debug("perf_evsel__alloc_counts(ncpus=%d)\n", cpus->nr);
 | 
						|
		goto out_close_fd;
 | 
						|
	}
 | 
						|
 | 
						|
	err = 0;
 | 
						|
 | 
						|
	for (cpu = 0; cpu < cpus->nr; ++cpu) {
 | 
						|
		unsigned int expected;
 | 
						|
 | 
						|
		if (cpus->map[cpu] >= CPU_SETSIZE)
 | 
						|
			continue;
 | 
						|
 | 
						|
		if (perf_evsel__read_on_cpu(evsel, cpu, 0) < 0) {
 | 
						|
			pr_debug("perf_evsel__read_on_cpu\n");
 | 
						|
			err = -1;
 | 
						|
			break;
 | 
						|
		}
 | 
						|
 | 
						|
		expected = nr_open_calls + cpu;
 | 
						|
		if (evsel->counts->cpu[cpu].val != expected) {
 | 
						|
			pr_debug("perf_evsel__read_on_cpu: expected to intercept %d calls on cpu %d, got %" PRIu64 "\n",
 | 
						|
				 expected, cpus->map[cpu], evsel->counts->cpu[cpu].val);
 | 
						|
			err = -1;
 | 
						|
		}
 | 
						|
	}
 | 
						|
 | 
						|
	perf_evsel__free_counts(evsel);
 | 
						|
out_close_fd:
 | 
						|
	perf_evsel__close_fd(evsel, 1, threads->nr);
 | 
						|
out_evsel_delete:
 | 
						|
	perf_evsel__delete(evsel);
 | 
						|
out_thread_map_delete:
 | 
						|
	thread_map__delete(threads);
 | 
						|
	return err;
 | 
						|
}
 |