RepoMirrors
/
kpatch
mirror of https://github.com/dynup/kpatch
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1346 from jpoimboe/locals-match-empty 

lookup: skip duplicate local symbol table check for files without locals
This commit is contained in:
Josh Poimboeuf 2023-07-20 18:12:15 -05:00 committed by GitHub
parent f8d2534210 75927e7bbb
commit 8a00d03bb8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 27 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
kpatch-build/lookup.c
View File

@ -165,6 +165,25 @@ static bool locals_match(struct lookup_table *table, int idx,
return true; return true;
} }
static bool file_has_locals(struct symbol *file_sym, struct list_head *sym_list)
{
struct symbol *sym = file_sym;
list_for_each_entry_continue(sym, sym_list, list) {
if (sym->type == STT_FILE)
break;
if (sym->type != STB_LOCAL)
continue;
if (maybe_discarded_sym(sym->name))
continue;
if (sym->type == STT_FUNC || sym->type == STT_OBJECT)
return true;
}
return false;
}
static void find_local_syms(struct lookup_table *table, struct symbol *file_sym, static void find_local_syms(struct lookup_table *table, struct symbol *file_sym,
struct list_head *sym_list) struct list_head *sym_list)
{ {
@ -184,6 +203,14 @@ static void find_local_syms(struct lookup_table *table, struct symbol *file_sym,
file_sym->name, table->objname); file_sym->name, table->objname);
lookup_table_file_sym = sym; lookup_table_file_sym = sym;
if (!file_has_locals(file_sym, sym_list)) {
/*
* If the file doesn't have local symbols, any empty
* match will do. Skip the duplicate check.
*/
break;
}
} }
if (!lookup_table_file_sym) if (!lookup_table_file_sym)