On Tue, 7 Apr 2026 13:44:37 GMT, Tobias Hartmann <[email protected]> wrote:

> The test randomly excludes methods from C1/C2 compilation to stress test the 
> calling convention. While this has been proven to be extremely helpful for 
> testing the Virtual Threads implementation, it's also extremely slow - 
> especially when we are unlucky enough for multiple methods to be excluded 
> both from C1 and C2 compilation. On slow machines and in combination with 
> other VM flags, test execution time can easily exceed an hour.
> 
> While we could now try to improve how these methods are selected, I don't 
> think it's worth it and we risk wasting significant machine hours just 
> running methods in the interpreter. I therefore disabled this test feature by 
> default. It can still be re-enabled for debugging / stress testing purposes 
> on demand. Long-term, we should add this capability to the VM. I filed 
> [JDK-8381751](https://bugs.openjdk.org/browse/JDK-8381751) for this.
> 
> Thanks,
> Tobias

That's unfortunate but I agree that we should better use the machine hours 
differently instead.

-------------

Marked as reviewed by chagedorn (Committer).

PR Review: 
https://git.openjdk.org/valhalla/pull/2306#pullrequestreview-4068567062

Reply via email to