[crac] RFR: Put restored Java process on foreground instead of restorewait
Roman Marchenko
rmarchenko at openjdk.org
Tue Oct 24 07:22:59 UTC 2023
On Mon, 23 Oct 2023 13:18:03 GMT, Radim Vansa <rvansa at openjdk.org> wrote:
> Solves a scenario where we restore the process in background, `fg` it and then try to Ctrl+C interrupt it.
>
> The usefulness of this actually depends on https://github.com/CRaC/criu/pull/14
>
> `criuengine` restorewait does not have tty on stdin/stdout/stderr, though one FD is open to tty; we have to find it.
src/java.base/unix/native/criuengine/criuengine.c line 391:
> 389: int fd = atoi(dp->d_name);
> 390: if (isatty(fd)) {
> 391: g_tty_fd = fd;
Shouldn't we break here?
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/133#discussion_r1369722656
More information about the crac-dev
mailing list