[crac] RFR: RFC: -XX:CPUFeatures=0xnumber for CPU migration [v24]

Jan Kratochvil jkratochvil at openjdk.org
Thu Jun 1 13:47:49 UTC 2023


On Wed, 31 May 2023 13:54:48 GMT, Jan Kratochvil <jkratochvil at openjdk.org> wrote:

>> src/hotspot/os_cpu/linux_x86/os_linux_x86.cpp line 224:
>> 
>>> 222:       stub = VM_Version::cpuinfo_cont_addr();
>>> 223:     }
>>> 224:   } else
>> 
>> Why do we need this block? Is not this duplicating logic from line 251?
>
> It was needed for some previous variant of the patch and it is a leftover. Removed. Sorry for not self-reviewing this version of my patch.

It is in fact needed: https://github.com/openjdk/crac/blob/08c2c0f2b140b1011fec39a1a83037df8e59577d/src/hotspot/os_cpu/linux_x86/os_linux_x86.cpp#L217
During CRaC restore there is somehow no Java thread created yet that time. I do not understand this much so I left it there as working but maybe some Java thread should be initialized earlier?

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

PR Review Comment: https://git.openjdk.org/crac/pull/41#discussion_r1213180053


More information about the crac-dev mailing list