On Thu, Jul 30, 2020 at 10:41:40AM +0100, Julien Thierry wrote: > +struct objtool_file *objtool_setup_file(const char *_objname) > +{ > + if (objname) { > + if (strcmp(objname, _objname)) { > + WARN("won't handle more than one file at a time"); > + return NULL; > + } > + return &file; > + } > + objname = _objname; > + > + file.elf = elf_open_read(objname, O_RDWR); > + if (!file.elf) > + return NULL; > + > + INIT_LIST_HEAD(&file.insn_list); > + hash_init(file.insn_hash); > + file.c_file = find_section_by_name(file.elf, ".comment"); > + file.ignore_unreachables = no_unreachable; > + file.hints = false; > + > + return &file; > +}
How about calling it objtool_open_read()? It's (sort of) a wrapper around elf_open_read(). -- Josh