On Wed, 5 Feb 2025 20:38:00 GMT, Kevin Rushforth <k...@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
