diff options
| author |   Julien Thierry <jthierry@redhat.com> | 2020-03-27 15:28:42 +0000 |
|---|---|---|
| committer |   Peter Zijlstra <peterz@infradead.org> | 2020-04-30 20:14:31 +0200 |
| commit | 9e98d62aa7ea1375052895650f3e6d362336c5c9 (patch) | |
| tree | 1a4ee6266ff4f2f579a6d9cfa69cc1ce4cfca5f1 /tools/objtool | |
| parent | objtool: Constify arch_decode_instruction() (diff) | |
| download | wireguard-linux-9e98d62aa7ea1375052895650f3e6d362336c5c9.tar.xz wireguard-linux-9e98d62aa7ea1375052895650f3e6d362336c5c9.zip | |
objtool: Remove check preventing branches within alternative
While jumping from outside an alternative region to the middle of an
alternative region is very likely wrong, jumping from an alternative
region into the same region is valid. It is a common pattern on arm64.
The first pattern is unlikely to happen in practice and checking only
for this adds a lot of complexity.
Just remove the current check.
Suggested-by: Josh Poimboeuf <jpoimboe@redhat.com>
Signed-off-by: Julien Thierry <jthierry@redhat.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Miroslav Benes <mbenes@suse.cz>
Link: https://lkml.kernel.org/r/20200327152847.15294-6-jthierry@redhat.com
Diffstat (limited to 'tools/objtool')
| -rw-r--r-- | tools/objtool/check.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/tools/objtool/check.c b/tools/objtool/check.c index 12e2aea42bb2..cc52da61eda3 100644 --- a/tools/objtool/check.c +++ b/tools/objtool/check.c @@ -2162,12 +2162,6 @@ static int validate_branch(struct objtool_file *file, struct symbol *func, sec = insn->sec; - if (insn->alt_group && list_empty(&insn->alts)) { - WARN_FUNC("don't know how to handle branch to middle of alternative instruction group", - sec, insn->offset); - return 1; - } - while (1) { next_insn = next_insn_same_sec(file, insn); |