RFR: 8318474: Fix memory reporter for thread_count

Johan Sjölen jsjolen at openjdk.org
Wed Oct 18 21:32:20 UTC 2023


Hi,

[JDK-8315362](https://bugs.openjdk.org/browse/JDK-8315362) did the right thing and always updated the `_thread_count` regardless of the 'mode' that we are in, but we forgot about this assert. So, let's get rid of the assert.

@tstuefe, @gerard-ziemski, I was wondering if we can't actually just use the `_thread_count` on AIX now when printing the thread count, shouldn't that be correct now? Regardless, let's not have this assert, as it breaks NMT tests on AIX.

@MBaesken, would you mind trying this patch out? It's on top of current master, so you can check it out directly.

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

Commit messages:
 - Remove unnecessary assert

Changes: https://git.openjdk.org/jdk/pull/16255/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=16255&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8318474
  Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/16255.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16255/head:pull/16255

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


More information about the hotspot-runtime-dev mailing list