[crac] RFR: Support extra criu flags from the environment [v2]

Volker Simonis simonis at openjdk.org
Thu Jun 30 17:12:04 UTC 2022


On Thu, 30 Jun 2022 06:43:52 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:

>> Volker Simonis has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Drop surplus arguments from CRAC_CRIU_OPTS and issue a warning
>
> src/java.base/unix/native/criuengine/criuengine.c line 116:
> 
>> 114:         }
>> 115:         *arg++ = NULL;
>> 116:         assert(ARRAY_SIZE(args) >= (size_t)(arg - args));
> 
> I'm not sure if we are compiling with asserts. Could you turn this and another assert below into a runtime check and explicit failure if arguments do not fit?

You're right. We compile product builds with `-DNDEBUG`. I've reworked the code to drop surplus arguments from `CRAC_CRIU_OPTS` and issue a warning if this was necessary.

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

PR: https://git.openjdk.org/crac/pull/26


More information about the crac-dev mailing list