On 13/12/2022 14:51, Arnaldo Carvalho de Melo wrote: > Em Tue, Dec 13, 2022 at 11:47:35AM +0000, James Clark escreveu: >> I didn't get any feedback on the RFC version of this that I posted a >> while back [1]. I'd still like to add the test, especially now that >> 6.1 has been released with this new feature, so I've rebased it onto >> perf/core and double checked that it's still working. >> >> Applies to perf/core (0c3852adae83) > > I'm applying this locally, would this be testable on a Firefly (roc-rk3399-pc): Thanks Arnaldo. It doesn't look like rk3399 has SVE, so test-record-user-regs-no-sve-aarch64 will run to check that the kernel won't give you the new register. So I suppose the answer to the question is partially. For test-record-user-regs-sve-aarch64, I've been running it on a Graviton 3 on AWS which has SVE: ubuntu@ip-10-252-130-213:~/linux$ head /proc/cpuinfo processor : 0 BogoMIPS : 2100.00 Features : fp asimd evtstrm aes pmull sha1 sha2 crc32 atomics fphp asimdhp cpuid asimdrdm jscvt fcma lrcpc dcpop sha3 sm3 sm4 asimddp sha512 sve asimdfhm dit uscat ilrcpc flagm ssbs paca pacg dcpodp svei8mm svebf16 i8mm bf16 dgh rng CPU implementer : 0x41 CPU architecture: 8 CPU variant : 0x1 CPU part : 0xd40 CPU revision : 1 processor : 1 > > acme@roc-rk3399-pc:~$ head /proc/cpuinfo > processor : 0 > BogoMIPS : 48.00 > Features : fp asimd evtstrm aes pmull sha1 sha2 crc32 cpuid > CPU implementer : 0x41 > CPU architecture: 8 > CPU variant : 0x0 > CPU part : 0xd03 > CPU revision : 4 > > processor : 1 > acme@roc-rk3399-pc:~$ > > - Arnaldo > >> Thanks >> James >> >> [1]: https://lore.kernel.org/bpf/20220927154104.869029-1-james.clark@xxxxxxx/ >> >> James Clark (4): >> perf test: Add ability to test exit code for attr tests >> perf test: Add mechanism for skipping attr tests on auxiliary vector >> values >> perf test: Add mechanism for skipping attr tests on kernel versions >> perf test arm64: Add attr tests for new VG register >> >> tools/perf/tests/attr.py | 71 +++++++++++++++++-- >> .../attr/test-record-user-regs-no-sve-aarch64 | 9 +++ >> .../test-record-user-regs-old-sve-aarch64 | 10 +++ >> .../attr/test-record-user-regs-sve-aarch64 | 14 ++++ >> 4 files changed, 99 insertions(+), 5 deletions(-) >> create mode 100644 tools/perf/tests/attr/test-record-user-regs-no-sve-aarch64 >> create mode 100644 tools/perf/tests/attr/test-record-user-regs-old-sve-aarch64 >> create mode 100644 tools/perf/tests/attr/test-record-user-regs-sve-aarch64 >> >> -- >> 2.25.1 >
