[crac] RFR: Persist memory in-JVM [v10]

Radim Vansa rvansa at openjdk.org
Fri Oct 6 08:10:17 UTC 2023


> This is a WIP for persisting various portions of JVM memory from within the process, rather than leaving that up to C/R engine (such as CRIU). In the future this will enable us to optimize loading (theoretically leading to faster startup), compress and encrypt the data.
> 
> At this moment the implementation of persisting thread stacks is in proof-of-concept shape, ~especially waking up the primordial thread includes some hacks. This could be improved by using a custom (global, ideally robust) futex instead of the internal futex used by `pthread_join`.~ Fix already implemented.
> 
> ~One of the concerns related to thread stacks is rseq used by glibc; without disabling this CRIU would attempt to 'fix' the rseqs (see `fixup_thread_rseq`) and touch the unmapped memory. CRIU uses special ptrace commands to check the status; I am not aware if it is possible to access this information from within the process using any public API.~ Solved. The JVM forks and the child ptraces JVM, recording the rseq info. Once we have the we can unregister the rseq before checkpoint and register it afterwards (here we have the advantage that we know the threads won't be in any critical section as we're in a safepoint).
> ~Currently this works with `/proc/sys/kernel/yama/ptrace_scope` set to `0`; we should make it work with `1` (default), too.~ Fixed.
> 
> Regarding persistence implementation, currently we store the memory in multiple files; first block (page-size aligned) contains some validation data and index of memory address - file offsets. The way this is implemented requires the size of index to be known before dumping memory. It might be more convenient (and portable for e.g. network-based storage) to use single file, and either keep the index in the 'fundamental' memory (C heap), put it at the end of file or to another index file.

Radim Vansa has updated the pull request incrementally with one additional commit since the last revision:

  Try fix other platforms

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

Changes:
  - all: https://git.openjdk.org/crac/pull/95/files
  - new: https://git.openjdk.org/crac/pull/95/files/11b4a04e..37819a6b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=crac&pr=95&range=09
 - incr: https://webrevs.openjdk.org/?repo=crac&pr=95&range=08-09

  Stats: 93 lines in 6 files changed: 31 ins; 45 del; 17 mod
  Patch: https://git.openjdk.org/crac/pull/95.diff
  Fetch: git fetch https://git.openjdk.org/crac.git pull/95/head:pull/95

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


More information about the crac-dev mailing list