On Fri, 23 Oct 2026 21:05:46 GMT, Mat Carter <[email protected]> wrote:

> Updated the tryMatchData call to also use the getVolatileWaiter() method.
> Added regression test
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

test/jdk/java/util/concurrent/LinkedTransferQueue/TransferRemove.java line 38:

> 36:  * @requires os.arch == "aarch64"
> 37:  * @library /test/lib
> 38:  * @run main/othervm -XX:-UseLSE -XX:ActiveProcessorCount=1 TransferRemove

@macarte How about configuring these two test configurations to run with a 
lower timeout, so that if they do hang, they aren't stalling for longer than 
necessary? 🤔

test/jdk/java/util/concurrent/LinkedTransferQueue/TransferRemove.java line 50:

> 48:     public static void main(String[] args) throws Exception {
> 49:         var threads = new ArrayList<Thread>();
> 50:         var queues = new ArrayList<LinkedTransferQueue<Object>>();

Suggestion:

        var threads = new ArrayList<Thread>(PAIRS * 2);
        var queues = new ArrayList<LinkedTransferQueue<Object>>(PAIRS);

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33551#discussion_r4387431967
PR Review Comment: https://git.openjdk.org/jdk/pull/33551#discussion_r4387446721

Reply via email to