On Fri, 27 Feb 2026 18:29:38 GMT, Chris Plummer <[email protected]> wrote:
>> We currently problem list all the SA tests when ZGC is being used. However, >> recently the CR they are problem listed under >> ([JDK-8307393](https://bugs.openjdk.org/browse/JDK-8307393)) was closed as >> WNF. Tests are not suppose to be problem listed under closed CRs. The >> proposed solution is to use the following in all the SA tests: >> >> ` * @requires vm.gc != "Z"` >> >> Note I kept `serviceability/sa/TestUniverse.java` enabled for ZGC as it was >> explicitly called out in the problem list comment as a sanity check to keep >> enabled. >> >> While adding the `@requires` I also did some minor cleanup in a few places >> to keep the `@requires` less cluttered and consistent between tests. >> Specifically I made it so `@requires vm.hasSA` is always on its own line and >> always first. >> >> Testing: all tier1 plus tier2, tier3, and tier5 svc testing. > > Chris Plummer has updated the pull request incrementally with one additional > commit since the last revision: > > fix missed copyright upate test/hotspot/jtreg/serviceability/sa/TestJhsdbJstackLineNumbers.java line 43: > 41: * @requires os.arch=="amd64" | os.arch=="x86_64" > 42: * @requires os.family=="windows" | os.family == "linux" | os.family == > "mac" > 43: * @requires vm.flagless Since ZGC is not the default gc general, and this test already has `@requires vm.flagless`. So the additional `@reauires vm.gc != "Z"` is not needed actually? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/29959#discussion_r2882195496
