RFR: 8348423: [TestBug] stress test Nodes initialization from a background thread [v8]
Andy Goryachev
angorya at openjdk.org
Wed Feb 5 21:50:40 UTC 2025
On Wed, 5 Feb 2025 20:38:00 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> Andy Goryachev has updated the pull request incrementally with one additional commit since the last revision:
>>
>> more jitter
>
> tests/system/src/test/java/test/robot/javafx/scene/NodeInitializationStressTest.java line 775:
>
>> 773: int threadCount = 1 + Runtime.getRuntime().availableProcessors() * 2;
>> 774: AtomicBoolean running = new AtomicBoolean(true);
>> 775: int additionalThreads = 2; // jiggler + tight loop
>
> I wonder if it is worth having more than 1 "tight loop" thread? Maybe dedicate 1/2 of the threads to a tight loop (`generator` in the loop) and 1/2 to access (`generator` once and `operation` in the loop) This is something that you could consider for a future improvement,
this is an interesting idea, esp. for intermittent tests. I'll keep this as is for now, but will use the idea if we encounter more stubborn tests.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1690#discussion_r1943701713
More information about the openjfx-dev
mailing list