From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753100AbcGTAfJ (ORCPT ); Tue, 19 Jul 2016 20:35:09 -0400 Received: from mail.kernel.org ([198.145.29.136]:39348 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752477AbcGTAfG (ORCPT ); Tue, 19 Jul 2016 20:35:06 -0400 Date: Wed, 20 Jul 2016 09:35:00 +0900 From: Masami Hiramatsu To: Adrian Hunter Cc: linux-kernel@vger.kernel.org, Arnaldo Carvalho de Melo , Jiri Olsa , Dan Williams , Masami Hiramatsu , Ingo Molnar , "H. Peter Anvin" , Thomas Gleixner , Andy Lutomirski , X86 ML Subject: Re: [PATCH 1/2] x86/insn: perf tools: Fix vcvtph2ps instruction decoding Message-Id: <20160720093500.a947a957dc39c2a004581824@kernel.org> In-Reply-To: <1468932408-9868-2-git-send-email-adrian.hunter@intel.com> References: <1468932408-9868-1-git-send-email-adrian.hunter@intel.com> <1468932408-9868-2-git-send-email-adrian.hunter@intel.com> X-Mailer: Sylpheed 3.5.0 (GTK+ 2.24.30; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 19 Jul 2016 15:46:47 +0300 Adrian Hunter wrote: > vcvtph2ps does not have an immediate operand, so remove the erroneous 'Ib' > from its opcode map entry. Add vcvtph2ps to the perf tools new > instructions test to verify it. Hmm, Intel SDM Vol.2D Table A-4 shows vcvtph2ps has Ib (Vol.2C Figure 5-6. also has same mistake in title), but yes, instruction set reference of VCVTPH2PS shows that has no immediate. So, I guess OPCODE MAP should be fixed. Looks good to me:) Acked-by: Masami Hiramatsu Thank you! > > Signed-off-by: Adrian Hunter > --- > arch/x86/lib/x86-opcode-map.txt | 2 +- > tools/perf/arch/x86/tests/insn-x86-dat-32.c | 10 ++++++---- > tools/perf/arch/x86/tests/insn-x86-dat-64.c | 10 ++++++---- > tools/perf/arch/x86/tests/insn-x86-dat-src.c | 4 ++++ > tools/perf/util/intel-pt-decoder/x86-opcode-map.txt | 2 +- > 5 files changed, 18 insertions(+), 10 deletions(-) > > diff --git a/arch/x86/lib/x86-opcode-map.txt b/arch/x86/lib/x86-opcode-map.txt > index d388de72eaca..28082de46f0d 100644 > --- a/arch/x86/lib/x86-opcode-map.txt > +++ b/arch/x86/lib/x86-opcode-map.txt > @@ -629,7 +629,7 @@ AVXcode: 2 > 10: pblendvb Vdq,Wdq (66) > 11: > 12: > -13: vcvtph2ps Vx,Wx,Ib (66),(v) > +13: vcvtph2ps Vx,Wx (66),(v) > 14: blendvps Vdq,Wdq (66) > 15: blendvpd Vdq,Wdq (66) > 16: vpermps Vqq,Hqq,Wqq (66),(v) > diff --git a/tools/perf/arch/x86/tests/insn-x86-dat-32.c b/tools/perf/arch/x86/tests/insn-x86-dat-32.c > index 3b491cfe204e..ca08e6eb988f 100644 > --- a/tools/perf/arch/x86/tests/insn-x86-dat-32.c > +++ b/tools/perf/arch/x86/tests/insn-x86-dat-32.c > @@ -6,6 +6,8 @@ > > {{0x0f, 0x31, }, 2, 0, "", "", > "0f 31 \trdtsc ",}, > +{{0xc4, 0xe2, 0x7d, 0x13, 0xeb, }, 5, 0, "", "", > +"c4 e2 7d 13 eb \tvcvtph2ps %xmm3,%ymm5",}, > {{0xf3, 0x0f, 0x1b, 0x00, }, 4, 0, "", "", > "f3 0f 1b 00 \tbndmk (%eax),%bnd0",}, > {{0xf3, 0x0f, 0x1b, 0x05, 0x78, 0x56, 0x34, 0x12, }, 8, 0, "", "", > @@ -309,19 +311,19 @@ > {{0x0f, 0x1b, 0x84, 0x08, 0x78, 0x56, 0x34, 0x12, }, 8, 0, "", "", > "0f 1b 84 08 78 56 34 12 \tbndstx %bnd0,0x12345678(%eax,%ecx,1)",}, > {{0xf2, 0xe8, 0xfc, 0xff, 0xff, 0xff, }, 6, 0xfffffffc, "call", "unconditional", > -"f2 e8 fc ff ff ff \tbnd call 3c3 ",}, > +"f2 e8 fc ff ff ff \tbnd call 3c8 ",}, > {{0xf2, 0xff, 0x10, }, 3, 0, "call", "indirect", > "f2 ff 10 \tbnd call *(%eax)",}, > {{0xf2, 0xc3, }, 2, 0, "ret", "indirect", > "f2 c3 \tbnd ret ",}, > {{0xf2, 0xe9, 0xfc, 0xff, 0xff, 0xff, }, 6, 0xfffffffc, "jmp", "unconditional", > -"f2 e9 fc ff ff ff \tbnd jmp 3ce ",}, > +"f2 e9 fc ff ff ff \tbnd jmp 3d3 ",}, > {{0xf2, 0xe9, 0xfc, 0xff, 0xff, 0xff, }, 6, 0xfffffffc, "jmp", "unconditional", > -"f2 e9 fc ff ff ff \tbnd jmp 3d4 ",}, > +"f2 e9 fc ff ff ff \tbnd jmp 3d9 ",}, > {{0xf2, 0xff, 0x21, }, 3, 0, "jmp", "indirect", > "f2 ff 21 \tbnd jmp *(%ecx)",}, > {{0xf2, 0x0f, 0x85, 0xfc, 0xff, 0xff, 0xff, }, 7, 0xfffffffc, "jcc", "conditional", > -"f2 0f 85 fc ff ff ff \tbnd jne 3de ",}, > +"f2 0f 85 fc ff ff ff \tbnd jne 3e3 ",}, > {{0x0f, 0x3a, 0xcc, 0xc1, 0x00, }, 5, 0, "", "", > "0f 3a cc c1 00 \tsha1rnds4 $0x0,%xmm1,%xmm0",}, > {{0x0f, 0x3a, 0xcc, 0xd7, 0x91, }, 5, 0, "", "", > diff --git a/tools/perf/arch/x86/tests/insn-x86-dat-64.c b/tools/perf/arch/x86/tests/insn-x86-dat-64.c > index 4fe7cce179c4..262d9d25a6fa 100644 > --- a/tools/perf/arch/x86/tests/insn-x86-dat-64.c > +++ b/tools/perf/arch/x86/tests/insn-x86-dat-64.c > @@ -6,6 +6,8 @@ > > {{0x0f, 0x31, }, 2, 0, "", "", > "0f 31 \trdtsc ",}, > +{{0xc4, 0xe2, 0x7d, 0x13, 0xeb, }, 5, 0, "", "", > +"c4 e2 7d 13 eb \tvcvtph2ps %xmm3,%ymm5",}, > {{0xf3, 0x0f, 0x1b, 0x00, }, 4, 0, "", "", > "f3 0f 1b 00 \tbndmk (%rax),%bnd0",}, > {{0xf3, 0x41, 0x0f, 0x1b, 0x00, }, 5, 0, "", "", > @@ -325,19 +327,19 @@ > {{0x0f, 0x1b, 0x84, 0x08, 0x78, 0x56, 0x34, 0x12, }, 8, 0, "", "", > "0f 1b 84 08 78 56 34 12 \tbndstx %bnd0,0x12345678(%rax,%rcx,1)",}, > {{0xf2, 0xe8, 0x00, 0x00, 0x00, 0x00, }, 6, 0, "call", "unconditional", > -"f2 e8 00 00 00 00 \tbnd callq 3f6 ",}, > +"f2 e8 00 00 00 00 \tbnd callq 3fb ",}, > {{0x67, 0xf2, 0xff, 0x10, }, 4, 0, "call", "indirect", > "67 f2 ff 10 \tbnd callq *(%eax)",}, > {{0xf2, 0xc3, }, 2, 0, "ret", "indirect", > "f2 c3 \tbnd retq ",}, > {{0xf2, 0xe9, 0x00, 0x00, 0x00, 0x00, }, 6, 0, "jmp", "unconditional", > -"f2 e9 00 00 00 00 \tbnd jmpq 402 ",}, > +"f2 e9 00 00 00 00 \tbnd jmpq 407 ",}, > {{0xf2, 0xe9, 0x00, 0x00, 0x00, 0x00, }, 6, 0, "jmp", "unconditional", > -"f2 e9 00 00 00 00 \tbnd jmpq 408 ",}, > +"f2 e9 00 00 00 00 \tbnd jmpq 40d ",}, > {{0x67, 0xf2, 0xff, 0x21, }, 4, 0, "jmp", "indirect", > "67 f2 ff 21 \tbnd jmpq *(%ecx)",}, > {{0xf2, 0x0f, 0x85, 0x00, 0x00, 0x00, 0x00, }, 7, 0, "jcc", "conditional", > -"f2 0f 85 00 00 00 00 \tbnd jne 413 ",}, > +"f2 0f 85 00 00 00 00 \tbnd jne 418 ",}, > {{0x0f, 0x3a, 0xcc, 0xc1, 0x00, }, 5, 0, "", "", > "0f 3a cc c1 00 \tsha1rnds4 $0x0,%xmm1,%xmm0",}, > {{0x0f, 0x3a, 0xcc, 0xd7, 0x91, }, 5, 0, "", "", > diff --git a/tools/perf/arch/x86/tests/insn-x86-dat-src.c b/tools/perf/arch/x86/tests/insn-x86-dat-src.c > index 41b1b1c62660..3cd677513e9e 100644 > --- a/tools/perf/arch/x86/tests/insn-x86-dat-src.c > +++ b/tools/perf/arch/x86/tests/insn-x86-dat-src.c > @@ -19,6 +19,10 @@ int main(void) > /* Following line is a marker for the awk script - do not change */ > asm volatile("rdtsc"); /* Start here */ > > + /* Test fix for vcvtph2ps in x86-opcode-map.txt */ > + > + asm volatile("vcvtph2ps %xmm3,%ymm5"); > + > #ifdef __x86_64__ > > /* bndmk m64, bnd */ > diff --git a/tools/perf/util/intel-pt-decoder/x86-opcode-map.txt b/tools/perf/util/intel-pt-decoder/x86-opcode-map.txt > index d388de72eaca..28082de46f0d 100644 > --- a/tools/perf/util/intel-pt-decoder/x86-opcode-map.txt > +++ b/tools/perf/util/intel-pt-decoder/x86-opcode-map.txt > @@ -629,7 +629,7 @@ AVXcode: 2 > 10: pblendvb Vdq,Wdq (66) > 11: > 12: > -13: vcvtph2ps Vx,Wx,Ib (66),(v) > +13: vcvtph2ps Vx,Wx (66),(v) > 14: blendvps Vdq,Wdq (66) > 15: blendvpd Vdq,Wdq (66) > 16: vpermps Vqq,Hqq,Wqq (66),(v) > -- > 1.9.1 > -- Masami Hiramatsu