Commit fd13f6c8 authored by Robert Richter's avatar Robert Richter

oprofile: comment cleanup

This fixes the coding style of some comments.
Signed-off-by: default avatarRobert Richter <robert.richter@amd.com>
parent 061e41fd
...@@ -65,8 +65,10 @@ static unsigned long reset_value[NUM_COUNTERS]; ...@@ -65,8 +65,10 @@ static unsigned long reset_value[NUM_COUNTERS];
#define IBS_FETCH_BEGIN 3 #define IBS_FETCH_BEGIN 3
#define IBS_OP_BEGIN 4 #define IBS_OP_BEGIN 4
/* The function interface needs to be fixed, something like add /*
data. Should then be added to linux/oprofile.h. */ * The function interface needs to be fixed, something like add
* data. Should then be added to linux/oprofile.h.
*/
extern void extern void
oprofile_add_ibs_sample(struct pt_regs *const regs, oprofile_add_ibs_sample(struct pt_regs *const regs,
unsigned int *const ibs_sample, int ibs_code); unsigned int *const ibs_sample, int ibs_code);
...@@ -106,7 +108,7 @@ struct ibs_op_sample { ...@@ -106,7 +108,7 @@ struct ibs_op_sample {
/* /*
* unitialize the APIC for the IBS interrupts if needed on AMD Family10h+ * unitialize the APIC for the IBS interrupts if needed on AMD Family10h+
*/ */
static void clear_ibs_nmi(void); static void clear_ibs_nmi(void);
static int ibs_allowed; /* AMD Family10h and later */ static int ibs_allowed; /* AMD Family10h and later */
...@@ -223,7 +225,7 @@ op_amd_handle_ibs(struct pt_regs * const regs, ...@@ -223,7 +225,7 @@ op_amd_handle_ibs(struct pt_regs * const regs,
(unsigned int *)&ibs_fetch, (unsigned int *)&ibs_fetch,
IBS_FETCH_BEGIN); IBS_FETCH_BEGIN);
/*reenable the IRQ */ /* reenable the IRQ */
rdmsr(MSR_AMD64_IBSFETCHCTL, low, high); rdmsr(MSR_AMD64_IBSFETCHCTL, low, high);
high &= ~IBS_FETCH_HIGH_VALID_BIT; high &= ~IBS_FETCH_HIGH_VALID_BIT;
high |= IBS_FETCH_HIGH_ENABLE; high |= IBS_FETCH_HIGH_ENABLE;
...@@ -331,8 +333,10 @@ static void op_amd_stop(struct op_msrs const * const msrs) ...@@ -331,8 +333,10 @@ static void op_amd_stop(struct op_msrs const * const msrs)
unsigned int low, high; unsigned int low, high;
int i; int i;
/* Subtle: stop on all counters to avoid race with /*
* setting our pm callback */ * Subtle: stop on all counters to avoid race with setting our
* pm callback
*/
for (i = 0 ; i < NUM_COUNTERS ; ++i) { for (i = 0 ; i < NUM_COUNTERS ; ++i) {
if (!reset_value[i]) if (!reset_value[i])
continue; continue;
...@@ -343,13 +347,15 @@ static void op_amd_stop(struct op_msrs const * const msrs) ...@@ -343,13 +347,15 @@ static void op_amd_stop(struct op_msrs const * const msrs)
#ifdef CONFIG_OPROFILE_IBS #ifdef CONFIG_OPROFILE_IBS
if (ibs_allowed && ibs_config.fetch_enabled) { if (ibs_allowed && ibs_config.fetch_enabled) {
low = 0; /* clear max count and enable */ /* clear max count and enable */
low = 0;
high = 0; high = 0;
wrmsr(MSR_AMD64_IBSFETCHCTL, low, high); wrmsr(MSR_AMD64_IBSFETCHCTL, low, high);
} }
if (ibs_allowed && ibs_config.op_enabled) { if (ibs_allowed && ibs_config.op_enabled) {
low = 0; /* clear max count and enable */ /* clear max count and enable */
low = 0;
high = 0; high = 0;
wrmsr(MSR_AMD64_IBSOPCTL, low, high); wrmsr(MSR_AMD64_IBSOPCTL, low, high);
} }
...@@ -443,10 +449,7 @@ static int pfm_amd64_setup_eilvt(void) ...@@ -443,10 +449,7 @@ static int pfm_amd64_setup_eilvt(void)
return 0; return 0;
} }
/* /* initialize the APIC for the IBS interrupts if available */
* initialize the APIC for the IBS interrupts
* if available (AMD Family10h rev B0 and later)
*/
static void setup_ibs(void) static void setup_ibs(void)
{ {
ibs_allowed = boot_cpu_has(X86_FEATURE_IBS); ibs_allowed = boot_cpu_has(X86_FEATURE_IBS);
...@@ -463,9 +466,7 @@ static void setup_ibs(void) ...@@ -463,9 +466,7 @@ static void setup_ibs(void)
} }
/* /* uninitialize the APIC for the IBS interrupts if needed */
* unitialize the APIC for the IBS interrupts if needed on AMD Family10h
* rev B0 and later */
static void clear_ibs_nmi(void) static void clear_ibs_nmi(void)
{ {
if (ibs_allowed) if (ibs_allowed)
......
...@@ -127,9 +127,10 @@ void end_cpu_work(void) ...@@ -127,9 +127,10 @@ void end_cpu_work(void)
/* Resets the cpu buffer to a sane state. */ /* Resets the cpu buffer to a sane state. */
void cpu_buffer_reset(struct oprofile_cpu_buffer *cpu_buf) void cpu_buffer_reset(struct oprofile_cpu_buffer *cpu_buf)
{ {
/* reset these to invalid values; the next sample /*
* collected will populate the buffer with proper * reset these to invalid values; the next sample collected
* values to initialize the buffer * will populate the buffer with proper values to initialize
* the buffer
*/ */
cpu_buf->last_is_kernel = -1; cpu_buf->last_is_kernel = -1;
cpu_buf->last_task = NULL; cpu_buf->last_task = NULL;
...@@ -151,8 +152,10 @@ static void increment_head(struct oprofile_cpu_buffer *b) ...@@ -151,8 +152,10 @@ static void increment_head(struct oprofile_cpu_buffer *b)
{ {
unsigned long new_head = b->head_pos + 1; unsigned long new_head = b->head_pos + 1;
/* Ensure anything written to the slot before we /*
* increment is visible */ * Ensure anything written to the slot before we increment is
* visible
*/
wmb(); wmb();
if (new_head < b->buffer_size) if (new_head < b->buffer_size)
...@@ -253,8 +256,10 @@ void oprofile_add_ext_sample(unsigned long pc, struct pt_regs * const regs, ...@@ -253,8 +256,10 @@ void oprofile_add_ext_sample(unsigned long pc, struct pt_regs * const regs,
if (!oprofile_begin_trace(cpu_buf)) if (!oprofile_begin_trace(cpu_buf))
return; return;
/* if log_sample() fail we can't backtrace since we lost the source /*
* of this event */ * if log_sample() fail we can't backtrace since we lost the
* source of this event
*/
if (log_sample(cpu_buf, pc, is_kernel, event)) if (log_sample(cpu_buf, pc, is_kernel, event))
oprofile_ops.backtrace(regs, backtrace_depth); oprofile_ops.backtrace(regs, backtrace_depth);
oprofile_end_trace(cpu_buf); oprofile_end_trace(cpu_buf);
...@@ -338,8 +343,10 @@ void oprofile_add_trace(unsigned long pc) ...@@ -338,8 +343,10 @@ void oprofile_add_trace(unsigned long pc)
return; return;
} }
/* broken frame can give an eip with the same value as an escape code, /*
* abort the trace if we get it */ * broken frame can give an eip with the same value as an
* escape code, abort the trace if we get it
*/
if (pc == ESCAPE_CODE) { if (pc == ESCAPE_CODE) {
cpu_buf->tracing = 0; cpu_buf->tracing = 0;
cpu_buf->backtrace_aborted++; cpu_buf->backtrace_aborted++;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment