On Wed, 29 Oct 2025 01:51:15 GMT, Chris Plummer <[email protected]> wrote:
>> Yasumasa Suenaga has updated the pull request incrementally with one >> additional commit since the last revision: >> >> TestJhsdbJstackMixedWithXComp.java should be run on Linux AMD64 only > > test/hotspot/jtreg/serviceability/sa/TestJhsdbJstackMixedWithXComp.java line > 63: > >> 61: System.err.println(out.getStderr()); >> 62: >> 63: out.stderrShouldBeEmptyIgnoreDeprecatedWarnings(); > > I might want you to change this to stderrShouldBeEmptyIgnoreVMWarnings(). We > have issues with our internal testing when using -XX:+UseLargePages that > sometimes results in a VM warning on stderr that causes this (and some other > tests) to fail. We are still deciding on the best approach to fixing this, > but I think the solution is most likely to be switching to > stderrShouldBeEmptyIgnoreDeprecatedWarnings(). Hopefully I'll know within the > next day. I updated the test to use `stderrShouldBeEmptyIgnoreVMWarnings()`. @plummercj let me know if I should make more changes. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27885#discussion_r2480178823
