Commit dd1553b8 authored by Masahiro Yamada's avatar Masahiro Yamada

scripts/kallsyms: decrease expand_symbol() / cleanup_symbol_name() calls

Currently, expand_symbol() is called many times to get the uncompressed
symbol names for sorting, and also for adding comments.

With the output order shuffled in the previous commit, the symbol data
are now written in the following order:

 (1) kallsyms_num_syms
 (2) kallsyms_names                         <-- need compressed names
 (3) kallsyms_markers
 (4) kallsyms_token_table
 (5) kallsyms_token_index
 (6) kallsyms_addressed / kallsyms_offsets  <-- need uncompressed names (for commenting)
 (7) kallsyms_relative_base
 (8) kallsyms_seq_of_names                  <-- need uncompressed names (for sorting)

The compressed names are only needed by (2).

Call expand_symbol() between (2) and (3) to restore the original symbol
names. This requires just one expand_symbol() call for each symbol.

Call cleanup_symbol_name() between (7) and (8) instead of during sorting.
It is allowed to overwrite the ->sym field because (8) just outputs the
index instead of the name of each symbol. Again, this requires just one
cleanup_symbol_name() call for each symbol.

This refactoring makes it ~30% faster.

[Before]

  $ time scripts/kallsyms --all-symbols --absolute-percpu --base-relative \
    .tmp_vmlinux.kallsyms2.syms >/dev/null

  real    0m1.027s
  user    0m1.010s
  sys     0m0.016s

[After]

  $ time scripts/kallsyms --all-symbols --absolute-percpu --base-relative \
    .tmp_vmlinux.kallsyms2.syms >/dev/null

  real    0m0.717s
  user    0m0.717s
  sys     0m0.000s
Signed-off-by: default avatarMasahiro Yamada <masahiroy@kernel.org>
parent 404bad70
...@@ -335,19 +335,10 @@ static int symbol_absolute(const struct sym_entry *s) ...@@ -335,19 +335,10 @@ static int symbol_absolute(const struct sym_entry *s)
return s->percpu_absolute; return s->percpu_absolute;
} }
static char * s_name(char *buf)
{
/* Skip the symbol type */
return buf + 1;
}
static void cleanup_symbol_name(char *s) static void cleanup_symbol_name(char *s)
{ {
char *p; char *p;
if (!lto_clang)
return;
/* /*
* ASCII[.] = 2e * ASCII[.] = 2e
* ASCII[0-9] = 30,39 * ASCII[0-9] = 30,39
...@@ -366,16 +357,10 @@ static void cleanup_symbol_name(char *s) ...@@ -366,16 +357,10 @@ static void cleanup_symbol_name(char *s)
static int compare_names(const void *a, const void *b) static int compare_names(const void *a, const void *b)
{ {
int ret; int ret;
char sa_namebuf[KSYM_NAME_LEN];
char sb_namebuf[KSYM_NAME_LEN];
const struct sym_entry *sa = *(const struct sym_entry **)a; const struct sym_entry *sa = *(const struct sym_entry **)a;
const struct sym_entry *sb = *(const struct sym_entry **)b; const struct sym_entry *sb = *(const struct sym_entry **)b;
expand_symbol(sa->sym, sa->len, sa_namebuf); ret = strcmp(sym_name(sa), sym_name(sb));
expand_symbol(sb->sym, sb->len, sb_namebuf);
cleanup_symbol_name(s_name(sa_namebuf));
cleanup_symbol_name(s_name(sb_namebuf));
ret = strcmp(s_name(sa_namebuf), s_name(sb_namebuf));
if (!ret) { if (!ret) {
if (sa->addr > sb->addr) if (sa->addr > sb->addr)
return 1; return 1;
...@@ -464,6 +449,15 @@ static void write_src(void) ...@@ -464,6 +449,15 @@ static void write_src(void)
} }
printf("\n"); printf("\n");
/*
* Now that we wrote out the compressed symbol names, restore the
* original names, which are needed in some of the later steps.
*/
for (i = 0; i < table_cnt; i++) {
expand_symbol(table[i]->sym, table[i]->len, buf);
strcpy((char *)table[i]->sym, buf);
}
output_label("kallsyms_markers"); output_label("kallsyms_markers");
for (i = 0; i < ((table_cnt + 255) >> 8); i++) for (i = 0; i < ((table_cnt + 255) >> 8); i++)
printf("\t.long\t%u\n", markers[i]); printf("\t.long\t%u\n", markers[i]);
...@@ -520,8 +514,7 @@ static void write_src(void) ...@@ -520,8 +514,7 @@ static void write_src(void)
table[i]->addr); table[i]->addr);
exit(EXIT_FAILURE); exit(EXIT_FAILURE);
} }
expand_symbol(table[i]->sym, table[i]->len, buf); printf("\t.long\t%#x /* %s */\n", (int)offset, table[i]->sym);
printf("\t.long\t%#x /* %s */\n", (int)offset, buf);
} else if (!symbol_absolute(table[i])) { } else if (!symbol_absolute(table[i])) {
output_address(table[i]->addr); output_address(table[i]->addr);
} else { } else {
...@@ -536,6 +529,10 @@ static void write_src(void) ...@@ -536,6 +529,10 @@ static void write_src(void)
printf("\n"); printf("\n");
} }
if (lto_clang)
for (i = 0; i < table_cnt; i++)
cleanup_symbol_name((char *)table[i]->sym);
sort_symbols_by_name(); sort_symbols_by_name();
output_label("kallsyms_seqs_of_names"); output_label("kallsyms_seqs_of_names");
for (i = 0; i < table_cnt; i++) for (i = 0; i < table_cnt; i++)
......
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