Skip to contentSkip to conversation

trace: Fix some mistakes in error messages

PATCH In review Frédéric Weisbecker 1 participant 2 messages Aug 11, 2008 – Aug 14, 2008 Tracing

lore
Share & tools
lore

Frédéric Weisbecker

author
#0 lore raw

PATCH 1/1 trace: Fix some mistakes in error messages

This patch fixes some mistakes on the tracer in warning messages when debugfs fails to create tracing files. Kernel version: 2.6.27-rc1

Regards.

Signed-off-by Frederic Weisbecker <[email protected]>

---

kernel/trace/trace.c 2008-08-11 23:10:39.000000000 +0200 renamed from kernel/trace/trace.c 2008-08-11 22:44:25.000000000 +0200 +5 -5
diff -rup a/kernel/trace/trace.c b/kernel/trace/trace.c
@@ -2881,12 +2881,12 @@ static __init void tracer_init_debugfs(v
    entry = debugfs_create_file("available_tracers", 0444, d_tracer,
                    &global_trace, &show_traces_fops);
    if (!entry)
-        pr_warning("Could not create debugfs 'trace' entry\n");
+        pr_warning("Could not create debugfs 'available_tracers' entry\n");
 
    entry = debugfs_create_file("current_tracer", 0444, d_tracer,
                    &global_trace, &set_tracer_fops);
    if (!entry)
-        pr_warning("Could not create debugfs 'trace' entry\n");
+        pr_warning("Could not create debugfs 'current_tracer' entry\n");
 
    entry = debugfs_create_file("tracing_max_latency", 0644, d_tracer,
                    &tracing_max_latency,

@@ -2899,7 +2899,7 @@ static __init void tracer_init_debugfs(v
                    &tracing_thresh, &tracing_max_lat_fops);
    if (!entry)
        pr_warning("Could not create debugfs "
-               "'tracing_threash' entry\n");
+               "'tracing_thresh' entry\n");
    entry = debugfs_create_file("README", 0644, d_tracer,
                    NULL, &tracing_readme_fops);
    if (!entry)

@@ -2909,13 +2909,13 @@ static __init void tracer_init_debugfs(v
                    NULL, &tracing_pipe_fops);
    if (!entry)
        pr_warning("Could not create debugfs "
-               "'tracing_threash' entry\n");
+               "'trace_pipe' entry\n");
 
    entry = debugfs_create_file("trace_entries", 0644, d_tracer,
                    &global_trace, &tracing_entries_fops);
    if (!entry)
        pr_warning("Could not create debugfs "
-               "'tracing_threash' entry\n");
+               "'trace_entries' entry\n");

#ifdef CONFIG_DYNAMIC_FTRACE
entry = debugfs_create_file("dyn_ftrace_total_info", 0444, d_tracer,

author
#1 lore raw

2008/8/11 Frédéric Weisbecker <[email protected]>

Quoted 41 lines

This patch fixes some mistakes on the tracer in warning messages when debugfs fails to create tracing files. Kernel version: 2.6.27-rc1

Regards.

Signed-off-by Frederic Weisbecker <[email protected]>

---
diff -rup a/kernel/trace/trace.c b/kernel/trace/trace.c

kernel/trace/trace.c 2008-08-11 23:10:39.000000000 +0200 renamed from kernel/trace/trace.c 2008-08-11 22:44:25.000000000 +0200 +4 -4
@@ -2881,12 +2881,12 @@ static __init void tracer_init_debugfs(v
   entry = debugfs_create_file("available_tracers", 0444, d_tracer,
                   &global_trace, &show_traces_fops);
   if (!entry)
-        pr_warning("Could not create debugfs 'trace' entry\n");
+        pr_warning("Could not create debugfs 'available_tracers' entry\n");
 
   entry = debugfs_create_file("current_tracer", 0444, d_tracer,
                   &global_trace, &set_tracer_fops);
   if (!entry)
-        pr_warning("Could not create debugfs 'trace' entry\n");
+        pr_warning("Could not create debugfs 'current_tracer' entry\n");
 
   entry = debugfs_create_file("tracing_max_latency", 0644, d_tracer,
                   &tracing_max_latency,

@@ -2899,7 +2899,7 @@ static __init void tracer_init_debugfs(v
                   &tracing_thresh, &tracing_max_lat_fops);
   if (!entry)
       pr_warning("Could not create debugfs "
-               "'tracing_threash' entry\n");
+               "'tracing_thresh' entry\n");
   entry = debugfs_create_file("README", 0644, d_tracer,
                   NULL, &tracing_readme_fops);
   if (!entry)

@@ -2909,13 +2909,13 @@ static __init void tracer_init_debugfs(v
                   NULL, &tracing_pipe_fops);
   if (!entry)
       pr_warning("Could not create debugfs "
-               "'tracing_threash' entry\n");
+               "'trace_pipe' entry\n");

kernel/trace/trace.c 2008-08-11 23:10:39.000000000 +0200 static __init void tracer_init_debugfs(v · line 2921

Quoted 10 lines
kernel/trace/trace.c 2008-08-11 23:10:39.000000000 +0200 renamed from kernel/trace/trace.c 2008-08-11 22:44:25.000000000 +0200 +1 -1
 
   entry = debugfs_create_file("trace_entries", 0644, d_tracer,
                   &global_trace, &tracing_entries_fops);
   if (!entry)
       pr_warning("Could not create debugfs "
-               "'tracing_threash' entry\n");
+               "'trace_entries' entry\n");
 
 #ifdef CONFIG_DYNAMIC_FTRACE
   entry = debugfs_create_file("dyn_ftrace_total_info", 0444, d_tracer,

I said my kernel version was 2.6.27-rc1 but in fact it still applies on last git tree.