On 12/11/19 9:05 AM, Alex Bennée wrote:
> +int main(int argc, char **argv)
> +{
> + unsigned int sve_feature = (get_cpu_reg(ID_AA64PFR0_EL1) >> 32) & 0xf;
> + /* Exit early if we don't support SVE at all */
> + if (sve_feature == 0x1) {
> + /* we also need to probe for the ioctl support */
> + if (getauxval(AT_HWCAP) & HWCAP_SVE) {
> + return do_sve_ioctl_test();
> + } else {
> + printf("SKIP: no HWCAP_SVE on this system\n");
> + return 0;
> + }
I don't see the point in the PFR0 test.
Testing HWCAP_SVE should be sufficient.
r~
