Re: RFR: 8344197: SM cleanup in java.util.concurrent [v2]

2024-11-15 Thread Alan Bateman
On Fri, 15 Nov 2024 15:47:43 GMT, Roger Riggs wrote: >> src/java.base/share/classes/java/util/concurrent/atomic/AtomicIntegerFieldUpdater.java >> line 399: >> >>> 397: if ((ccl != null) && (ccl != cl) && >>> 398: ((cl == null) || !isAncestor(cl, ccl))) { >>>

Re: RFR: 8344197: SM cleanup in java.util.concurrent [v2]

2024-11-15 Thread Roger Riggs
On Fri, 15 Nov 2024 13:30:34 GMT, Doug Lea wrote: >> src/java.base/share/classes/java/util/concurrent/ForkJoinPool.java line 1108: >> >>> 1106: implements ForkJoinWorkerThreadFactory { >>> 1107: public final ForkJoinWorkerThread newThread(ForkJoinPool >>> pool) { >>> 1108:

Re: RFR: 8344197: SM cleanup in java.util.concurrent [v2]

2024-11-15 Thread Roger Riggs
> Refactored to remove use of doPrivileged() and use of SecurityManager. > The DefaultForkJoinWorkerThreadFactory no longer uses the SM to target a > common thread pool. > > A careful review is requested. Roger Riggs has updated the pull request incrementally with one additional commit since th