RFR: 8309258: RISC-V: Add riscv_hwprobe syscall [v2]
Robbin Ehn
rehn at openjdk.org
Wed Jun 14 14:24:14 UTC 2023
On Wed, 14 Jun 2023 08:15:55 GMT, Robbin Ehn <rehn at openjdk.org> wrote:
>> src/hotspot/os_cpu/linux_riscv/vm_version_linux_riscv.cpp line 90:
>>
>>> 88: assert(ext_D.feature_bit() == HWCAP_ISA_D, "Bit for D must follow Linux HWCAP");
>>> 89: assert(ext_C.feature_bit() == HWCAP_ISA_C, "Bit for C must follow Linux HWCAP");
>>> 90: assert(ext_V.feature_bit() == HWCAP_ISA_V, "Bit for V must follow Linux HWCAP");
>>
>> I'm somewhat confused about the hw_cap feature bits. Kernel asm/hwcap.h seems only to define caps up to C for riscv. This code mentions (in VMVersion) Q, H, V.
>>
>> Are these future flags?
>>
>> Here we don't check for Q, should we check that too?
>
> This is a moving target more is getting added.
> The basic standard ones follow this patterns.
> E.g. https://lore.kernel.org/lkml/20230428190609.3239486-2-evan@rivosinc.com/
>
> Yes Q should be added and the other ones. I'll just add Q in this change-set,
Added H and Q.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14445#discussion_r1229698443
More information about the hotspot-dev
mailing list