RFR: 8262386: resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java timed out [v10]

Chris Plummer cjplummer at openjdk.java.net
Fri May 14 03:56:46 UTC 2021


On Fri, 14 May 2021 02:08:13 GMT, Lin Zang <lzang at openjdk.org> wrote:

>> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/utilities/HeapHprofBinWriter.java line 1528:
>> 
>>> 1526:             }
>>> 1527:             writeInteger(0);
>>> 1528:             writeInteger(size);
>> 
>> You've added `writeInteger(size)` here, but I'm not sure where this used to be done.
>
> There used to be 2 writeInterger(0), and the second one is write the segment size with dummy value 0. I will add a comment here. Thanks

Ok, I missed that. I think where a comment is needed is with the first `writerInteger(0)`. It's not clear to me what that one is for.

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

PR: https://git.openjdk.java.net/jdk/pull/2803


More information about the serviceability-dev mailing list