[crac] RFR: Make CheckpointException/RestoreException aggregate-only [v8]

Radim Vansa duke at openjdk.org
Mon Jun 12 09:37:16 UTC 2023


On Fri, 9 Jun 2023 17:44:04 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:

>> Shouldn't a dry-run return `JVM_CHECKPOINT_OK`? Which test is that, `DryRunTest` works for me?
>
> The test does not cover this case: when a Resource throws something and we go to dry run JVM inspection, but the inspection does not find anything. 
> 
> Turns out, ContextOrderTest fails bacuse the same reason.
> 
> JVM dry-run now always returns CHECKPOINT_ERROR by implementation. But in general the the code should depend on the inspection result: OK if no offending descriptors found, ERROR (better to be FD_ERROR) when there are open file descriptors.

Fixed in https://github.com/openjdk/crac/pull/80

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

PR Review Comment: https://git.openjdk.org/crac/pull/64#discussion_r1226371508


More information about the crac-dev mailing list