RFR: 8338383: Implement JEP 491: Synchronize Virtual Threads without Pinning [v9]

Fei Yang fyang at openjdk.org
Sat Nov 2 02:44:55 UTC 2024


On Thu, 31 Oct 2024 20:02:31 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

>> src/hotspot/cpu/riscv/continuationFreezeThaw_riscv.inline.hpp line 273:
>> 
>>> 271:         ? frame_sp + fsize - frame::sender_sp_offset
>>> 272:         // we need to re-read fp because it may be an oop and we might have fixed the frame.
>>> 273:         : *(intptr_t**)(hf.sp() - 2);
>> 
>> Suggestion:
>> 
>>         : *(intptr_t**)(hf.sp() - frame::sender_sp_offset);
>
> Changed.

Note that `frame::sender_sp_offset` is 0 instead of 2 on linux-riscv64, which is different from aarch64 or x86-64. So I think we should revert this change: https://github.com/openjdk/jdk/pull/21565/commits/12213a70c1cf0639555f0f302237fd012549c4dd. @pchilano : Could you please help do that?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21565#discussion_r1826453713


More information about the core-libs-dev mailing list