Skip to content

Commit

Permalink
perf, x86: Extract PEBS/BTS buffer free routines
Browse files Browse the repository at this point in the history
So that we may grow additional call-sites..

Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Acked-by: Stephane Eranian <eranian@google.com>
LKML-Reference: <20101019134808.196793164@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
  • Loading branch information
Peter Zijlstra authored and Ingo Molnar committed Oct 22, 2010
1 parent d4429f6 commit b39f88a
Showing 1 changed file with 25 additions and 5 deletions.
30 changes: 25 additions & 5 deletions arch/x86/kernel/cpu/perf_event_intel_ds.c
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,28 @@ static void fini_debug_store_on_cpu(int cpu)
wrmsr_on_cpu(cpu, MSR_IA32_DS_AREA, 0, 0);
}

static void release_pebs_buffer(int cpu)
{
struct debug_store *ds = per_cpu(cpu_hw_events, cpu).ds;

if (!ds || !x86_pmu.pebs)
return;

kfree((void *)(unsigned long)ds->pebs_buffer_base);
ds->pebs_buffer_base = 0;
}

static void release_bts_buffer(int cpu)
{
struct debug_store *ds = per_cpu(cpu_hw_events, cpu).ds;

if (!ds || !x86_pmu.bts)
return;

kfree((void *)(unsigned long)ds->bts_buffer_base);
ds->bts_buffer_base = 0;
}

static void release_ds_buffers(void)
{
int cpu;
Expand All @@ -82,7 +104,6 @@ static void release_ds_buffers(void)
return;

get_online_cpus();

for_each_online_cpu(cpu)
fini_debug_store_on_cpu(cpu);

Expand All @@ -92,13 +113,12 @@ static void release_ds_buffers(void)
if (!ds)
continue;

per_cpu(cpu_hw_events, cpu).ds = NULL;
release_pebs_buffer(cpu);
release_bts_buffer(cpu);

kfree((void *)(unsigned long)ds->pebs_buffer_base);
kfree((void *)(unsigned long)ds->bts_buffer_base);
per_cpu(cpu_hw_events, cpu).ds = NULL;
kfree(ds);
}

put_online_cpus();
}

Expand Down

0 comments on commit b39f88a

Please sign in to comment.