RFR: 8318474: Fix memory reporter for thread_count
Thomas Stuefe
stuefe at openjdk.org
Thu Oct 19 11:04:38 UTC 2023
On Wed, 18 Oct 2023 21:25:20 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:
> 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.
I think this is fine. What do you expect @MBaesken to see though? All you do is remove an assert 😀
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16255#issuecomment-1770579660
More information about the hotspot-runtime-dev
mailing list