RFR: 8317692: jcmd GC.heap_dump performance regression after JDK-8292818

Alex Menkov amenkov at openjdk.org
Wed Oct 11 01:00:10 UTC 2023


On Fri, 6 Oct 2023 20:56:13 GMT, Hannes Greule <hgreule at openjdk.org> wrote:

> See the bug description for more information.
> 
> This implementation brings down the time to take a heap dump on the example application in the bug report to <2 seconds on my machine.

src/hotspot/share/oops/fieldStreams.hpp line 194:

> 192:   void prepare() {
> 193:     _next_klass = next_klass_with_fields();
> 194:     // special case: the base klass has no fields. If any supertype has any fields, use that directly.

"base klass" sounds misleading.
I think "initial" would be clearer

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16083#discussion_r1353752601


More information about the serviceability-dev mailing list