From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932310AbcLNKsR (ORCPT ); Wed, 14 Dec 2016 05:48:17 -0500 Received: from mail-wj0-f195.google.com ([209.85.210.195]:34961 "EHLO mail-wj0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755789AbcLNKsP (ORCPT ); Wed, 14 Dec 2016 05:48:15 -0500 Subject: Re: [PATCH] objtool: fix build To: Josh Poimboeuf References: <20161205124137.343-1-jslaby@suse.cz> <20161205124137.343-2-jslaby@suse.cz> <20161205204009.dw53cip5wyt5mcav@treble> Cc: linux-kernel@vger.kernel.org From: Jiri Slaby Message-ID: <2ca6160a-9b1b-91bb-edf7-c5a9e60bfded@suse.cz> Date: Wed, 14 Dec 2016 11:48:12 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.0 MIME-Version: 1.0 In-Reply-To: <20161205204009.dw53cip5wyt5mcav@treble> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/05/2016, 09:40 PM, Josh Poimboeuf wrote: > On Mon, Dec 05, 2016 at 01:41:37PM +0100, Jiri Slaby wrote: >> 0x8d opcode was handled twice. Fixed. >> >> Signed-off-by: Jiri Slaby > > 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). You must have the fix from upstream first to have this applied: commit 69042bf2001b44e81cd86ab11a4637b9d9a14c5a Author: Jiri Slaby Date: Mon Dec 5 11:55:51 2016 +0100 objtool: Fix bytes check of lea's rex_prefix Thanks. >> --- >> 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 >> > -- js suse labs