Integrated: 8310873: Re-enable locked_create_entry symbol check in runtime/NMT/CheckForProperDetailStackTrace.java for RISC-V
Feilong Jiang
fjiang at openjdk.org
Tue Jun 27 01:53:27 UTC 2023
On Mon, 26 Jun 2023 09:16:53 GMT, Feilong Jiang <fjiang at openjdk.org> wrote:
> The `Platform.isRISCV64()` check was once added in our internal development. The reason was that we have different C Frame and Java Frame layouts at first. So we could not get the right stacks without the RISC-V specific `get_native_stack`. After defining RISC-V related `get_native_stack` method (refactored by https://github.com/openjdk/jdk-sandbox/commit/db2415748747a0912749bb8fc160a8948021a924), it could return the expected symbols. So we can remove the platform check for RISC-V safely.
>
> runtime/NMT/CheckForProperDetailStackTrace.java still passed without `Platform.isRISCV64()` check.
>
> Testing:
> - [x] fastdebug build (cross/native compile)
> - [x] release build (cross/native compile)
This pull request has now been integrated.
Changeset: 87e6fab2
Author: Feilong Jiang <fjiang at openjdk.org>
Committer: Fei Yang <fyang at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/87e6fab2c498558d4593f6c0537c998a61779aaf
Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
8310873: Re-enable locked_create_entry symbol check in runtime/NMT/CheckForProperDetailStackTrace.java for RISC-V
Reviewed-by: fyang, rehn
-------------
PR: https://git.openjdk.org/jdk/pull/14650
More information about the hotspot-runtime-dev
mailing list