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