diff options
| author | Steven Rostedt <srostedt@redhat.com> | 2010-10-20 21:51:26 -0400 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2010-10-21 08:55:06 -0400 |
| commit | dd49a38cf30944be27892c10b1c0e5b3fa73bcb2 (patch) | |
| tree | 6f517e3e643923556c92a647918984fc2ea36bb8 /kernel/trace | |
| parent | 750ed158bf6c782d2813da1bca2c824365a0b777 (diff) | |
| download | kernel_samsung_aries-dd49a38cf30944be27892c10b1c0e5b3fa73bcb2.zip kernel_samsung_aries-dd49a38cf30944be27892c10b1c0e5b3fa73bcb2.tar.gz kernel_samsung_aries-dd49a38cf30944be27892c10b1c0e5b3fa73bcb2.tar.bz2 | |
tracing: Do not limit the size of the number of CPU buffers
The tracing per_cpu buffers were limited to 999 CPUs for a mear
savings in stack space of a char array. Up the array to 30 characters
which is more than enough to hold a 64 bit number.
Reported-by: Robin Holt <holt@sgi.com>
Suggested-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace')
| -rw-r--r-- | kernel/trace/trace.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 001bcd2..82d9b81 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -3996,13 +3996,9 @@ static void tracing_init_debugfs_percpu(long cpu) { struct dentry *d_percpu = tracing_dentry_percpu(); struct dentry *d_cpu; - /* strlen(cpu) + MAX(log10(cpu)) + '\0' */ - char cpu_dir[7]; + char cpu_dir[30]; /* 30 characters should be more than enough */ - if (cpu > 999 || cpu < 0) - return; - - sprintf(cpu_dir, "cpu%ld", cpu); + snprintf(cpu_dir, 30, "cpu%ld", cpu); d_cpu = debugfs_create_dir(cpu_dir, d_percpu); if (!d_cpu) { pr_warning("Could not create debugfs '%s' entry\n", cpu_dir); |