RFR: 8367098: RISC-V: sync CPU features with related JVM flags for dependant ones

Fei Yang fyang at openjdk.org
Tue Sep 9 06:30:16 UTC 2025


On Mon, 8 Sep 2025 12:15:31 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to review this patch?
> 
> some extensions depends on another one, for example, zvbb depends on rvv.
> That means if rvv is disabled or not supported by the CPU, UseZvbb should return false, and zvbb should not appear in cpu string.
> But the currently, the cpu string contains zvbb even if rvv is disabled, which is not right.
> This needs to be fixed.
> 
> Thanks
> 
> before fix, output for `-XX-UseRVV -Xlog:cpu*=debug`
> 
> [0.064s][info ][os,cpu] CPU: total 32 (initial active 32) qemu rv64 rvi rvm rva rvf rvd rvc rvv zicbom zicboz zicbop zba zbb zbs zbkb zcb zfa zfh zfhmin zicsr zifencei zic64b ztso zacas zvbb zvbc zvfh zicond
> 
> 
> after fix, output for `-XX-UseRVV -Xlog:cpu*=debug`
> 
> [0.065s][info ][os,cpu] CPU: total 32 (initial active 32) qemu rv64 rvi rvm rva rvf rvd rvc zicbom zicboz zicbop zba zbb zbs zbkb zcb zfa zfh zfhmin zicsr zifencei zic64b ztso zacas zicond

LGTM! Thanks for fixing this!

-------------

Marked as reviewed by fyang (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27142#pullrequestreview-3199554769


More information about the hotspot-dev mailing list