On Mon, 27 Oct 2025 18:08:40 GMT, Chad Rakoczy <[email protected]> wrote:
>>> @chadrako, is PR ready for testing now? >> >> Yes > >> @chadrako I think my suggestion was not correct. We should revert back to >> your first changes for `@requires`. Original code was correct and only >> `serviceability/jvmti/NMethodRelocation/NMethodRelocationTest.java` missed >> it. > > Since the tests get run with different GCs anyways I don't think we need to > explicitly require the GC that they run with and just have one test config FTR, even with this fix, we still see failures: [JDK-8371121](https://bugs.openjdk.org/browse/JDK-8371121), [JDK-8371046](https://bugs.openjdk.org/browse/JDK-8371046), [JDK-8369150](https://bugs.openjdk.org/browse/JDK-8369150). @chadrako It would be great if you could prioritize fixing these remaining issues, as the failures cause quite some noise in our testing. Thanks! ------------- PR Comment: https://git.openjdk.org/jdk/pull/27659#issuecomment-3501148078
