Messages in this thread Patch in this message | | | From | 平松雅巳 / HIRAMATU,MASAMI <> | Subject | RE: [PATCH V2 3/7] x86/insn: perf tools: Pedantically tweak opcode map for MPX instructions | Date | Thu, 3 Sep 2015 16:01:36 +0000 |
| |
> From: Adrian Hunter [mailto:adrian.hunter@intel.com] > > The MPX instructions are presently not described in the SDM > opcode maps, and there are not encoding characters for bnd > registers, address method or operand type. So the kernel > opcode map is using 'Gv' for bnd registers and 'Ev' for > everything else. That is fine because the instruction > decoder does not use that information anyway, except as > an indication that there is a ModR/M byte. > > Nevertheless, in some cases the 'Gv' and 'Ev' are the wrong > way around, BNDLDX and BNDSTX have 2 operands not 3, and it > wouldn't hurt to identify the mandatory prefixes. > > This has no effect on the decoding of valid instructions, > but the addition of the mandatory prefixes will cause some > invalid instructions to error out that wouldn't have > previously. > > Note that perf tools has a copy of the instruction decoder > and provides a test for new instructions which includes MPX > instructions e.g. > > $ perf test "x86 ins" > 39: Test x86 instruction decoder - new instructions : Ok > > Or to see the details: > > $ perf test -v "x86 ins"
Thanks for fixing it :)
Acked-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
> > Signed-off-by: Adrian Hunter <adrian.hunter@intel.com> > --- > arch/x86/lib/x86-opcode-map.txt | 8 ++++++-- > tools/perf/util/intel-pt-decoder/x86-opcode-map.txt | 8 ++++++-- > 2 files changed, 12 insertions(+), 4 deletions(-) > > diff --git a/arch/x86/lib/x86-opcode-map.txt b/arch/x86/lib/x86-opcode-map.txt > index 816488c0b97e..a02a195d219c 100644 > --- a/arch/x86/lib/x86-opcode-map.txt > +++ b/arch/x86/lib/x86-opcode-map.txt > @@ -353,8 +353,12 @@ AVXcode: 1 > 17: vmovhps Mq,Vq (v1) | vmovhpd Mq,Vq (66),(v1) > 18: Grp16 (1A) > 19: > -1a: BNDCL Ev,Gv | BNDCU Ev,Gv | BNDMOV Gv,Ev | BNDLDX Gv,Ev,Gv > -1b: BNDCN Ev,Gv | BNDMOV Ev,Gv | BNDMK Gv,Ev | BNDSTX Ev,GV,Gv > +# Intel SDM opcode map does not list MPX instructions. For now using Gv for > +# bnd registers and Ev for everything else is OK because the instruction > +# decoder does not use the information except as an indication that there is > +# a ModR/M byte. > +1a: BNDCL Gv,Ev (F3) | BNDCU Gv,Ev (F2) | BNDMOV Gv,Ev (66) | BNDLDX Gv,Ev > +1b: BNDCN Gv,Ev (F2) | BNDMOV Ev,Gv (66) | BNDMK Gv,Ev (F3) | BNDSTX Ev,Gv > 1c: > 1d: > 1e: > 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 816488c0b97e..a02a195d219c 100644 > --- a/tools/perf/util/intel-pt-decoder/x86-opcode-map.txt > +++ b/tools/perf/util/intel-pt-decoder/x86-opcode-map.txt > @@ -353,8 +353,12 @@ AVXcode: 1 > 17: vmovhps Mq,Vq (v1) | vmovhpd Mq,Vq (66),(v1) > 18: Grp16 (1A) > 19: > -1a: BNDCL Ev,Gv | BNDCU Ev,Gv | BNDMOV Gv,Ev | BNDLDX Gv,Ev,Gv > -1b: BNDCN Ev,Gv | BNDMOV Ev,Gv | BNDMK Gv,Ev | BNDSTX Ev,GV,Gv > +# Intel SDM opcode map does not list MPX instructions. For now using Gv for > +# bnd registers and Ev for everything else is OK because the instruction > +# decoder does not use the information except as an indication that there is > +# a ModR/M byte. > +1a: BNDCL Gv,Ev (F3) | BNDCU Gv,Ev (F2) | BNDMOV Gv,Ev (66) | BNDLDX Gv,Ev > +1b: BNDCN Gv,Ev (F2) | BNDMOV Ev,Gv (66) | BNDMK Gv,Ev (F3) | BNDSTX Ev,Gv > 1c: > 1d: > 1e: > -- > 1.9.1
| |