[crac] RFR: CRaC: Fix fds opened for logging [v2]
Radim Vansa
rvansa at openjdk.org
Mon Oct 2 14:56:04 UTC 2023
On Mon, 2 Oct 2023 14:49:56 GMT, Jan Kratochvil <jkratochvil at openjdk.org> wrote:
>> CRaC: Fix fds opened for logging
>
> Jan Kratochvil has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains eight additional commits since the last revision:
>
> - AsyncLogWriter: add stop() and resume()
> - Merge branch 'crac' into crac-logfd
> - b229ea41: <no commit message found>
> - 8f161825: <no commit message found>
> - d8a454cb: <no commit message found>
> - Merge branch 'crac' into crac-logfd
> - +testcase
> - CRaC: Fix fds opened for logging
src/hotspot/share/runtime/crac.cpp line 431:
> 429: Universe::heap()->finish_collection();
> 430:
> 431: AsyncLogWriter::instance()->flush();
As a non-java thread, AsyncLogWriter does not participate in the safepoint protocol. What happens if another non-java thread enqueues a message to be written while the output is closed?
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/113#discussion_r1337075009
More information about the crac-dev
mailing list