[crac] RFR: Fix CPUFeatures crash on new->old CPU [v5]
Jan Kratochvil
jkratochvil at openjdk.org
Wed Mar 27 08:36:41 UTC 2024
On Tue, 26 Mar 2024 16:05:45 GMT, Radim Vansa <rvansa at openjdk.org> wrote:
>> Jan Kratochvil has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fix github compilation signedness problem
>
> Sorry for keeping this waiting for a long time. In the past I wasn't sure about using `/tmp/` rather than `shm_open` target but since `os::get_temp_directory()` has this part hard-coded, too, I think it's acceptable to use that one.
>
> I would prefer `crac_restore_finalize` to use an enum rather than a `char` but otherwise this looks good - and if it fixes some usecase, let's integrate it.
@rvansa Sorry for the review but we have been last week talking with @AntonKozlov I should rather do the full rework (move the CPUFeatures protection to the parent JDK before the JDK is restored) than this fix-up.
-------------
PR Comment: https://git.openjdk.org/crac/pull/134#issuecomment-2022205320
More information about the crac-dev
mailing list