RFR: 8277948: AArch64: Print the correct native stack if -XX:+PreserveFramePointer when crash [v2]

Denghui Dong ddong at openjdk.java.net
Fri Jan 28 00:50:12 UTC 2022


On Tue, 18 Jan 2022 13:43:25 GMT, Andrew Haley <aph at openjdk.org> wrote:

>> Denghui Dong has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   fix pfl() crash problem and rename from_thread to from_anchor
>
> So, here's my thinking for now. `_from_anchor` really means _this SP is trustworthy_, and perhaps we need a different name which suggests that. `sp_ok_to_use()` or `sp_is_trusted()` or somesuch? We do at least need a comment which explains that unless this boolean is true, the SP value in a frame is basically garbage, although it will point to somewhere within the stack. With that change, this patch can be integrated.
> In the longer term, I think we should look at using libunwind to obtain a precise native stack trace, and then we can get rid of all the old kludges.

Thanks. @theRealAph @adinn

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

PR: https://git.openjdk.java.net/jdk/pull/6597


More information about the hotspot-dev mailing list