diff options
author | Samuel Thibault <samuel.thibault@ens-lyon.org> | 2016-04-20 00:32:15 +0200 |
---|---|---|
committer | Samuel Thibault <samuel.thibault@ens-lyon.org> | 2016-04-20 00:32:15 +0200 |
commit | 675fd4eecd75a7156c83bb814ecfdbbcfbf27ea1 (patch) | |
tree | ca1ef6507b938fe491bdf51a2fe6137de52d0f4c /kern | |
parent | c908225cb5def2c6df3cf889ca458db8b633d395 (diff) | |
download | gnumach-675fd4eecd75a7156c83bb814ecfdbbcfbf27ea1.tar.gz gnumach-675fd4eecd75a7156c83bb814ecfdbbcfbf27ea1.tar.bz2 gnumach-675fd4eecd75a7156c83bb814ecfdbbcfbf27ea1.zip |
Add kernel profiling through sampling
* NEWS: Advertise feature.
* configfrac.ac (--enable-kernsample): Add option.
* kern/pc_sample.h (take_pc_sample): Add usermode and pc parameter.
(take_pc_sample_macro): Take usermode and pc parameters, pass as such to
take_pc_sample.
* kern/pc_sample.c (take_pc_sample): Use pc parameter when usermode is 1.
* kern/mach_clock.c (clock_interrupt): Add pc parameter. Pass usermode and
pc to take_pc_sample_macro call.
* i386/i386/hardclock.c (hardclock): Pass regs->eip to clock_interrupt call
on normal interrupts, NULL on interrupt interrupt.
* vm/vm_fault.c (vm_fault_cleanup): Set usermode to 1 and pc to NULL in
take_pc_sample_macro call.
Diffstat (limited to 'kern')
-rw-r--r-- | kern/mach_clock.c | 10 | ||||
-rw-r--r-- | kern/mach_clock.h | 3 | ||||
-rw-r--r-- | kern/pc_sample.c | 10 | ||||
-rw-r--r-- | kern/pc_sample.h | 10 |
4 files changed, 23 insertions, 10 deletions
diff --git a/kern/mach_clock.c b/kern/mach_clock.c index 1817ce22..d6ebf23e 100644 --- a/kern/mach_clock.c +++ b/kern/mach_clock.c @@ -133,7 +133,8 @@ timer_elt_data_t timer_head; /* ordered list of timeouts */ void clock_interrupt( int usec, /* microseconds per tick */ boolean_t usermode, /* executing user code */ - boolean_t basepri) /* at base priority */ + boolean_t basepri, /* at base priority */ + vm_offset_t pc) /* address of interrupted instruction */ { int my_cpu = cpu_number(); thread_t thread = current_thread(); @@ -184,8 +185,11 @@ void clock_interrupt( * This had better be MP safe. It might be interesting * to keep track of cpu in the sample. */ - if (usermode) { - take_pc_sample_macro(thread, SAMPLED_PC_PERIODIC); +#ifndef MACH_KERNSAMPLE + if (usermode) +#endif + { + take_pc_sample_macro(thread, SAMPLED_PC_PERIODIC, usermode, pc); } #endif /* MACH_PCSAMPLE */ diff --git a/kern/mach_clock.h b/kern/mach_clock.h index 1af0cdae..977b43be 100644 --- a/kern/mach_clock.h +++ b/kern/mach_clock.h @@ -62,7 +62,8 @@ typedef struct timer_elt *timer_elt_t; extern void clock_interrupt( int usec, boolean_t usermode, - boolean_t basepri); + boolean_t basepri, + vm_offset_t pc); extern void softclock (void); diff --git a/kern/pc_sample.c b/kern/pc_sample.c index fcb9d71b..e9f0b16a 100644 --- a/kern/pc_sample.c +++ b/kern/pc_sample.c @@ -46,12 +46,18 @@ typedef sampled_pc_t sampled_pcs[MAX_PC_SAMPLES]; void take_pc_sample( const thread_t t, sample_control_t *cp, - sampled_pc_flavor_t flavor) + sampled_pc_flavor_t flavor, + boolean_t usermode, + vm_offset_t kern_pc) { vm_offset_t pc; struct sampled_pc *sample; - pc = interrupted_pc(t); + if (usermode) + pc = interrupted_pc(t); + else + pc = kern_pc; + cp->seqno++; sample = &((sampled_pc_t *)cp->buffer)[cp->seqno % MAX_PC_SAMPLES]; sample->id = (vm_offset_t)t; diff --git a/kern/pc_sample.h b/kern/pc_sample.h index 4832cb9f..04ca6671 100644 --- a/kern/pc_sample.h +++ b/kern/pc_sample.h @@ -71,22 +71,24 @@ typedef struct sample_control sample_control_t; extern void take_pc_sample( thread_t thread, sample_control_t *cp, - sampled_pc_flavor_t flavor); + sampled_pc_flavor_t flavor, + boolean_t usermode, + vm_offset_t pc); /* * Macro to do quick flavor check for sampling, * on both threads and tasks. */ -#define take_pc_sample_macro(thread, flavor) \ +#define take_pc_sample_macro(thread, flavor, usermode, pc) \ MACRO_BEGIN \ task_t task; \ \ if ((thread)->pc_sample.sampletypes & (flavor)) \ - take_pc_sample((thread), &(thread)->pc_sample, (flavor)); \ + take_pc_sample((thread), &(thread)->pc_sample, (flavor), usermode, pc); \ \ task = (thread)->task; \ if (task->pc_sample.sampletypes & (flavor)) \ - take_pc_sample((thread), &task->pc_sample, (flavor)); \ + take_pc_sample((thread), &task->pc_sample, (flavor), usermode, pc); \ MACRO_END #endif /* _KERN_PC_SAMPLE_H_ */ |