PWN-Hunter/android_kernel_xiaomi_sdm845
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tracing/fastboot: get the initcall name before it disappears

Browse Source 
After some initcall traces, some initcall names may be inconsistent.
That's because these functions will disappear from the .init section
and also their name from the symbols table.

So we have to copy the name of the function in a buffer large enough
during the trace appending. It is not costly for the ring_buffer because
the number of initcall entries is commonly not really large.

Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
This commit is contained in:
Frederic Weisbecker 2008-10-02 13:26:05 +02:00 committed by Ingo Molnar
parent cb5ab74204
commit 5601020feb
3 changed files with 15 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
include/linux/ftrace.h
View File

@ -7,6 +7,7 @@
#include <linux/fs.h> #include <linux/fs.h>
#include <linux/init.h> #include <linux/init.h>
#include <linux/types.h> #include <linux/types.h>
#include <linux/kallsyms.h>
extern int ftrace_enabled; extern int ftrace_enabled;
extern int extern int
@ -213,7 +214,7 @@ ftrace_init_module(unsigned long *start, unsigned long *end) { }
struct boot_trace { struct boot_trace {
pid_t caller; pid_t caller;
initcall_t func; char func[KSYM_NAME_LEN];
int result; int result;
unsigned long long duration; unsigned long long duration;
ktime_t calltime; ktime_t calltime;
@ -221,10 +222,10 @@ struct boot_trace {
}; };
#ifdef CONFIG_BOOT_TRACER #ifdef CONFIG_BOOT_TRACER
extern void trace_boot(struct boot_trace *it); extern void trace_boot(struct boot_trace *it, initcall_t fn);
extern void start_boot_trace(void); extern void start_boot_trace(void);
#else #else
static inline void trace_boot(struct boot_trace *it) { } static inline void trace_boot(struct boot_trace *it, initcall_t fn) { }
static inline void start_boot_trace(void) { } static inline void start_boot_trace(void) { }
#endif #endif

3
init/main.c
View File

@ -712,7 +712,6 @@ int do_one_initcall(initcall_t fn)
if (initcall_debug) { if (initcall_debug) {
it.caller = task_pid_nr(current); it.caller = task_pid_nr(current);
it.func = fn;
printk("calling %pF @ %i\n", fn, it.caller); printk("calling %pF @ %i\n", fn, it.caller);
it.calltime = ktime_get(); it.calltime = ktime_get();
} }
@ -725,7 +724,7 @@ int do_one_initcall(initcall_t fn)
it.duration = (unsigned long long) delta.tv64 >> 20; it.duration = (unsigned long long) delta.tv64 >> 20;
printk("initcall %pF returned %d after %Ld msecs\n", fn, printk("initcall %pF returned %d after %Ld msecs\n", fn,
it.result, it.duration); it.result, it.duration);
trace_boot(&it); trace_boot(&it, fn);
} }
msgbuf[0] = 0; msgbuf[0] = 0;

14
kernel/trace/trace_boot.c
View File

@ -8,6 +8,7 @@
#include <linux/init.h> #include <linux/init.h>
#include <linux/debugfs.h> #include <linux/debugfs.h>
#include <linux/ftrace.h> #include <linux/ftrace.h>
#include <linux/kallsyms.h>
#include "trace.h" #include "trace.h"
@ -56,17 +57,19 @@ static enum print_line_t initcall_print_line(struct trace_iterator *iter)
struct timespec rettime = ktime_to_timespec(it->rettime); struct timespec rettime = ktime_to_timespec(it->rettime);
if (entry->type == TRACE_BOOT) { if (entry->type == TRACE_BOOT) {
ret = trace_seq_printf(s, "[%5ld.%06ld] calling %pF @ %i\n", ret = trace_seq_printf(s, "[%5ld.%06ld] calling %s @ %i\n",
calltime.tv_sec, calltime.tv_sec,
calltime.tv_nsec, calltime.tv_nsec,
it->func, it->caller); it->func, it->caller);
if (!ret) if (!ret)
return TRACE_TYPE_PARTIAL_LINE; return TRACE_TYPE_PARTIAL_LINE;
ret = trace_seq_printf(s, "[%5ld.%06ld] initcall %pF "
ret = trace_seq_printf(s, "[%5ld.%06ld] initcall %s "
"returned %d after %lld msecs\n", "returned %d after %lld msecs\n",
rettime.tv_sec, rettime.tv_sec,
rettime.tv_nsec, rettime.tv_nsec,
it->func, it->result, it->duration); it->func, it->result, it->duration);
if (!ret) if (!ret)
return TRACE_TYPE_PARTIAL_LINE; return TRACE_TYPE_PARTIAL_LINE;
return TRACE_TYPE_HANDLED; return TRACE_TYPE_HANDLED;
@ -83,8 +86,7 @@ struct tracer boot_tracer __read_mostly =
.print_line = initcall_print_line, .print_line = initcall_print_line,
}; };
void trace_boot(struct boot_trace *it, initcall_t fn)
void trace_boot(struct boot_trace *it)
{ {
struct ring_buffer_event *event; struct ring_buffer_event *event;
struct trace_boot *entry; struct trace_boot *entry;
@ -95,6 +97,10 @@ void trace_boot(struct boot_trace *it)
if (!trace_boot_enabled) if (!trace_boot_enabled)
return; return;
/* Get its name now since this function could
* disappear because it is in the .init section.
*/
sprint_symbol(it->func, (unsigned long)fn);
preempt_disable(); preempt_disable();
data = tr->data[smp_processor_id()]; data = tr->data[smp_processor_id()];