Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 30 Sep 2020 10:19:51 -0500
From:   Josh Poimboeuf <jpoimboe@redhat.com>
To:     Vasily Gorbik <gor@linux.ibm.com>
Cc:     Peter Zijlstra <peterz@infradead.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>,
        x86@kernel.org, "H. Peter Anvin" <hpa@zytor.com>,
        Miroslav Benes <mbenes@suse.cz>,
        Alexandre Chartre <alexandre.chartre@oracle.com>,
        Julien Thierry <jthierry@redhat.com>,
        linux-kernel@vger.kernel.org
Subject: Re: [RFC PATCH v2 1/2] objtool: x86 instruction decoder and big
 endian cross compiles
Message-ID: <20200930151951.wwsleh6m4buybqhf@treble>
References: <20200930101251.GX2628@hirez.programming.kicks-ass.net>
 <cover.thread-ca645c.your-ad-here.call-01601468251-ext-3723@work.hours>
 <patch-1.thread-ca645c.git-a25fdca3bf6d.your-ad-here.call-01601468251-ext-3723@work.hours>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <patch-1.thread-ca645c.git-a25fdca3bf6d.your-ad-here.call-01601468251-ext-3723@work.hours>
Precedence: bulk

On Wed, Sep 30, 2020 at 02:24:19PM +0200, Vasily Gorbik wrote:
> From: Martin Schwidefsky <schwidefsky@de.ibm.com>
> 
> Make the x86 instruction decoder of the objtool usable on big endian
> machines. This is useful for compile tests on non x86, big endian
> hardware.
> 
> Co-developed-by: Vasily Gorbik <gor@linux.ibm.com>
> [ gor: more endianness problems findings fixes / rebasing ]
> Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
> Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
> ---
>  arch/x86/include/asm/insn.h       | 42 ++++++++++++++
>  arch/x86/lib/insn.c               | 95 +++++++++++++------------------
>  tools/arch/x86/include/asm/insn.h | 42 ++++++++++++++
>  tools/arch/x86/lib/insn.c         | 95 +++++++++++++------------------
>  tools/objtool/arch/x86/special.c  |  2 +-
>  tools/objtool/check.c             |  4 +-
>  tools/objtool/elf.c               | 34 ++++++-----
>  7 files changed, 188 insertions(+), 126 deletions(-)

Can the insn changes be done in a separate patch from the objtool
changes?

Also, 'Masami Hiramatsu <mhiramat@kernel.org>' should be on Cc for the
insn changes since he wrote the original code.

-- 
Josh