[riscv-port] RFR: 8279664: riscv: JFR crashes at 0x0 [v2]

Fei Yang fyang at openjdk.java.net
Fri Jan 21 03:18:12 UTC 2022


On Fri, 21 Jan 2022 03:09:30 GMT, Xiaolin Zheng <xlinzheng at openjdk.org> wrote:

>> src/hotspot/cpu/riscv/macroAssembler_riscv.hpp line 63:
>> 
>>> 61: 
>>> 62:   void leave() {
>>> 63:     // Note: setting sp above the address of alive variables could result in undefined behaviors:
>> 
>> I would suggest move this code comment before function "void enter()" in the same file.
>> Maybe rephrasing like this:
>> "SP must be updated to the right place before saving/restoring RA and FP because signal
>>  based thread suspend/resume could happen asynchronously."
>
> Thanks for the suggestion, changed.

I think it's better to go like this:

  // Stack frame creation/removal
  // Note that SP must be updated to the right place before saving/restoring RA and FP
  // because signal based thread suspend/resume could happen asynchronously.

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

PR: https://git.openjdk.java.net/riscv-port/pull/49


More information about the riscv-port-dev mailing list