[crac] RFR: Persist memory in-JVM [v6]

Radim Vansa rvansa at openjdk.org
Fri Sep 29 06:33:38 UTC 2023


On Mon, 25 Sep 2023 17:25:22 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:

>> Radim Vansa has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 33 commits:
>> 
>>  - Move MemoryPersister impl to own file
>>  - Merge branch 'crac' into persist_memory
>>  - Backport of API from future changes for other persistent memory features
>>  - Another assembly fix
>>  - Don't fork when we're not unregistering rseq
>>  - Fix assembly loop
>>  - fix whitespaces
>>  - fix whitespaces
>>  - Address review comments, fix rseq on GLIBC < 2.35
>>  - Merge branch 'crac' into persist_memory
>>  - ... and 23 more: https://git.openjdk.org/crac/compare/8fcfc112...1a8fc70c
>
> src/hotspot/os/linux/crac_linux.cpp line 614:
> 
>> 612:   while (persist_futex) {
>> 613:      syscall(SYS_futex, &persist_futex, FUTEX_WAIT_PRIVATE, 1, nullptr, nullptr, 0);
>> 614:   }
> 
> Nit: in asm it
> 
> do {
>   syscall(SYS_futex, &persist_futex, FUTEX_WAIT_PRIVATE, 1, nullptr, nullptr, 0);
> } while (persist_futex);
> 
> is implemented.

👍  Also I should add the error handling - originally I had it without and that led to busy loops when I've set it incorrectly.

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

PR Review Comment: https://git.openjdk.org/crac/pull/95#discussion_r1340946729


More information about the crac-dev mailing list