lkml.org 
[lkml]   [2016]   [Dec]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] objtool: fix build
On Mon, Dec 05, 2016 at 01:41:37PM +0100, Jiri Slaby wrote:
> 0x8d opcode was handled twice. Fixed.
>
> Signed-off-by: Jiri Slaby <jslaby@suse.cz>

I applied the other patch to the objtool-dwarf branch, but this one
doesn't apply (the branch already has the changes this patch is trying
to make).

> ---
> tools/objtool/arch/x86/decode.c | 23 ++++++++++++++---------
> 1 file changed, 14 insertions(+), 9 deletions(-)
>
> diff --git a/tools/objtool/arch/x86/decode.c b/tools/objtool/arch/x86/decode.c
> index ca4ede5ddd8c..6011ccbf9384 100644
> --- a/tools/objtool/arch/x86/decode.c
> +++ b/tools/objtool/arch/x86/decode.c
> @@ -291,6 +291,20 @@ int arch_decode_instruction(struct elf *elf, struct section *sec,
> break;
> }
>
> + if (insn.rex_prefix.nbytes &&
> + insn.rex_prefix.bytes[0] == 0x48 &&
> + insn.modrm.nbytes && insn.modrm.bytes[0] == 0x2c &&
> + insn.sib.nbytes && insn.sib.bytes[0] == 0x24) {
> + /* lea %(rsp), %rbp */
> + *type = INSN_STACK;
> + op->dest.type = OP_DEST_REG;
> + op->dest.reg = CFI_BP;
> + op->src.type = OP_SRC_ADD;
> + op->src.reg = CFI_SP;
> + op->src.offset = 0;
> + break;
> + }
> +
> if (insn.rex_prefix.nbytes && insn.modrm.nbytes &&
> insn.sib.nbytes && insn.rex_prefix.bytes[0] == 0x4c &&
> insn.modrm.bytes[0] == 0x54 && insn.sib.bytes[0] == 0x24 &&
> @@ -342,15 +356,6 @@ int arch_decode_instruction(struct elf *elf, struct section *sec,
> op->src.type = OP_SRC_POP;
> break;
>
> - case 0x8d:
> - if (insn.rex_prefix.nbytes &&
> - insn.rex_prefix.bytes[0] == 0x48 &&
> - insn.modrm.nbytes && insn.modrm.bytes[0] == 0x2c &&
> - insn.sib.nbytes && insn.sib.bytes[0] == 0x24)
> - /* lea %(rsp), %rbp */
> - *type = INSN_FP_SETUP;
> - break;
> -
> case 0x90:
> *type = INSN_NOP;
> break;
> --
> 2.11.0
>

--
Josh

\
 
 \ /
  Last update: 2016-12-05 21:40    [W:0.057 / U:0.424 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site