diff options
| author | Jesper Nilsson <jesper.nilsson@axis.com> | 2010-08-04 05:44:08 -0400 |
|---|---|---|
| committer | Jesper Nilsson <jesper.nilsson@axis.com> | 2010-08-04 07:02:40 -0400 |
| commit | c1c8f558749cbf2a7ed16b6ae6e19a4238b6fa33 (patch) | |
| tree | b329939fdc5347532cb44261892914493d27e64e | |
| parent | 72e08db187cbbfca00583c28b64ae81a35f4a287 (diff) | |
CRIS: Return something from profile write
Even if it doesn't matter, we should return something.
Also, clean up some formatting.
Signed-off-by: Jesper Nilsson <jesper.nilsson@axis.com>
| -rw-r--r-- | arch/cris/kernel/profile.c | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/arch/cris/kernel/profile.c b/arch/cris/kernel/profile.c index b917549a7d94..195ec5fa0dd2 100644 --- a/arch/cris/kernel/profile.c +++ b/arch/cris/kernel/profile.c | |||
| @@ -9,12 +9,11 @@ | |||
| 9 | 9 | ||
| 10 | #define SAMPLE_BUFFER_SIZE 8192 | 10 | #define SAMPLE_BUFFER_SIZE 8192 |
| 11 | 11 | ||
| 12 | static char* sample_buffer; | 12 | static char *sample_buffer; |
| 13 | static char* sample_buffer_pos; | 13 | static char *sample_buffer_pos; |
| 14 | static int prof_running = 0; | 14 | static int prof_running = 0; |
| 15 | 15 | ||
| 16 | void | 16 | void cris_profile_sample(struct pt_regs *regs) |
| 17 | cris_profile_sample(struct pt_regs* regs) | ||
| 18 | { | 17 | { |
| 19 | if (!prof_running) | 18 | if (!prof_running) |
| 20 | return; | 19 | return; |
| @@ -24,7 +23,7 @@ cris_profile_sample(struct pt_regs* regs) | |||
| 24 | else | 23 | else |
| 25 | *(unsigned int*)sample_buffer_pos = 0; | 24 | *(unsigned int*)sample_buffer_pos = 0; |
| 26 | 25 | ||
| 27 | *(unsigned int*)(sample_buffer_pos + 4) = instruction_pointer(regs); | 26 | *(unsigned int *)(sample_buffer_pos + 4) = instruction_pointer(regs); |
| 28 | sample_buffer_pos += 8; | 27 | sample_buffer_pos += 8; |
| 29 | 28 | ||
| 30 | if (sample_buffer_pos == sample_buffer + SAMPLE_BUFFER_SIZE) | 29 | if (sample_buffer_pos == sample_buffer + SAMPLE_BUFFER_SIZE) |
| @@ -54,6 +53,7 @@ write_cris_profile(struct file *file, const char __user *buf, | |||
| 54 | { | 53 | { |
| 55 | sample_buffer_pos = sample_buffer; | 54 | sample_buffer_pos = sample_buffer; |
| 56 | memset(sample_buffer, 0, SAMPLE_BUFFER_SIZE); | 55 | memset(sample_buffer, 0, SAMPLE_BUFFER_SIZE); |
| 56 | return count < SAMPLE_BUFFER_SIZE ? count : SAMPLE_BUFFER_SIZE; | ||
| 57 | } | 57 | } |
| 58 | 58 | ||
| 59 | static const struct file_operations cris_proc_profile_operations = { | 59 | static const struct file_operations cris_proc_profile_operations = { |
| @@ -61,8 +61,7 @@ static const struct file_operations cris_proc_profile_operations = { | |||
| 61 | .write = write_cris_profile, | 61 | .write = write_cris_profile, |
| 62 | }; | 62 | }; |
| 63 | 63 | ||
| 64 | static int | 64 | static int __init init_cris_profile(void) |
| 65 | __init init_cris_profile(void) | ||
| 66 | { | 65 | { |
| 67 | struct proc_dir_entry *entry; | 66 | struct proc_dir_entry *entry; |
| 68 | 67 | ||
| @@ -82,5 +81,5 @@ __init init_cris_profile(void) | |||
| 82 | 81 | ||
| 83 | return 0; | 82 | return 0; |
| 84 | } | 83 | } |
| 85 | |||
| 86 | __initcall(init_cris_profile); | 84 | __initcall(init_cris_profile); |
| 85 | |||
