[crac] RFR: PID adjustment on checkpoint [v3]
    Roman Marchenko 
    rmarchenko at openjdk.org
       
    Tue Jun 27 09:04:37 UTC 2023
    
    
  
On Wed, 21 Jun 2023 13:25:01 GMT, Roman Marchenko <rmarchenko at openjdk.org> wrote:
>> src/java.base/share/native/launcher/main.c line 331:
>> 
>>> 329:         const int crac_min_pid = 0 < env_min_pid ? env_min_pid : crac_min_pid_default;
>>> 330: 
>>> 331:         if (getpid() <= crac_min_pid) {
>> 
>> A nit: probably `getpid() < crac_min_pid`?
>
> This should work for pid==1 as well. 
> The possible values for crac_min_pid are 1, 2, 3, ...
> In case of `getpid() < crac_min_pid`, it needs to have an additional check.
As we discussed this offline, now PID adjustment is not applied by default, it works only if CracMinPid option is set by an user. So 'fork & wait' now is applied for pid==1 or for PID adjustment.
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/86#discussion_r1243389704
    
    
More information about the crac-dev
mailing list