RFR: JDK-8296931: NMT tests slowed down considerably by JDK-8242181

David Holmes dholmes at openjdk.org
Thu Nov 17 02:56:23 UTC 2022


On Mon, 14 Nov 2022 12:43:41 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> We noticed that NMT tests on our slower PPC machines started failing.
> 
> The reason is that NMT detail reports have become 2-5x slower. This is caused by us now parsing the dwarf debug information to extract source information for each PC in each call stack. That is nice but costly.
> 
> The slowdown is not limited to PPC, it affects all Elf platforms. On my Linux x64 box, runtime/NMT/VirtualAllocCommitMerge.java increased from 20 to 90 seconds.
> 
> ---
> 
> This patch simply removes source info from NMT call stacks. They are not that important for pinpointing leaks and such. I considered more involved solutions, like making them optional via an argument to the NMT report command, but decided against it. The added benefit would be small, not worth much complexity.
> 
> With this patch, on my box with -conc 4 all NMT together are about 2.5 x faster (2m56 -> 1m09).

Sorry have been swamped past couple of days.

>From the style guide:

> As a general rule don't add bug numbers to comments (they would soon overwhelm the code). But if the bug report contains significant information that can't reasonably be added as a comment, then refer to the bug report.

I would probably not have added it in this case but its a judgment call.

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

PR: https://git.openjdk.org/jdk/pull/11135


More information about the hotspot-dev mailing list