RFR: 8350824: New async logging gtest StallingModePreventsDroppedMessages fails [v2]
Johan Sjölen
jsjolen at openjdk.org
Thu Feb 27 09:33:30 UTC 2025
> Hi,
>
> When I added async stalling mode I also added this gtest early in the process, and as tier1/GHA passed I did not look at the test again. However, the test does not run until much deeper in the CI/CD process for us, as that's when we turn on async logging explicitly for all gtests.
>
> Looking at it now, it's clearly buggy. It doesn't check which async mode UL is in, and as `drop` is default, that makes it bound to fail. Besides, the goal of it is to check whether async stalling mode drops messages or not during stress. We already have an excellent test for that in JTReg, and this does run early as it spawns its own VM with the correct asynchronous mode specified. My suggested fix is to delete the gtest and add a dropped message check to the JTReg test instead.
Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
bool in C++, boolean in Java, great :-)
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/23819/files
- new: https://git.openjdk.org/jdk/pull/23819/files/269b2261..05792d0c
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=23819&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=23819&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/23819.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/23819/head:pull/23819
PR: https://git.openjdk.org/jdk/pull/23819
More information about the hotspot-runtime-dev
mailing list