Commit 706c81f8 authored by Steven Rostedt's avatar Steven Rostedt Committed by Steven Rostedt

ftrace: Remove extra helper functions

The ftrace_record_ip() and ftrace_alloc_dyn_node() were from the
time of the ftrace daemon. Although they were still used, they
still make things a bit more complex than necessary.

Move the code into the one function that uses it, and remove the
helper functions.
Signed-off-by: default avatarSteven Rostedt <rostedt@goodmis.org>
parent 9fd49328
...@@ -1520,35 +1520,6 @@ static void ftrace_hash_rec_enable(struct ftrace_ops *ops, ...@@ -1520,35 +1520,6 @@ static void ftrace_hash_rec_enable(struct ftrace_ops *ops,
__ftrace_hash_rec_update(ops, filter_hash, 1); __ftrace_hash_rec_update(ops, filter_hash, 1);
} }
static struct dyn_ftrace *ftrace_alloc_dyn_node(unsigned long ip)
{
if (ftrace_pages->index == ftrace_pages->size) {
/* We should have allocated enough */
if (WARN_ON(!ftrace_pages->next))
return NULL;
ftrace_pages = ftrace_pages->next;
}
return &ftrace_pages->records[ftrace_pages->index++];
}
static struct dyn_ftrace *
ftrace_record_ip(unsigned long ip)
{
struct dyn_ftrace *rec;
if (ftrace_disabled)
return NULL;
rec = ftrace_alloc_dyn_node(ip);
if (!rec)
return NULL;
rec->ip = ip;
return rec;
}
static void print_ip_ins(const char *fmt, unsigned char *p) static void print_ip_ins(const char *fmt, unsigned char *p)
{ {
int i; int i;
...@@ -3693,7 +3664,9 @@ static int ftrace_process_locs(struct module *mod, ...@@ -3693,7 +3664,9 @@ static int ftrace_process_locs(struct module *mod,
unsigned long *start, unsigned long *start,
unsigned long *end) unsigned long *end)
{ {
struct ftrace_page *start_pg;
struct ftrace_page *pg; struct ftrace_page *pg;
struct dyn_ftrace *rec;
unsigned long count; unsigned long count;
unsigned long *p; unsigned long *p;
unsigned long addr; unsigned long addr;
...@@ -3708,8 +3681,8 @@ static int ftrace_process_locs(struct module *mod, ...@@ -3708,8 +3681,8 @@ static int ftrace_process_locs(struct module *mod,
sort(start, count, sizeof(*start), sort(start, count, sizeof(*start),
ftrace_cmp_ips, ftrace_swap_ips); ftrace_cmp_ips, ftrace_swap_ips);
pg = ftrace_allocate_pages(count); start_pg = ftrace_allocate_pages(count);
if (!pg) if (!start_pg)
return -ENOMEM; return -ENOMEM;
mutex_lock(&ftrace_lock); mutex_lock(&ftrace_lock);
...@@ -3722,7 +3695,7 @@ static int ftrace_process_locs(struct module *mod, ...@@ -3722,7 +3695,7 @@ static int ftrace_process_locs(struct module *mod,
if (!mod) { if (!mod) {
WARN_ON(ftrace_pages || ftrace_pages_start); WARN_ON(ftrace_pages || ftrace_pages_start);
/* First initialization */ /* First initialization */
ftrace_pages = ftrace_pages_start = pg; ftrace_pages = ftrace_pages_start = start_pg;
} else { } else {
if (!ftrace_pages) if (!ftrace_pages)
goto out; goto out;
...@@ -3733,11 +3706,11 @@ static int ftrace_process_locs(struct module *mod, ...@@ -3733,11 +3706,11 @@ static int ftrace_process_locs(struct module *mod,
ftrace_pages = ftrace_pages->next; ftrace_pages = ftrace_pages->next;
} }
ftrace_pages->next = pg; ftrace_pages->next = start_pg;
ftrace_pages = pg;
} }
p = start; p = start;
pg = start_pg;
while (p < end) { while (p < end) {
addr = ftrace_call_adjust(*p++); addr = ftrace_call_adjust(*p++);
/* /*
...@@ -3748,12 +3721,26 @@ static int ftrace_process_locs(struct module *mod, ...@@ -3748,12 +3721,26 @@ static int ftrace_process_locs(struct module *mod,
*/ */
if (!addr) if (!addr)
continue; continue;
if (!ftrace_record_ip(addr))
break; if (pg->index == pg->size) {
/* We should have allocated enough */
if (WARN_ON(!pg->next))
break;
pg = pg->next;
}
rec = &pg->records[pg->index++];
rec->ip = addr;
} }
/* We should have used all pages */
WARN_ON(pg->next);
/* Assign the last page to ftrace_pages */
ftrace_pages = pg;
/* These new locations need to be initialized */ /* These new locations need to be initialized */
ftrace_new_pgs = pg; ftrace_new_pgs = start_pg;
/* /*
* We only need to disable interrupts on start up * We only need to disable interrupts on start up
......
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