On Mon, 12 May 2025 11:45:29 GMT, Per Minborg <pminb...@openjdk.org> wrote:

>> This PR proposes to add a safety net around closing the executor. 
>> Apparently, in some rare configuration, the VT is not run/notified correctly.
>> 
>> Not completing the test for such a configuration is unlikely to mask 
>> potential issues that this test is supposed to reveal.
>
> Per Minborg has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Add @bug definition

test/jdk/java/foreign/TestBufferStackStress2.java line 30:

> 28:  * @requires vm.continuations
> 29:  *
> 30:  * @bug 8356114, 8356658

jtreg doesn't like the presence of comma in those values and the GitHub job has 
caught that failure in this test.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25177#discussion_r2084707435

Reply via email to