Commit c92960fc authored by Robert Richter's avatar Robert Richter

oprofile: whitespace fixes

Signed-off-by: default avatarRobert Richter <robert.richter@amd.com>
parent ccd755c2
...@@ -530,14 +530,14 @@ static void op_amd_exit(void) ...@@ -530,14 +530,14 @@ static void op_amd_exit(void)
#endif #endif
struct op_x86_model_spec const op_amd_spec = { struct op_x86_model_spec const op_amd_spec = {
.init = op_amd_init, .init = op_amd_init,
.exit = op_amd_exit, .exit = op_amd_exit,
.num_counters = NUM_COUNTERS, .num_counters = NUM_COUNTERS,
.num_controls = NUM_CONTROLS, .num_controls = NUM_CONTROLS,
.fill_in_addresses = &op_amd_fill_in_addresses, .fill_in_addresses = &op_amd_fill_in_addresses,
.setup_ctrs = &op_amd_setup_ctrs, .setup_ctrs = &op_amd_setup_ctrs,
.check_ctrs = &op_amd_check_ctrs, .check_ctrs = &op_amd_check_ctrs,
.start = &op_amd_start, .start = &op_amd_start,
.stop = &op_amd_stop, .stop = &op_amd_stop,
.shutdown = &op_amd_shutdown .shutdown = &op_amd_shutdown
}; };
...@@ -698,24 +698,24 @@ static void p4_shutdown(struct op_msrs const * const msrs) ...@@ -698,24 +698,24 @@ static void p4_shutdown(struct op_msrs const * const msrs)
#ifdef CONFIG_SMP #ifdef CONFIG_SMP
struct op_x86_model_spec const op_p4_ht2_spec = { struct op_x86_model_spec const op_p4_ht2_spec = {
.num_counters = NUM_COUNTERS_HT2, .num_counters = NUM_COUNTERS_HT2,
.num_controls = NUM_CONTROLS_HT2, .num_controls = NUM_CONTROLS_HT2,
.fill_in_addresses = &p4_fill_in_addresses, .fill_in_addresses = &p4_fill_in_addresses,
.setup_ctrs = &p4_setup_ctrs, .setup_ctrs = &p4_setup_ctrs,
.check_ctrs = &p4_check_ctrs, .check_ctrs = &p4_check_ctrs,
.start = &p4_start, .start = &p4_start,
.stop = &p4_stop, .stop = &p4_stop,
.shutdown = &p4_shutdown .shutdown = &p4_shutdown
}; };
#endif #endif
struct op_x86_model_spec const op_p4_spec = { struct op_x86_model_spec const op_p4_spec = {
.num_counters = NUM_COUNTERS_NON_HT, .num_counters = NUM_COUNTERS_NON_HT,
.num_controls = NUM_CONTROLS_NON_HT, .num_controls = NUM_CONTROLS_NON_HT,
.fill_in_addresses = &p4_fill_in_addresses, .fill_in_addresses = &p4_fill_in_addresses,
.setup_ctrs = &p4_setup_ctrs, .setup_ctrs = &p4_setup_ctrs,
.check_ctrs = &p4_check_ctrs, .check_ctrs = &p4_check_ctrs,
.start = &p4_start, .start = &p4_start,
.stop = &p4_stop, .stop = &p4_stop,
.shutdown = &p4_shutdown .shutdown = &p4_shutdown
}; };
...@@ -181,12 +181,12 @@ static void ppro_shutdown(struct op_msrs const * const msrs) ...@@ -181,12 +181,12 @@ static void ppro_shutdown(struct op_msrs const * const msrs)
struct op_x86_model_spec const op_ppro_spec = { struct op_x86_model_spec const op_ppro_spec = {
.num_counters = NUM_COUNTERS, .num_counters = NUM_COUNTERS,
.num_controls = NUM_CONTROLS, .num_controls = NUM_CONTROLS,
.fill_in_addresses = &ppro_fill_in_addresses, .fill_in_addresses = &ppro_fill_in_addresses,
.setup_ctrs = &ppro_setup_ctrs, .setup_ctrs = &ppro_setup_ctrs,
.check_ctrs = &ppro_check_ctrs, .check_ctrs = &ppro_check_ctrs,
.start = &ppro_start, .start = &ppro_start,
.stop = &ppro_stop, .stop = &ppro_stop,
.shutdown = &ppro_shutdown .shutdown = &ppro_shutdown
}; };
...@@ -41,7 +41,6 @@ static cpumask_t marked_cpus = CPU_MASK_NONE; ...@@ -41,7 +41,6 @@ static cpumask_t marked_cpus = CPU_MASK_NONE;
static DEFINE_SPINLOCK(task_mortuary); static DEFINE_SPINLOCK(task_mortuary);
static void process_task_mortuary(void); static void process_task_mortuary(void);
/* Take ownership of the task struct and place it on the /* Take ownership of the task struct and place it on the
* list for processing. Only after two full buffer syncs * list for processing. Only after two full buffer syncs
* does the task eventually get freed, because by then * does the task eventually get freed, because by then
......
...@@ -19,7 +19,7 @@ ...@@ -19,7 +19,7 @@
#include "cpu_buffer.h" #include "cpu_buffer.h"
#include "buffer_sync.h" #include "buffer_sync.h"
#include "oprofile_stats.h" #include "oprofile_stats.h"
struct oprofile_operations oprofile_ops; struct oprofile_operations oprofile_ops;
unsigned long oprofile_started; unsigned long oprofile_started;
...@@ -36,7 +36,7 @@ static int timer = 0; ...@@ -36,7 +36,7 @@ static int timer = 0;
int oprofile_setup(void) int oprofile_setup(void)
{ {
int err; int err;
mutex_lock(&start_mutex); mutex_lock(&start_mutex);
if ((err = alloc_cpu_buffers())) if ((err = alloc_cpu_buffers()))
...@@ -44,10 +44,10 @@ int oprofile_setup(void) ...@@ -44,10 +44,10 @@ int oprofile_setup(void)
if ((err = alloc_event_buffer())) if ((err = alloc_event_buffer()))
goto out1; goto out1;
if (oprofile_ops.setup && (err = oprofile_ops.setup())) if (oprofile_ops.setup && (err = oprofile_ops.setup()))
goto out2; goto out2;
/* Note even though this starts part of the /* Note even though this starts part of the
* profiling overhead, it's necessary to prevent * profiling overhead, it's necessary to prevent
* us missing task deaths and eventually oopsing * us missing task deaths and eventually oopsing
...@@ -74,7 +74,7 @@ int oprofile_setup(void) ...@@ -74,7 +74,7 @@ int oprofile_setup(void)
is_setup = 1; is_setup = 1;
mutex_unlock(&start_mutex); mutex_unlock(&start_mutex);
return 0; return 0;
out3: out3:
if (oprofile_ops.shutdown) if (oprofile_ops.shutdown)
oprofile_ops.shutdown(); oprofile_ops.shutdown();
...@@ -92,17 +92,17 @@ int oprofile_setup(void) ...@@ -92,17 +92,17 @@ int oprofile_setup(void)
int oprofile_start(void) int oprofile_start(void)
{ {
int err = -EINVAL; int err = -EINVAL;
mutex_lock(&start_mutex); mutex_lock(&start_mutex);
if (!is_setup) if (!is_setup)
goto out; goto out;
err = 0; err = 0;
if (oprofile_started) if (oprofile_started)
goto out; goto out;
oprofile_reset_stats(); oprofile_reset_stats();
if ((err = oprofile_ops.start())) if ((err = oprofile_ops.start()))
...@@ -114,7 +114,7 @@ int oprofile_start(void) ...@@ -114,7 +114,7 @@ int oprofile_start(void)
return err; return err;
} }
/* echo 0>/dev/oprofile/enable */ /* echo 0>/dev/oprofile/enable */
void oprofile_stop(void) void oprofile_stop(void)
{ {
...@@ -204,13 +204,13 @@ static void __exit oprofile_exit(void) ...@@ -204,13 +204,13 @@ static void __exit oprofile_exit(void)
oprofile_arch_exit(); oprofile_arch_exit();
} }
module_init(oprofile_init); module_init(oprofile_init);
module_exit(oprofile_exit); module_exit(oprofile_exit);
module_param_named(timer, timer, int, 0644); module_param_named(timer, timer, int, 0644);
MODULE_PARM_DESC(timer, "force use of timer interrupt"); MODULE_PARM_DESC(timer, "force use of timer interrupt");
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
MODULE_AUTHOR("John Levon <levon@movementarian.org>"); MODULE_AUTHOR("John Levon <levon@movementarian.org>");
MODULE_DESCRIPTION("OProfile system profiler"); MODULE_DESCRIPTION("OProfile system profiler");
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