[crac] RFR: PID adjustment on checkpoint [v10]
Roman Marchenko
rmarchenko at openjdk.org
Thu Jun 29 17:13:32 UTC 2023
On Wed, 28 Jun 2023 11:57:56 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:
>> Roman Marchenko has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Added FIXME for further steps
>
> src/java.base/share/native/launcher/main.c line 213:
>
>> 211: static void spin_last_pid(int pid) {
>> 212: const int MaxSpinCount = pid < 1000 ? 1000 : pid;
>> 213: for (int child = fork(), prev = 0, cnt = MaxSpinCount; child < pid; child = fork(), --cnt) {
>
> Since waitpid is called only if `child < pid`, does this mean the last child that satisfy pid requirement is left unwaited?
Yes, you're correct.
Do you think it's potentially dangerous or consumes resources?
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/86#discussion_r1246915249
More information about the crac-dev
mailing list