path: root/drivers/oprofile/cpu_buffer.h
diff options
authorPhilippe Elie <>2007-11-14 16:58:48 -0800
committerLinus Torvalds <>2007-11-14 18:45:37 -0800
commitdf9d177aa28d50e64bae6fbd6b263833079e3571 (patch)
treeb265158232a39d9d45cac0a97d5fdc17e7ca237d /drivers/oprofile/cpu_buffer.h
parent03ad369ac900116f35da7505f768ebbd481d09a4 (diff)
oProfile: oops when profile_pc() returns ~0LU
Instruction pointer returned by profile_pc() can be a random value. This break the assumption than we can safely set struct op_sample.eip field to a magic value to signal to the per-cpu buffer reader side special event like task switch ending up in a segfault in get_task_mm() when profile_pc() return ~0UL. Fixed by sanitizing the sampled eip and reject/log invalid eip. Problem reported by Sami Farin, patch tested by him. Signed-off-by: Philippe Elie <> Tested-by: Sami Farin <> Cc: <> Signed-off-by: Andrew Morton <> Signed-off-by: Linus Torvalds <>
Diffstat (limited to 'drivers/oprofile/cpu_buffer.h')
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/oprofile/cpu_buffer.h b/drivers/oprofile/cpu_buffer.h
index 49900d9e3235..c66c025abe75 100644
--- a/drivers/oprofile/cpu_buffer.h
+++ b/drivers/oprofile/cpu_buffer.h
@@ -42,6 +42,7 @@ struct oprofile_cpu_buffer {
unsigned long sample_received;
unsigned long sample_lost_overflow;
unsigned long backtrace_aborted;
+ unsigned long sample_invalid_eip;
int cpu;
struct delayed_work work;
} ____cacheline_aligned;