Re: [PATCH V2 0/4] x86/insn: perf tools: Add AVX-512 support to the instruction decoder

Adrian Hunter
2016-07-21T11:22:57Z

On 20/07/16 21:11, Masami Hiramatsu wrote:
> On Wed, 20 Jul 2016 11:30:33 +0300
> Adrian Hunter <adrian.hunter@intel.com> wrote:
>
>> Hi
>>
>> Here are patches to add support for Intel's AVX-512 instructions to the
>> instruction decoder. Also there is a patch to fix vcvtph2ps.
>>
>> AVX-512 instructions are documented in Intel Architecture Instruction Set
>> Extensions Programming Reference (February 2016).
>>
>> These patches do not conflict with Dan's removal of pcommit.
>>
>> Changes in V2:
>>
>> Split patches: separate out perf tools and new instructions test.
>> Amend decription of 'evo' annotation in x86-opcode-map.txt
>
> Thanks for updating!
>
> Acked-by: Masami Hiramatsu <mhiramat@kernel.org>
>
> for the series :)
>
> Thank you,

Thanks to Masami's quick ack's, I think it would be reasonable to queue
these patches for v4.8 unless anyone objects.

Arnaldo, if you are willing, please consider queuing these.

>
>>
>>
>> Adrian Hunter (4):
>> x86/insn: perf tools: Fix vcvtph2ps instruction decoding
>> x86/insn: Add AVX-512 support to the instruction decoder
>> perf tools: Add AVX-512 support to the instruction decoder used by Intel PT
>> perf tools: Add AVX-512 instructions to the new instructions test
>>
>> arch/x86/include/asm/inat.h | 17 +-
>> arch/x86/include/asm/insn.h | 12 +-
>> arch/x86/lib/insn.c | 18 +-
>> arch/x86/lib/x86-opcode-map.txt | 263 ++-
>> arch/x86/tools/gen-insn-attr-x86.awk | 11 +-
>> tools/perf/arch/x86/tests/insn-x86-dat-32.c | 1018 ++++++++++-
>> tools/perf/arch/x86/tests/insn-x86-dat-64.c | 940 +++++++++-
>> tools/perf/arch/x86/tests/insn-x86-dat-src.c | 1789 ++++++++++++++++++++
>> .../util/intel-pt-decoder/gen-insn-attr-x86.awk | 11 +-
>> tools/perf/util/intel-pt-decoder/inat.h | 17 +-
>> tools/perf/util/intel-pt-decoder/insn.c | 18 +-
>> tools/perf/util/intel-pt-decoder/insn.h | 12 +-
>> .../perf/util/intel-pt-decoder/x86-opcode-map.txt | 263 ++-
>> 13 files changed, 4179 insertions(+), 210 deletions(-)
>>
>>
>> Regards
>> Adrian
>
>