[jdk11u-dev] RFR: 8267666: Add option to jcmd GC.heap_dump to use existing file

[Ralf Schmelter]

On Tue, 13 Jul 2021 21:08:33 GMT, Christoph Langer <clanger at openjdk.org> wrote:

> Backport of the heap dump enhancement. CSR includes update releases.
> 
> Patch did not apply cleanly, I had to resolve/modify
> src/hotspot/share/services/diagnosticCommand.cpp
> src/hotspot/share/services/diagnosticCommand.hpp
> src/hotspot/share/services/heapDumper.cpp
> src/hotspot/share/services/heapDumper.hpp
> 
> Following files do not exist in 11u:
> src/hotspot/share/services/heapDumperCompression.cpp
> src/hotspot/share/services/heapDumperCompression.hpp
> Changes had been made in heapDumper.cpp/heapDumper.hpp
> 
> I have built it on linuxx86_64 and verified that the modified test succeeds. Will also run it through SAP's test suite before merging.

Looks good to me.

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

Marked as reviewed by rschmelter (no project role).

PR: https://git.openjdk.java.net/jdk11u-dev/pull/131