Commit f0f70adb authored by Peter Zijlstra's avatar Peter Zijlstra

objtool: Rename func_for_each_insn_all()

Now that func_for_each_insn() is available, rename
func_for_each_insn_all(). This gets us:

  sym_for_each_insn()  - iterate on symbol offset/len
  func_for_each_insn() - iterate on insn->func
Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: default avatarMiroslav Benes <mbenes@suse.cz>
Acked-by: default avatarJosh Poimboeuf <jpoimboe@redhat.com>
Link: https://lkml.kernel.org/r/20200324160924.083720147@infradead.org
parent dbf4aeb0
...@@ -72,7 +72,7 @@ static struct instruction *next_insn_same_func(struct objtool_file *file, ...@@ -72,7 +72,7 @@ static struct instruction *next_insn_same_func(struct objtool_file *file,
return find_insn(file, func->cfunc->sec, func->cfunc->offset); return find_insn(file, func->cfunc->sec, func->cfunc->offset);
} }
#define func_for_each_insn_all(file, func, insn) \ #define func_for_each_insn(file, func, insn) \
for (insn = find_insn(file, func->sec, func->offset); \ for (insn = find_insn(file, func->sec, func->offset); \
insn; \ insn; \
insn = next_insn_same_func(file, insn)) insn = next_insn_same_func(file, insn))
...@@ -170,7 +170,7 @@ static bool __dead_end_function(struct objtool_file *file, struct symbol *func, ...@@ -170,7 +170,7 @@ static bool __dead_end_function(struct objtool_file *file, struct symbol *func,
if (!insn->func) if (!insn->func)
return false; return false;
func_for_each_insn_all(file, func, insn) { func_for_each_insn(file, func, insn) {
empty = false; empty = false;
if (insn->type == INSN_RETURN) if (insn->type == INSN_RETURN)
...@@ -185,7 +185,7 @@ static bool __dead_end_function(struct objtool_file *file, struct symbol *func, ...@@ -185,7 +185,7 @@ static bool __dead_end_function(struct objtool_file *file, struct symbol *func,
* case, the function's dead-end status depends on whether the target * case, the function's dead-end status depends on whether the target
* of the sibling call returns. * of the sibling call returns.
*/ */
func_for_each_insn_all(file, func, insn) { func_for_each_insn(file, func, insn) {
if (is_sibling_call(insn)) { if (is_sibling_call(insn)) {
struct instruction *dest = insn->jump_dest; struct instruction *dest = insn->jump_dest;
...@@ -430,7 +430,7 @@ static void add_ignores(struct objtool_file *file) ...@@ -430,7 +430,7 @@ static void add_ignores(struct objtool_file *file)
continue; continue;
} }
func_for_each_insn_all(file, func, insn) func_for_each_insn(file, func, insn)
insn->ignore = true; insn->ignore = true;
} }
} }
...@@ -1122,7 +1122,7 @@ static void mark_func_jump_tables(struct objtool_file *file, ...@@ -1122,7 +1122,7 @@ static void mark_func_jump_tables(struct objtool_file *file,
struct instruction *insn, *last = NULL; struct instruction *insn, *last = NULL;
struct rela *rela; struct rela *rela;
func_for_each_insn_all(file, func, insn) { func_for_each_insn(file, func, insn) {
if (!last) if (!last)
last = insn; last = insn;
...@@ -1157,7 +1157,7 @@ static int add_func_jump_tables(struct objtool_file *file, ...@@ -1157,7 +1157,7 @@ static int add_func_jump_tables(struct objtool_file *file,
struct instruction *insn; struct instruction *insn;
int ret; int ret;
func_for_each_insn_all(file, func, insn) { func_for_each_insn(file, func, insn) {
if (!insn->jump_table) if (!insn->jump_table)
continue; continue;
......
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