Commit c8cdd2be authored by Rasmus Villemoes's avatar Rasmus Villemoes Committed by Linus Torvalds

init/main.c: simplify initcall_blacklisted()

Using kasprintf to get the function name makes us look up the name
twice, along with all the vsnprintf overhead of parsing the format
string etc.  It also means there is an allocation failure case to deal
with.  Since symbol_string in vsprintf.c would anyway allocate an array
of size KSYM_SYMBOL_LEN on the stack, that might as well be done up
here.

Moreover, since this is a debug feature and the blacklisted_initcalls
list is usually empty, we might as well test that and thus avoid looking
up the symbol name even once in the common case.
Signed-off-by: default avatarRasmus Villemoes <linux@rasmusvillemoes.dk>
Acked-by: default avatarRusty Russell <rusty@rustcorp.com.au>
Acked-by: default avatarPrarit Bhargava <prarit@redhat.com>
Cc: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 4108124f
...@@ -706,21 +706,20 @@ static int __init initcall_blacklist(char *str) ...@@ -706,21 +706,20 @@ static int __init initcall_blacklist(char *str)
static bool __init_or_module initcall_blacklisted(initcall_t fn) static bool __init_or_module initcall_blacklisted(initcall_t fn)
{ {
struct blacklist_entry *entry; struct blacklist_entry *entry;
char *fn_name; char fn_name[KSYM_SYMBOL_LEN];
fn_name = kasprintf(GFP_KERNEL, "%pf", fn); if (list_empty(&blacklisted_initcalls))
if (!fn_name)
return false; return false;
sprint_symbol_no_offset(fn_name, (unsigned long)fn);
list_for_each_entry(entry, &blacklisted_initcalls, next) { list_for_each_entry(entry, &blacklisted_initcalls, next) {
if (!strcmp(fn_name, entry->buf)) { if (!strcmp(fn_name, entry->buf)) {
pr_debug("initcall %s blacklisted\n", fn_name); pr_debug("initcall %s blacklisted\n", fn_name);
kfree(fn_name);
return true; return true;
} }
} }
kfree(fn_name);
return false; return false;
} }
#else #else
......
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