lkml.org 
[lkml]   [2022]   [Jan]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 0/6] x86/insn: Add instructions to instruction decoder
Em Mon, Jan 03, 2022 at 03:22:59PM -0300, Arnaldo Carvalho de Melo escreveu:
> Em Mon, Jan 03, 2022 at 05:40:00PM +0200, Adrian Hunter escreveu:
> > On 03/01/2022 17:01, Arnaldo Carvalho de Melo wrote:
> > > Em Mon, Jan 03, 2022 at 11:56:08AM -0300, Arnaldo Carvalho de Melo escreveu:
> > > So since this needs rebasing, could you please split it into two
> > > patchsets, one for tools/ and another for arch and send two batches?
> > > This way I'll pick the tools/ one, the x86 maintainers the other bits.

> > > Otherwise the x86 maintainers would have to test if the tools/ bits
> > > works as expected, etc.

> > It seems not to be happening for me. What tree/branch is it?

> tmp.perf/core

> https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/log/?h=tmp.perf/core

> I merged torvalds/master today, but it should work as that file was last
> changed looong ago:

> ⬢[acme@toolbox perf]$ git log arch/x86/lib/x86-opcode-map.txt | head
> commit 5790921bc18b1eb5c0c61371e31114fd4c4b0154
> Author: Yu-cheng Yu <yu-cheng.yu@intel.com>
> Date: Tue Feb 4 09:14:24 2020 -0800
>
> x86/insn: Add Control-flow Enforcement (CET) instructions to the opcode map
>
> Add the following CET instructions to the opcode map:
>
> INCSSP:
> Increment Shadow Stack pointer (SSP).
> ⬢[acme@toolbox perf]$
>
> Perhaps related to b4? I'll try updating it and retrying...

Now it should work:

⬢[acme@toolbox perf]$ b4 am -ctsl --cc-trailers c7ac518e-82fb-eb51-7778-5e9b1aaed9a9@intel.com
<SNIP>
Cover: ./20211202_adrian_hunter_x86_insn_add_instructions_to_instruction_decoder.cover
Link: https://lore.kernel.org/r/20211202095029.2165714-1-adrian.hunter@intel.com
Base: applies clean to current tree
git checkout -b 20211202_adrian_hunter_intel_com HEAD
git am ./20211202_adrian_hunter_x86_insn_add_instructions_to_instruction_decoder.mbx
⬢[acme@toolbox perf]$

Lets see...

⬢[acme@toolbox perf]$ git am ./20211202_adrian_hunter_x86_insn_add_instructions_to_instruction_decoder.mbx
Applying: perf tests: Add AMX instructions to x86 instruction decoder test
Applying: x86/insn: Add AMX instructions to x86 instruction decoder
Applying: perf tests: Add misc instructions to x86 instruction decoder test
Applying: x86/insn: Add misc instructions to x86 instruction decoder
Applying: perf tests: Add AVX512-FP16 instructions to x86 instruction decoder test
Applying: x86/insn: Add AVX512-FP16 instructions to x86 instruction decoder
⬢[acme@toolbox perf]$

Yeah, it works, builds and the 'perf test instr' test case succeeds, so
if Peter or Borislav so wishes please process this patchkit as-is:

Acked-by: Arnaldo Carvalho de Melo <acme@redhat.com>

But please consider sending separate patchkits in the future, one for
kernel stuff, the other for tools/.

Thanks,

- Arnaldo

⬢[acme@toolbox perf]$ rm -rf /tmp/build/perf ; mkdir -p /tmp/build/perf ;
⬢[acme@toolbox perf]$ m
make: Entering directory '/var/home/acme/git/perf/tools/perf'
BUILD: Doing 'make -j32' parallel build
HOSTCC /tmp/build/perf/fixdep.o
HOSTLD /tmp/build/perf/fixdep-in.o
LINK /tmp/build/perf/fixdep
Warning: Kernel ABI header at 'tools/include/uapi/linux/perf_event.h' differs from latest version at 'include/uapi/linux/perf_event.h'
diff -u tools/include/uapi/linux/perf_event.h include/uapi/linux/perf_event.h

Auto-detecting system features:
... dwarf: [ on ]
... dwarf_getlocations: [ on ]
... glibc: [ on ]
... libbfd: [ on ]
... libbfd-buildid: [ on ]
... libcap: [ on ]
... libelf: [ on ]
... libnuma: [ on ]
... numa_num_possible_cpus: [ on ]
... libperl: [ on ]
... libpython: [ on ]
... libcrypto: [ on ]
... libunwind: [ on ]
... libdw-dwarf-unwind: [ on ]
... zlib: [ on ]
... lzma: [ on ]
... get_cpuid: [ on ]
... bpf: [ on ]
... libaio: [ on ]
... libzstd: [ on ]
... disassembler-four-args: [ on ]


GEN /tmp/build/perf/common-cmds.h
CC /tmp/build/perf/help.o
CC /tmp/build/perf/pager.o
CC /tmp/build/perf/parse-options.o
CC /tmp/build/perf/sigchain.o
CC /tmp/build/perf/exec-cmd.o
PERF_VERSION = 5.16.rc8.g0f0a5b2d75a4
<SNIP>
CC /tmp/build/perf/util/thread-stack.o
CC /tmp/build/perf/util/intel-pt.o
CC /tmp/build/perf/util/arm-spe.o
MKDIR /tmp/build/perf/util/arm-spe-decoder/
MKDIR /tmp/build/perf/util/intel-pt-decoder/
MKDIR /tmp/build/perf/util/intel-pt-decoder/
CC /tmp/build/perf/util/s390-cpumsf.o
CC /tmp/build/perf/util/parse-branch-options.o
MKDIR /tmp/build/perf/util/intel-pt-decoder/
MKDIR /tmp/build/perf/util/cs-etm-decoder/
CC /tmp/build/perf/util/arm-spe-decoder/arm-spe-pkt-decoder.o
CC /tmp/build/perf/util/dump-insn.o
CC /tmp/build/perf/util/cs-etm.o
CC /tmp/build/perf/util/intel-pt-decoder/intel-pt-pkt-decoder.o
GEN /tmp/build/perf/util/intel-pt-decoder/inat-tables.c
MKDIR /tmp/build/perf/util/intel-pt-decoder/
CC /tmp/build/perf/util/parse-sublevel-options.o
CC /tmp/build/perf/util/intel-pt-decoder/intel-pt-decoder.o
CC /tmp/build/perf/util/intel-pt-decoder/intel-pt-insn-decoder.o
CC /tmp/build/perf/util/intel-pt-decoder/intel-pt-log.o
<SNIP>
LD /tmp/build/perf/util/intel-pt-decoder/perf-in.o
<SNIP>
INSTALL perf_completion-script
INSTALL perf-tip
make: Leaving directory '/var/home/acme/git/perf/tools/perf'

Performance counter stats for 'make -k BUILD_BPF_SKEL=1 CORESIGHT=1 PYTHON=python3 O=/tmp/build/perf -C tools/perf install-bin':

59,896,945,805 cycles:u
54,732,597,938 instructions:u # 0.91 insn per cycle

17.699684374 seconds time elapsed

14.750269000 seconds user
12.750256000 seconds sys


On branch perf/core
Untracked files:
(use "git add <file>..." to include in what will be committed)
20211202_adrian_hunter_x86_insn_add_instructions_to_instruction_decoder.cover
20211202_adrian_hunter_x86_insn_add_instructions_to_instruction_decoder.mbx

nothing added to commit but untracked files present (use "git add" to track)
19: 'import perf' in python : Ok
⬢[acme@toolbox perf]$ perf test instr
76: x86 instruction decoder - new instructions : Ok
⬢[acme@toolbox perf]$


\
 
 \ /
  Last update: 2022-01-03 19:31    [W:0.170 / U:0.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site