On Mon, 7 Jul 2025 00:39:20 GMT, Zhengyu Gu <z...@openjdk.org> wrote:

> but OOM and heap dump have to be, in a sense of atomic

I see; it can indeed be annoying that heap-dump doesn't capture the intended 
snapshot -- a java thread can trigger a gc that changes heap-content. However, 
AFAICS, fixing that non-atomic issue requires updating callers of 
`report_java_out_of_memory`. Therefore, I am inclined to think it should be 
addressed in its own ticket. It's a preexisting issue on master as well, not 
sth newly introduced by this PR.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25000#discussion_r2189268507

Reply via email to