linux-kernel - Re: [PATCH v2 0/4] Add VG register attr test with kernel version and feature detection

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <634087f2-1a7a-0f61-51e8-9e6068d59e9d@arm.com>
Date:   Tue, 13 Dec 2022 15:30:37 +0000
From:   James Clark <james.clark@....com>
To:     Arnaldo Carvalho de Melo <acme@...nel.org>
Cc:     linux-perf-users@...r.kernel.org,
        Peter Zijlstra <peterz@...radead.org>,
        Ingo Molnar <mingo@...hat.com>,
        Mark Rutland <mark.rutland@....com>,
        Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
        Jiri Olsa <jolsa@...nel.org>,
        Namhyung Kim <namhyung@...nel.org>,
        linux-kernel@...r.kernel.org, bpf@...r.kernel.org
Subject: Re: [PATCH v2 0/4] Add VG register attr test with kernel version and
 feature detection



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@...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@...-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@...-rk3399-pc:~$
> 
> - Arnaldo
>  
>> Thanks
>> James
>>
>> [1]: https://lore.kernel.org/bpf/20220927154104.869029-1-james.clark@arm.com/
>>
>> 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
>