[crac] RFR: Refactoring - extracted crac files

Roman Marchenko rmarchenko at openjdk.org
Fri Jun 16 11:49:30 UTC 2023


On Fri, 16 Jun 2023 11:35:22 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:

>> CRaC-related functionality is moved to `crac*.hpp/cpp` files, and now it is in `crac` class instead of `os`.
>
> src/hotspot/share/runtime/arguments.cpp line 3235:
> 
>> 3233: #endif // CAN_SHOW_REGISTERS_ON_ASSERT
>> 3234: 
>> 3235:   if (CRaCCheckpointTo && !crac::Linux::prepare_checkpoint()) {
> 
> Does it really make sense to have `crac::Linux` instead of just `crac::`? The most of the operations are not related to Linux, so it may be even better to start abstracting them from Linux in this PR, to avoid another similar renaming in the same place.

Ok, let's get rid of crac::linux now.

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

PR Review Comment: https://git.openjdk.org/crac/pull/84#discussion_r1232141645


More information about the crac-dev mailing list