--- jsr166/src/jsr166y/ForkJoinPool.java 2010/04/18 13:59:57 1.55 +++ jsr166/src/jsr166y/ForkJoinPool.java 2010/08/17 18:30:32 1.64 @@ -21,7 +21,7 @@ import java.util.concurrent.CountDownLat /** * An {@link ExecutorService} for running {@link ForkJoinTask}s. * A {@code ForkJoinPool} provides the entry point for submissions - * from non-{@code ForkJoinTask}s, as well as management and + * from non-{@code ForkJoinTask} clients, as well as management and * monitoring operations. * *

A {@code ForkJoinPool} differs from other kinds of {@link @@ -30,33 +30,19 @@ import java.util.concurrent.CountDownLat * execute subtasks created by other active tasks (eventually blocking * waiting for work if none exist). This enables efficient processing * when most tasks spawn other subtasks (as do most {@code - * ForkJoinTask}s). A {@code ForkJoinPool} may also be used for mixed - * execution of some plain {@code Runnable}- or {@code Callable}- - * based activities along with {@code ForkJoinTask}s. When setting - * {@linkplain #setAsyncMode async mode}, a {@code ForkJoinPool} may - * also be appropriate for use with fine-grained tasks of any form - * that are never joined. Otherwise, other {@code ExecutorService} - * implementations are typically more appropriate choices. + * ForkJoinTask}s). When setting asyncMode to true in + * constructors, {@code ForkJoinPool}s may also be appropriate for use + * with event-style tasks that are never joined. * *

A {@code ForkJoinPool} is constructed with a given target * parallelism level; by default, equal to the number of available - * processors. Unless configured otherwise via {@link - * #setMaintainsParallelism}, the pool attempts to maintain this - * number of active (or available) threads by dynamically adding, - * suspending, or resuming internal worker threads, even if some tasks - * are stalled waiting to join others. However, no such adjustments - * are performed in the face of blocked IO or other unmanaged - * synchronization. The nested {@link ManagedBlocker} interface - * enables extension of the kinds of synchronization accommodated. - * The target parallelism level may also be changed dynamically - * ({@link #setParallelism}). The total number of threads may be - * limited using method {@link #setMaximumPoolSize}, in which case it - * may become possible for the activities of a pool to stall due to - * the lack of available threads to process new tasks. When the pool - * is executing tasks, these and other configuration setting methods - * may only gradually affect actual pool sizes. It is normally best - * practice to invoke these methods only when the pool is known to be - * quiescent. + * processors. The pool attempts to maintain enough active (or + * available) threads by dynamically adding, suspending, or resuming + * internal worker threads, even if some tasks are stalled waiting to + * join others. However, no such adjustments are guaranteed in the + * face of blocked IO or other unmanaged synchronization. The nested + * {@link ManagedBlocker} interface enables extension of the kinds of + * synchronization accommodated. * *

In addition to execution and lifecycle control methods, this * class provides status check methods (for example @@ -65,6 +51,40 @@ import java.util.concurrent.CountDownLat * {@link #toString} returns indications of pool state in a * convenient form for informal monitoring. * + *

As is the case with other ExecutorServices, there are three + * main task execution methods summarized in the following + * table. These are designed to be used by clients not already engaged + * in fork/join computations in the current pool. The main forms of + * these methods accept instances of {@code ForkJoinTask}, but + * overloaded forms also allow mixed execution of plain {@code + * Runnable}- or {@code Callable}- based activities as well. However, + * tasks that are already executing in a pool should normally + * NOT use these pool execution methods, but instead use the + * within-computation forms listed in the table. + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + *
Call from non-fork/join clients Call from within fork/join computations
Arange async execution {@link #execute(ForkJoinTask)} {@link ForkJoinTask#fork}
Await and obtain result {@link #invoke(ForkJoinTask)} {@link ForkJoinTask#invoke}
Arrange exec and obtain Future {@link #submit(ForkJoinTask)} {@link ForkJoinTask#fork} (ForkJoinTasks are Futures)
+ * *

Sample Usage. Normally a single {@code ForkJoinPool} is * used for all parallel task execution in a program or subsystem. * Otherwise, use would not usually outweigh the construction and @@ -89,7 +109,8 @@ import java.util.concurrent.CountDownLat * {@code IllegalArgumentException}. * *

This implementation rejects submitted tasks (that is, by throwing - * {@link RejectedExecutionException}) only when the pool is shut down. + * {@link RejectedExecutionException}) only when the pool is shut down + * or internal resources have been exhausted. * * @since 1.7 * @author Doug Lea @@ -116,21 +137,59 @@ public class ForkJoinPool extends Abstra * of tasks profit from cache affinities, but others are harmed by * cache pollution effects.) * + * Beyond work-stealing support and essential bookkeeping, the + * main responsibility of this framework is to take actions when + * one worker is waiting to join a task stolen (or always held by) + * another. Becauae we are multiplexing many tasks on to a pool + * of workers, we can't just let them block (as in Thread.join). + * We also cannot just reassign the joiner's run-time stack with + * another and replace it later, which would be a form of + * "continuation", that even if possible is not necessarily a good + * idea. Given that the creation costs of most threads on most + * systems mainly surrounds setting up runtime stacks, thread + * creation and switching is usually not much more expensive than + * stack creation and switching, and is more flexible). Instead we + * combine two tactics: + * + * Helping: Arranging for the joiner to execute some task that it + * would be running if the steal had not occurred. Method + * ForkJoinWorkerThread.helpJoinTask tracks joining->stealing + * links to try to find such a task. + * + * Compensating: Unless there are already enough live threads, + * method helpMaintainParallelism() may create or or + * re-activate a spare thread to compensate for blocked + * joiners until they unblock. + * + * Because the determining existence of conservatively safe + * helping targets, the availability of already-created spares, + * and the apparent need to create new spares are all racy and + * require heuristic guidance, we rely on multiple retries of + * each. Further, because it is impossible to keep exactly the + * target (parallelism) number of threads running at any given + * time, we allow compensation during joins to fail, and enlist + * all other threads to help out whenever they are not otherwise + * occupied (i.e., mainly in method preStep). + * + * The ManagedBlocker extension API can't use helping so relies + * only on compensation in method awaitBlocker. + * * The main throughput advantages of work-stealing stem from * decentralized control -- workers mostly steal tasks from each * other. We do not want to negate this by creating bottlenecks - * implementing the management responsibilities of this class. So - * we use a collection of techniques that avoid, reduce, or cope - * well with contention. These entail several instances of - * bit-packing into CASable fields to maintain only the minimally - * required atomicity. To enable such packing, we restrict maximum - * parallelism to (1<<15)-1 (enabling twice this to fit into a 16 - * bit field), which is far in excess of normal operating range. - * Even though updates to some of these bookkeeping fields do - * sometimes contend with each other, they don't normally - * cache-contend with updates to others enough to warrant memory - * padding or isolation. So they are all held as fields of - * ForkJoinPool objects. The main capabilities are as follows: + * implementing other management responsibilities. So we use a + * collection of techniques that avoid, reduce, or cope well with + * contention. These entail several instances of bit-packing into + * CASable fields to maintain only the minimally required + * atomicity. To enable such packing, we restrict maximum + * parallelism to (1<<15)-1 (enabling twice this (to accommodate + * unbalanced increments and decrements) to fit into a 16 bit + * field, which is far in excess of normal operating range. Even + * though updates to some of these bookkeeping fields do sometimes + * contend with each other, they don't normally cache-contend with + * updates to others enough to warrant memory padding or + * isolation. So they are all held as fields of ForkJoinPool + * objects. The main capabilities are as follows: * * 1. Creating and removing workers. Workers are recorded in the * "workers" array. This is an array as opposed to some other data @@ -140,39 +199,38 @@ public class ForkJoinPool extends Abstra * (workerLock) but the array is otherwise concurrently readable, * and accessed directly by workers. To simplify index-based * operations, the array size is always a power of two, and all - * readers must tolerate null slots. Currently, all but the first - * worker thread creation is on-demand, triggered by task - * submissions, replacement of terminated workers, and/or - * compensation for blocked workers. However, all other support - * code is set up to work with other policies. + * readers must tolerate null slots. Currently, all worker thread + * creation is on-demand, triggered by task submissions, + * replacement of terminated workers, and/or compensation for + * blocked workers. However, all other support code is set up to + * work with other policies. + * + * To ensure that we do not hold on to worker references that + * would prevent GC, ALL accesses to workers are via indices into + * the workers array (which is one source of some of the unusual + * code constructions here). In essence, the workers array serves + * as a WeakReference mechanism. Thus for example the event queue + * stores worker indices, not worker references. Access to the + * workers in associated methods (for example releaseEventWaiters) + * must both index-check and null-check the IDs. All such accesses + * ignore bad IDs by returning out early from what they are doing, + * since this can only be associated with shutdown, in which case + * it is OK to give up. On termination, we just clobber these + * data structures without trying to use them. * * 2. Bookkeeping for dynamically adding and removing workers. We - * maintain a given level of parallelism (or, if - * maintainsParallelism is false, at least avoid starvation). When - * some workers are known to be blocked (on joins or via + * aim to approximately maintain the given level of parallelism. + * When some workers are known to be blocked (on joins or via * ManagedBlocker), we may create or resume others to take their * place until they unblock (see below). Implementing this * requires counts of the number of "running" threads (i.e., those * that are neither blocked nor artifically suspended) as well as * the total number. These two values are packed into one field, * "workerCounts" because we need accurate snapshots when deciding - * to create, resume or suspend. To support these decisions, - * updates must be prospective (not retrospective). For example, - * the running count is decremented before blocking by a thread - * about to block, but incremented by the thread about to unblock - * it. (In a few cases, these prospective updates may need to be - * rolled back, for example when deciding to create a new worker - * but the thread factory fails or returns null. In these cases, - * we are no worse off wrt other decisions than we would be - * otherwise.) Updates to the workerCounts field sometimes - * transiently encounter a fair amount of contention when join - * dependencies are such that many threads block or unblock at - * about the same time. We alleviate this by sometimes bundling - * updates (for example blocking one thread on join and resuming a - * spare cancel each other out), and in most other cases - * performing an alternative action (like releasing waiters and - * finding spares; see below) as a more productive form of - * backoff. + * to create, resume or suspend. Note however that the + * correspondance of these counts to reality is not guaranteed. In + * particular updates for unblocked threads may lag until they + * actually wake up. * * 3. Maintaining global run state. The run state of the pool * consists of a runLevel (SHUTDOWN, TERMINATING, etc) similar to @@ -201,7 +259,7 @@ public class ForkJoinPool extends Abstra * workers that previously could not find a task to now find one: * Submission of a new task to the pool, or another worker pushing * a task onto a previously empty queue. (We also use this - * mechanism for termination and reconfiguration actions that + * mechanism for configuration and termination actions that * require wakeups of idle workers). Each worker maintains its * last known event count, and blocks when a scan for work did not * find a task AND its lastEventCount matches the current @@ -212,95 +270,71 @@ public class ForkJoinPool extends Abstra * a record (field nextEventWaiter) for the next waiting worker. * In addition to allowing simpler decisions about need for * wakeup, the event count bits in eventWaiters serve the role of - * tags to avoid ABA errors in Treiber stacks. To reduce delays - * in task diffusion, workers not otherwise occupied may invoke - * method releaseWaiters, that removes and signals (unparks) - * workers not waiting on current count. To minimize task - * production stalls associate with signalling, any worker pushing - * a task on an empty queue invokes the weaker method signalWork, - * that only releases idle workers until it detects interference - * by other threads trying to release, and lets them take - * over. The net effect is a tree-like diffusion of signals, where - * released threads and possibly others) help with unparks. To - * further reduce contention effects a bit, failed CASes to - * increment field eventCount are tolerated without retries. + * tags to avoid ABA errors in Treiber stacks. Upon any wakeup, + * released threads also try to release others (but give up upon + * contention to reduce useless flailing). The net effect is a + * tree-like diffusion of signals, where released threads (and + * possibly others) help with unparks. To further reduce + * contention effects a bit, failed CASes to increment field + * eventCount are tolerated without retries in signalWork. * Conceptually they are merged into the same event, which is OK * when their only purpose is to enable workers to scan for work. * * 5. Managing suspension of extra workers. When a worker is about * to block waiting for a join (or via ManagedBlockers), we may * create a new thread to maintain parallelism level, or at least - * avoid starvation (see below). Usually, extra threads are needed - * for only very short periods, yet join dependencies are such - * that we sometimes need them in bursts. Rather than create new - * threads each time this happens, we suspend no-longer-needed - * extra ones as "spares". For most purposes, we don't distinguish - * "extra" spare threads from normal "core" threads: On each call - * to preStep (the only point at which we can do this) a worker + * avoid starvation. Usually, extra threads are needed for only + * very short periods, yet join dependencies are such that we + * sometimes need them in bursts. Rather than create new threads + * each time this happens, we suspend no-longer-needed extra ones + * as "spares". For most purposes, we don't distinguish "extra" + * spare threads from normal "core" threads: On each call to + * preStep (the only point at which we can do this) a worker * checks to see if there are now too many running workers, and if - * so, suspends itself. Methods preJoin and doBlock look for + * so, suspends itself. Method helpMaintainParallelism looks for * suspended threads to resume before considering creating a new - * replacement. We don't need a special data structure to maintain - * spares; simply scanning the workers array looking for - * worker.isSuspended() is fine because the calling thread is - * otherwise not doing anything useful anyway; we are at least as - * happy if after locating a spare, the caller doesn't actually - * block because the join is ready before we try to adjust and - * compensate. Note that this is intrinsically racy. One thread + * replacement. The spares themselves are encoded on another + * variant of a Treiber Stack, headed at field "spareWaiters". + * Note that the use of spares is intrinsically racy. One thread * may become a spare at about the same time as another is * needlessly being created. We counteract this and related slop * in part by requiring resumed spares to immediately recheck (in - * preStep) to see whether they they should re-suspend. The only - * effective difference between "extra" and "core" threads is that - * we allow the "extra" ones to time out and die if they are not - * resumed within a keep-alive interval of a few seconds. This is - * implemented mainly within ForkJoinWorkerThread, but requires - * some coordination (isTrimmed() -- meaning killed while - * suspended) to correctly maintain pool counts. - * - * 6. Deciding when to create new workers. The main dynamic - * control in this class is deciding when to create extra threads, - * in methods preJoin and doBlock. We always need to create one - * when the number of running threads becomes zero. But because - * blocked joins are typically dependent, we don't necessarily - * need or want one-to-one replacement. Using a one-to-one - * compensation rule often leads to enough useless overhead - * creating, suspending, resuming, and/or killing threads to - * signficantly degrade throughput. We use a rule reflecting the - * idea that, the more spare threads you already have, the more - * evidence you need to create another one; where "evidence" is - * expressed as the current deficit -- target minus running - * threads. To reduce flickering and drift around target values, - * the relation is quadratic: adding a spare if (dc*dc)>=(sc*pc) - * (where dc is deficit, sc is number of spare threads and pc is - * target parallelism.) This effectively reduces churn at the - * price of systematically undershooting target parallelism when - * many threads are blocked. However, biasing toward undeshooting - * partially compensates for the above mechanics to suspend extra - * threads, that normally lead to overshoot because we can only - * suspend workers in-between top-level actions. It also better - * copes with the fact that some of the methods in this class tend - * to never become compiled (but are interpreted), so some - * components of the entire set of controls might execute many + * preStep) to see whether they they should re-suspend. + * + * 6. Killing off unneeded workers. The Spare and Event queues use + * similar mechanisms to shed unused workers: The oldest (first) + * waiter uses a timed rather than hard wait. When this wait times + * out without a normal wakeup, it tries to shutdown any one (for + * convenience the newest) other waiter via tryShutdownSpare or + * tryShutdownWaiter, respectively. The wakeup rates for spares + * are much shorter than for waiters. Together, they will + * eventually reduce the number of worker threads to a minimum of + * one after a long enough period without use. + * + * 7. Deciding when to create new workers. The main dynamic + * control in this class is deciding when to create extra threads + * in method helpMaintainParallelism. We would like to keep + * exactly #parallelism threads running, which is an impossble + * task. We always need to create one when the number of running + * threads would become zero and all workers are busy. Beyond + * this, we must rely on heuristics that work well in the the + * presence of transients phenomena such as GC stalls, dynamic + * compilation, and wake-up lags. These transients are extremely + * common -- we are normally trying to fully saturate the CPUs on + * a machine, so almost any activity other than running tasks + * impedes accuracy. Our main defense is to allow some slack in + * creation thresholds, using rules that reflect the fact that the + * more threads we have running, the more likely that we are + * underestimating the number running threads. (We also include + * some heuristic use of Thread.yield when all workers appear to + * be busy, to improve likelihood of counts settling.) The rules + * also better cope with the fact that some of the methods in this + * class tend to never become compiled (but are interpreted), so + * some components of the entire set of controls might execute 100 * times faster than others. And similarly for cases where the * apparent lack of work is just due to GC stalls and other * transient system activity. * - * 7. Maintaining other configuration parameters and monitoring - * statistics. Updates to fields controlling parallelism level, - * max size, etc can only meaningfully take effect for individual - * threads upon their next top-level actions; i.e., between - * stealing/running tasks/submission, which are separated by calls - * to preStep. Memory ordering for these (assumed infrequent) - * reconfiguration calls is ensured by using reads and writes to - * volatile field workerCounts (that must be read in preStep anyway) - * as "fences" -- user-level reads are preceded by reads of - * workCounts, and writes are followed by no-op CAS to - * workerCounts. The values reported by other management and - * monitoring methods are either computed on demand, or are kept - * in fields that are only updated when threads are otherwise - * idle. - * * Beware that there is a lot of representation-level coupling * among classes ForkJoinPool, ForkJoinWorkerThread, and * ForkJoinTask. For example, direct access to "workers" array by @@ -312,11 +346,13 @@ public class ForkJoinPool extends Abstra * * Style notes: There are lots of inline assignments (of form * "while ((local = field) != 0)") which are usually the simplest - * way to ensure read orderings. Also several occurrences of the - * unusual "do {} while(!cas...)" which is the simplest way to - * force an update of a CAS'ed variable. There are also a few - * other coding oddities that help some methods perform reasonably - * even when interpreted (not compiled). + * way to ensure the required read orderings (which are sometimes + * critical). Also several occurrences of the unusual "do {} + * while(!cas...)" which is the simplest way to force an update of + * a CAS'ed variable. There are also other coding oddities that + * help some methods perform reasonably even when interpreted (not + * compiled), at the expense of some messy constructions that + * reduce byte code counts. * * The order of declarations in this file is: (1) statics (2) * fields (along with constants used when unpacking some of them) @@ -345,7 +381,7 @@ public class ForkJoinPool extends Abstra * Default ForkJoinWorkerThreadFactory implementation; creates a * new ForkJoinWorkerThread. */ - static class DefaultForkJoinWorkerThreadFactory + static class DefaultForkJoinWorkerThreadFactory implements ForkJoinWorkerThreadFactory { public ForkJoinWorkerThread newThread(ForkJoinPool pool) { return new ForkJoinWorkerThread(pool); @@ -384,10 +420,21 @@ public class ForkJoinPool extends Abstra new AtomicInteger(); /** - * Absolute bound for parallelism level. Twice this number must - * fit into a 16bit field to enable word-packing for some counts. + * The wakeup interval (in nanoseconds) for the oldest worker + * worker waiting for an event invokes tryShutdownWaiter to shrink + * the number of workers. The exact value does not matter too + * much, but should be long enough to slowly release resources + * during long periods without use without disrupting normal use. */ - private static final int MAX_THREADS = 0x7fff; + private static final long SHRINK_RATE_NANOS = + 60L * 1000L * 1000L * 1000L; // one minute + + /** + * Absolute bound for parallelism level. Twice this number plus + * one (i.e., 0xfff) must fit into a 16bit field to enable + * word-packing for some counts and indices. + */ + private static final int MAX_WORKERS = 0x7fff; /** * Array holding all worker threads in the pool. Array size must @@ -413,7 +460,7 @@ public class ForkJoinPool extends Abstra /** * Latch released upon termination. */ - private final CountDownLatch terminationLatch; + private final Phaser termination; /** * Creation factory for worker threads. @@ -429,23 +476,34 @@ public class ForkJoinPool extends Abstra /** * Encoded record of top of treiber stack of threads waiting for * events. The top 32 bits contain the count being waited for. The - * bottom word contains one plus the pool index of waiting worker - * thread. + * bottom 16 bits contains one plus the pool index of waiting + * worker thread. (Bits 16-31 are unused.) */ private volatile long eventWaiters; private static final int EVENT_COUNT_SHIFT = 32; - private static final long WAITER_INDEX_MASK = (1L << EVENT_COUNT_SHIFT)-1L; + private static final long WAITER_ID_MASK = (1L << 16) - 1L; /** * A counter for events that may wake up worker threads: * - Submission of a new task to the pool * - A worker pushing a task on an empty queue - * - termination and reconfiguration + * - termination */ private volatile int eventCount; /** + * Encoded record of top of treiber stack of spare threads waiting + * for resumption. The top 16 bits contain an arbitrary count to + * avoid ABA effects. The bottom 16bits contains one plus the pool + * index of waiting worker thread. + */ + private volatile int spareWaiters; + + private static final int SPARE_COUNT_SHIFT = 16; + private static final int SPARE_ID_MASK = (1 << 16) - 1; + + /** * Lifecycle control. The low word contains the number of workers * that are (probably) executing tasks. This value is atomically * incremented before a worker gets a task to run, and decremented @@ -456,8 +514,12 @@ public class ForkJoinPool extends Abstra * These are bundled together to ensure consistent read for * termination checks (i.e., that runLevel is at least SHUTDOWN * and active threads is zero). + * + * Notes: Most direct CASes are dependent on these bitfield + * positions. Also, this field is non-private to enable direct + * performance-sensitive CASes in ForkJoinWorkerThread. */ - private volatile int runState; + volatile int runState; // Note: The order among run level values matters. private static final int RUNLEVEL_SHIFT = 16; @@ -465,7 +527,6 @@ public class ForkJoinPool extends Abstra private static final int TERMINATING = 1 << (RUNLEVEL_SHIFT + 1); private static final int TERMINATED = 1 << (RUNLEVEL_SHIFT + 2); private static final int ACTIVE_COUNT_MASK = (1 << RUNLEVEL_SHIFT) - 1; - private static final int ONE_ACTIVE = 1; // active update delta /** * Holds number of total (i.e., created and not yet terminated) @@ -474,8 +535,7 @@ public class ForkJoinPool extends Abstra * making decisions about creating and suspending spare * threads. Updated only by CAS. Note that adding a new worker * requires incrementing both counts, since workers start off in - * running state. This field is also used for memory-fencing - * configuration parameters. + * running state. */ private volatile int workerCounts; @@ -484,74 +544,83 @@ public class ForkJoinPool extends Abstra private static final int ONE_RUNNING = 1; private static final int ONE_TOTAL = 1 << TOTAL_COUNT_SHIFT; - /* - * Fields parallelism. maxPoolSize, locallyFifo, - * maintainsParallelism, and ueh are non-volatile, but external - * reads/writes use workerCount fences to ensure visability. - */ - /** * The target parallelism level. + * Accessed directly by ForkJoinWorkerThreads. */ - private int parallelism; - - /** - * The maximum allowed pool size. - */ - private int maxPoolSize; + final int parallelism; /** * True if use local fifo, not default lifo, for local polling - * Replicated by ForkJoinWorkerThreads + * Read by, and replicated by ForkJoinWorkerThreads */ - private boolean locallyFifo; + final boolean locallyFifo; /** - * Controls whether to add spares to maintain parallelism + * The uncaught exception handler used when any worker abruptly + * terminates. */ - private boolean maintainsParallelism; - - /** - * The uncaught exception handler used when any worker - * abruptly terminates - */ - private Thread.UncaughtExceptionHandler ueh; + private final Thread.UncaughtExceptionHandler ueh; /** * Pool number, just for assigning useful names to worker threads */ private final int poolNumber; - // utilities for updating fields + + // Utilities for CASing fields. Note that most of these + // are usually manually inlined by callers /** - * Adds delta to running count. Used mainly by ForkJoinTask. - * - * @param delta the number to add + * Increments running count part of workerCounts */ - final void updateRunningCount(int delta) { - int wc; + final void incrementRunningCount() { + int c; do {} while (!UNSAFE.compareAndSwapInt(this, workerCountsOffset, - wc = workerCounts, - wc + delta)); + c = workerCounts, + c + ONE_RUNNING)); } /** - * Write fence for user modifications of pool parameters - * (parallelism. etc). Note that it doesn't matter if CAS fails. + * Tries to decrement running count unless already zero */ - private void workerCountWriteFence() { - int wc; - UNSAFE.compareAndSwapInt(this, workerCountsOffset, - wc = workerCounts, wc); + final boolean tryDecrementRunningCount() { + int wc = workerCounts; + if ((wc & RUNNING_COUNT_MASK) == 0) + return false; + return UNSAFE.compareAndSwapInt(this, workerCountsOffset, + wc, wc - ONE_RUNNING); } /** - * Read fence for external reads of pool parameters - * (parallelism. maxPoolSize, etc). + * Forces decrement of encoded workerCounts, awaiting nonzero if + * (rarely) necessary when other count updates lag. + * + * @param dr -- either zero or ONE_RUNNING + * @param dt == either zero or ONE_TOTAL */ - private void workerCountReadFence() { - int ignore = workerCounts; + private void decrementWorkerCounts(int dr, int dt) { + for (;;) { + int wc = workerCounts; + if ((wc & RUNNING_COUNT_MASK) - dr < 0 || + (wc >>> TOTAL_COUNT_SHIFT) - dt < 0) { + if ((runState & TERMINATED) != 0) + return; // lagging termination on a backout + Thread.yield(); + } + if (UNSAFE.compareAndSwapInt(this, workerCountsOffset, + wc, wc - (dr + dt))) + return; + } + } + + /** + * Increments event count + */ + private void advanceEventCount() { + int c; + do {} while(!UNSAFE.compareAndSwapInt(this, eventCountOffset, + c = eventCount, c+1)); } /** @@ -563,7 +632,7 @@ public class ForkJoinPool extends Abstra final boolean tryIncrementActiveCount() { int c; return UNSAFE.compareAndSwapInt(this, runStateOffset, - c = runState, c + ONE_ACTIVE); + c = runState, c + 1); } /** @@ -573,7 +642,7 @@ public class ForkJoinPool extends Abstra final boolean tryDecrementActiveCount() { int c; return UNSAFE.compareAndSwapInt(this, runStateOffset, - c = runState, c - ONE_ACTIVE); + c = runState, c - 1); } /** @@ -602,12 +671,12 @@ public class ForkJoinPool extends Abstra lock.lock(); try { ForkJoinWorkerThread[] ws = workers; - int len = ws.length; - if (k < 0 || k >= len || ws[k] != null) { - for (k = 0; k < len && ws[k] != null; ++k) + int n = ws.length; + if (k < 0 || k >= n || ws[k] != null) { + for (k = 0; k < n && ws[k] != null; ++k) ; - if (k == len) - ws = Arrays.copyOf(ws, len << 1); + if (k == n) + ws = Arrays.copyOf(ws, n << 1); } ws[k] = w; workers = ws; // volatile array write ensures slot visibility @@ -641,7 +710,7 @@ public class ForkJoinPool extends Abstra * are already updated to accommodate the worker, so adjusts on * failure. * - * @return new worker or null if creation failed + * @return the worker, or null on failure */ private ForkJoinWorkerThread addWorker() { ForkJoinWorkerThread w = null; @@ -649,390 +718,438 @@ public class ForkJoinPool extends Abstra w = factory.newThread(this); } finally { // Adjust on either null or exceptional factory return if (w == null) { - onWorkerCreationFailure(); - return null; + decrementWorkerCounts(ONE_RUNNING, ONE_TOTAL); + tryTerminate(false); // in case of failure during shutdown } } - w.start(recordWorker(w), locallyFifo, ueh); - return w; - } - - /** - * Adjusts counts upon failure to create worker - */ - private void onWorkerCreationFailure() { - int c; - do {} while (!UNSAFE.compareAndSwapInt(this, workerCountsOffset, - c = workerCounts, - c - (ONE_RUNNING|ONE_TOTAL))); - tryTerminate(false); // in case of failure during shutdown - } - - /** - * Create enough total workers to establish target parallelism, - * giving up if terminating or addWorker fails - */ - private void ensureEnoughTotalWorkers() { - int wc; - while (runState < TERMINATING && - ((wc = workerCounts) >>> TOTAL_COUNT_SHIFT) < parallelism) { - if ((UNSAFE.compareAndSwapInt(this, workerCountsOffset, - wc, wc + (ONE_RUNNING|ONE_TOTAL)) && - addWorker() == null)) - break; + if (w != null) { + w.start(recordWorker(w), ueh); + advanceEventCount(); } + return w; } /** * Final callback from terminating worker. Removes record of * worker from array, and adjusts counts. If pool is shutting - * down, tries to complete terminatation, else possibly replaces - * the worker. + * down, tries to complete terminatation. * * @param w the worker */ final void workerTerminated(ForkJoinWorkerThread w) { - if (w.active) { // force inactive - w.active = false; - do {} while (!tryDecrementActiveCount()); - } forgetWorker(w); - - // decrement total count, and if was running, running count - int unit = w.isTrimmed()? ONE_TOTAL : (ONE_RUNNING|ONE_TOTAL); - int wc; - do {} while (!UNSAFE.compareAndSwapInt(this, workerCountsOffset, - wc = workerCounts, wc - unit)); - - accumulateStealCount(w); // collect final count - if (!tryTerminate(false)) - ensureEnoughTotalWorkers(); + decrementWorkerCounts(w.isTrimmed()? 0 : ONE_RUNNING, ONE_TOTAL); + while (w.stealCount != 0) // collect final count + tryAccumulateStealCount(w); + tryTerminate(false); } // Waiting for and signalling events /** - * Ensures eventCount on exit is different (mod 2^32) than on - * entry. CAS failures are OK -- any change in count suffices. - */ - private void advanceEventCount() { - int c; - UNSAFE.compareAndSwapInt(this, eventCountOffset, c = eventCount, c+1); + * Releases workers blocked on a count not equal to current count. + * Normally called after precheck that eventWaiters isn't zero to + * avoid wasted array checks. Gives up upon a change in count or + * contention, letting other workers take over. + */ + private void releaseEventWaiters() { + ForkJoinWorkerThread[] ws = workers; + int n = ws.length; + long h = eventWaiters; + int ec = eventCount; + ForkJoinWorkerThread w; int id; + while ((int)(h >>> EVENT_COUNT_SHIFT) != ec && + (id = ((int)(h & WAITER_ID_MASK)) - 1) >= 0 && + id < n && (w = ws[id]) != null && + UNSAFE.compareAndSwapLong(this, eventWaitersOffset, + h, h = w.nextWaiter)) { + LockSupport.unpark(w); + if (eventWaiters != h || eventCount != ec) + break; + } } /** - * Releases workers blocked on a count not equal to current count. + * Tries to advance eventCount and releases waiters. Called only + * from workers. */ - final void releaseWaiters() { - long top; - int id; - while ((id = (int)((top = eventWaiters) & WAITER_INDEX_MASK)) > 0 && - (int)(top >>> EVENT_COUNT_SHIFT) != eventCount) { - ForkJoinWorkerThread[] ws = workers; - ForkJoinWorkerThread w; - if (ws.length >= id && (w = ws[id - 1]) != null && - UNSAFE.compareAndSwapLong(this, eventWaitersOffset, - top, w.nextWaiter)) - LockSupport.unpark(w); - } + final void signalWork() { + int c; // try to increment event count -- CAS failure OK + UNSAFE.compareAndSwapInt(this, eventCountOffset, c = eventCount, c+1); + if (eventWaiters != 0L) + releaseEventWaiters(); } /** - * Advances eventCount and releases waiters until interference by - * other releasing threads is detected. + * Adds the given worker to event queue and blocks until + * terminating or event count advances from the workers + * lastEventCount value + * + * @param w the calling worker thread */ - final void signalWork() { - int ec; - UNSAFE.compareAndSwapInt(this, eventCountOffset, ec=eventCount, ec+1); - outer:for (;;) { - long top = eventWaiters; - ec = eventCount; - for (;;) { - ForkJoinWorkerThread[] ws; ForkJoinWorkerThread w; - int id = (int)(top & WAITER_INDEX_MASK); - if (id <= 0 || (int)(top >>> EVENT_COUNT_SHIFT) == ec) - return; - if ((ws = workers).length < id || (w = ws[id - 1]) == null || - !UNSAFE.compareAndSwapLong(this, eventWaitersOffset, - top, top = w.nextWaiter)) - continue outer; // possibly stale; reread - LockSupport.unpark(w); - if (top != eventWaiters) // let someone else take over - return; + private void eventSync(ForkJoinWorkerThread w) { + int ec = w.lastEventCount; + long nh = (((long)ec) << EVENT_COUNT_SHIFT) | ((long)(w.poolIndex+1)); + long h; + while ((runState < SHUTDOWN || !tryTerminate(false)) && + (((int)((h = eventWaiters) & WAITER_ID_MASK)) == 0 || + (int)(h >>> EVENT_COUNT_SHIFT) == ec) && + eventCount == ec) { + if (UNSAFE.compareAndSwapLong(this, eventWaitersOffset, + w.nextWaiter = h, nh)) { + awaitEvent(w, ec); + break; } } } /** - * If worker is inactive, blocks until terminating or event count - * advances from last value held by worker; in any case helps - * release others. + * Blocks the given worker (that has already been entered as an + * event waiter) until terminating or event count advances from + * the given value. The oldest (first) waiter uses a timed wait to + * occasionally one-by-one shrink the number of workers (to a + * minumum of one) if the pool has not been used for extended + * periods. * * @param w the calling worker thread + * @param ec the count */ - private void eventSync(ForkJoinWorkerThread w) { - if (!w.active) { - int prev = w.lastEventCount; - long nextTop = (((long)prev << EVENT_COUNT_SHIFT) | - ((long)(w.poolIndex + 1))); - long top; - while ((runState < SHUTDOWN || !tryTerminate(false)) && - (((int)(top = eventWaiters) & WAITER_INDEX_MASK) == 0 || - (int)(top >>> EVENT_COUNT_SHIFT) == prev) && - eventCount == prev) { - if (UNSAFE.compareAndSwapLong(this, eventWaitersOffset, - w.nextWaiter = top, nextTop)) { - accumulateStealCount(w); // transfer steals while idle - Thread.interrupted(); // clear/ignore interrupt - while (eventCount == prev) - w.doPark(); + private void awaitEvent(ForkJoinWorkerThread w, int ec) { + while (eventCount == ec) { + if (tryAccumulateStealCount(w)) { // transfer while idle + boolean untimed = (w.nextWaiter != 0L || + (workerCounts & RUNNING_COUNT_MASK) <= 1); + long startTime = untimed? 0 : System.nanoTime(); + Thread.interrupted(); // clear/ignore interrupt + if (eventCount != ec || !w.isRunning() || + runState >= TERMINATING) // recheck after clear break; + if (untimed) + LockSupport.park(w); + else { + LockSupport.parkNanos(w, SHRINK_RATE_NANOS); + if (eventCount != ec || !w.isRunning() || + runState >= TERMINATING) + break; + if (System.nanoTime() - startTime >= SHRINK_RATE_NANOS) + tryShutdownWaiter(ec); } } - w.lastEventCount = eventCount; } - releaseWaiters(); } /** - * Callback from workers invoked upon each top-level action (i.e., - * stealing a task or taking a submission and running - * it). Performs one or both of the following: - * - * * If the worker cannot find work, updates its active status to - * inactive and updates activeCount unless there is contention, in - * which case it may try again (either in this or a subsequent - * call). Additionally, awaits the next task event and/or helps - * wake up other releasable waiters. - * - * * If there are too many running threads, suspends this worker - * (first forcing inactivation if necessary). If it is not - * resumed before a keepAlive elapses, the worker may be "trimmed" - * -- killed while suspended within suspendAsSpare. Otherwise, - * upon resume it rechecks to make sure that it is still needed. + * Callback from the oldest waiter in awaitEvent waking up after a + * period of non-use. Tries (once) to shutdown an event waiter (or + * a spare, if one exists). Note that we don't need CAS or locks + * here because the method is called only from one thread + * occasionally waking (and even misfires are OK). Note that + * until the shutdown worker fully terminates, workerCounts + * will overestimate total count, which is tolerable. + * + * @param ec the event count waited on by caller (to abort + * attempt if count has since changed). + */ + private void tryShutdownWaiter(int ec) { + if (spareWaiters != 0) { // prefer killing spares + tryShutdownSpare(); + return; + } + ForkJoinWorkerThread[] ws = workers; + int n = ws.length; + long h = eventWaiters; + ForkJoinWorkerThread w; int id; long nh; + if (runState == 0 && + submissionQueue.isEmpty() && + eventCount == ec && + (id = ((int)(h & WAITER_ID_MASK)) - 1) >= 0 && + id < n && (w = ws[id]) != null && + (nh = w.nextWaiter) != 0L && // keep at least one worker + UNSAFE.compareAndSwapLong(this, eventWaitersOffset, h, nh)) { + w.shutdown(); + LockSupport.unpark(w); + } + releaseEventWaiters(); + } + + // Maintaining spares + + /** + * Pushes worker onto the spare stack + */ + final void pushSpare(ForkJoinWorkerThread w) { + int ns = (++w.spareCount << SPARE_COUNT_SHIFT) | (w.poolIndex + 1); + do {} while (!UNSAFE.compareAndSwapInt(this, spareWaitersOffset, + w.nextSpare = spareWaiters,ns)); + } + + /** + * Callback from oldest spare occasionally waking up. Tries + * (once) to shutdown a spare. Same idea as tryShutdownWaiter. + */ + final void tryShutdownSpare() { + int sw, id; + ForkJoinWorkerThread w; + ForkJoinWorkerThread[] ws; + if ((id = ((sw = spareWaiters) & SPARE_ID_MASK) - 1) >= 0 && + id < (ws = workers).length && (w = ws[id]) != null && + (workerCounts & RUNNING_COUNT_MASK) >= parallelism && + UNSAFE.compareAndSwapInt(this, spareWaitersOffset, + sw, w.nextSpare)) { + w.shutdown(); + LockSupport.unpark(w); + advanceEventCount(); + } + } + + /** + * Tries (once) to resume a spare if worker counts match + * the given count. * - * @param w the worker - * @param worked false if the worker scanned for work but didn't - * find any (in which case it may block waiting for work). + * @param wc workerCounts value on invocation of this method */ - final void preStep(ForkJoinWorkerThread w, boolean worked) { - boolean active = w.active; - boolean inactivate = !worked & active; - for (;;) { - if (inactivate) { - int c = runState; - if (UNSAFE.compareAndSwapInt(this, runStateOffset, - c, c - ONE_ACTIVE)) - inactivate = active = w.active = false; - } - int wc = workerCounts; - if ((wc & RUNNING_COUNT_MASK) <= parallelism) { - if (!worked) - eventSync(w); - return; + private void tryResumeSpare(int wc) { + ForkJoinWorkerThread[] ws = workers; + int n = ws.length; + int sw, id, rs; ForkJoinWorkerThread w; + if ((id = ((sw = spareWaiters) & SPARE_ID_MASK) - 1) >= 0 && + id < n && (w = ws[id]) != null && + (rs = runState) < TERMINATING && + eventWaiters == 0L && workerCounts == wc) { + // In case all workers busy, heuristically back off to let settle + Thread.yield(); + if (eventWaiters == 0L && runState == rs && // recheck + workerCounts == wc && spareWaiters == sw && + UNSAFE.compareAndSwapInt(this, spareWaitersOffset, + sw, w.nextSpare)) { + int c; // increment running count before resume + do {} while(!UNSAFE.compareAndSwapInt + (this, workerCountsOffset, + c = workerCounts, c + ONE_RUNNING)); + if (w.tryUnsuspend()) + LockSupport.unpark(w); + else // back out if w was shutdown + decrementWorkerCounts(ONE_RUNNING, 0); } - if (!(inactivate |= active) && // must inactivate to suspend - UNSAFE.compareAndSwapInt(this, workerCountsOffset, - wc, wc - ONE_RUNNING) && - !w.suspendAsSpare()) // false if trimmed - return; } } + // adding workers on demand + /** - * Adjusts counts and creates or resumes compensating threads for - * a worker about to block on task joinMe, returning early if - * joinMe becomes ready. First tries resuming an existing spare - * (which usually also avoids any count adjustment), but must then - * decrement running count to determine whether a new thread is - * needed. See above for fuller explanation. + * Adds one or more workers if needed to establish target parallelism. + * Retries upon contention. */ - final void preJoin(ForkJoinTask joinMe) { - boolean dec = false; // true when running count decremented - for (;;) { - releaseWaiters(); // help other threads progress - - if (joinMe.status < 0) // surround spare search with done checks - return; - ForkJoinWorkerThread spare = null; - for (ForkJoinWorkerThread w : workers) { - if (w != null && w.isSuspended()) { - spare = w; + private void addWorkerIfBelowTarget() { + int pc = parallelism; + int wc; + while (((wc = workerCounts) >>> TOTAL_COUNT_SHIFT) < pc && + runState < TERMINATING) { + if (UNSAFE.compareAndSwapInt(this, workerCountsOffset, wc, + wc + (ONE_RUNNING|ONE_TOTAL))) { + if (addWorker() == null) break; - } - } - if (joinMe.status < 0) - return; - - if (spare != null && spare.tryUnsuspend()) { - if (dec || joinMe.requestSignal() < 0) { - int c; - do {} while (!UNSAFE.compareAndSwapInt(this, - workerCountsOffset, - c = workerCounts, - c + ONE_RUNNING)); - } // else no net count change - LockSupport.unpark(spare); - return; } + } + } - int wc = workerCounts; // decrement running count - if (!dec && (wc & RUNNING_COUNT_MASK) != 0 && - (dec = UNSAFE.compareAndSwapInt(this, workerCountsOffset, - wc, wc -= ONE_RUNNING)) && - joinMe.requestSignal() < 0) { // cannot block - int c; // back out - do {} while (!UNSAFE.compareAndSwapInt(this, - workerCountsOffset, - c = workerCounts, - c + ONE_RUNNING)); - return; - } + /** + * Tries (once) to add a new worker if all existing workers are + * busy, and there are either no running workers or the deficit is + * at least twice the surplus. + * + * @param wc workerCounts value on invocation of this method + */ + private void tryAddWorkerIfBusy(int wc) { + int tc, rc, rs; + int pc = parallelism; + if ((tc = wc >>> TOTAL_COUNT_SHIFT) < MAX_WORKERS && + ((rc = wc & RUNNING_COUNT_MASK) == 0 || + rc < pc - ((tc - pc) << 1)) && + (rs = runState) < TERMINATING && + (rs & ACTIVE_COUNT_MASK) == tc) { + // Since all workers busy, heuristically back off to let settle + Thread.yield(); + if (eventWaiters == 0L && spareWaiters == 0 && // recheck + runState == rs && workerCounts == wc && + UNSAFE.compareAndSwapInt(this, workerCountsOffset, wc, + wc + (ONE_RUNNING|ONE_TOTAL))) + addWorker(); + } + } - if (dec) { - int tc = wc >>> TOTAL_COUNT_SHIFT; - int pc = parallelism; - int dc = pc - (wc & RUNNING_COUNT_MASK); // deficit count - if ((dc < pc && (dc <= 0 || (dc * dc < (tc - pc) * pc) || - !maintainsParallelism)) || - tc >= maxPoolSize) // cannot add - return; - if (spare == null && - UNSAFE.compareAndSwapInt(this, workerCountsOffset, wc, - wc + (ONE_RUNNING|ONE_TOTAL))) { - addWorker(); - return; - } - } + /** + * Does at most one of: + * + * 1. Help wake up existing workers waiting for work via + * releaseEventWaiters. (If any exist, then it doesn't + * matter right now if under target parallelism level.) + * + * 2. If a spare exists, try (once) to resume it via tryResumeSpare. + * + * 3. If there are not enough total workers, add some + * via addWorkerIfBelowTarget; + * + * 4. Try (once) to add a new worker if all existing workers + * are busy, via tryAddWorkerIfBusy + */ + private void helpMaintainParallelism() { + long h; int pc, wc; + if (((int)((h = eventWaiters) & WAITER_ID_MASK)) != 0) { + if ((int)(h >>> EVENT_COUNT_SHIFT) != eventCount) + releaseEventWaiters(); // avoid useless call + } + else if ((pc = parallelism) > + ((wc = workerCounts) & RUNNING_COUNT_MASK)) { + if (spareWaiters != 0) + tryResumeSpare(wc); + else if ((wc >>> TOTAL_COUNT_SHIFT) < pc) + addWorkerIfBelowTarget(); + else + tryAddWorkerIfBusy(wc); } } /** - * Same idea as preJoin but with too many differing details to - * integrate: There are no task-based signal counts, and only one - * way to do the actual blocking. So for simplicity it is directly - * incorporated into this method. + * Callback from workers invoked upon each top-level action (i.e., + * stealing a task or taking a submission and running it). + * Performs one or more of the following: + * + * 1. If the worker is active, try to set its active status to + * inactive and update activeCount. On contention, we may try + * again on this or subsequent call. + * + * 2. Release any existing event waiters that are now relesable + * + * 3. If there are too many running threads, suspend this worker + * (first forcing inactive if necessary). If it is not + * needed, it may be killed while suspended via + * tryShutdownSpare. Otherwise, upon resume it rechecks to make + * sure that it is still needed. + * + * 4. If more than 1 miss, await the next task event via + * eventSync (first forcing inactivation if necessary), upon + * which worker may also be killed, via tryShutdownWaiter. + * + * 5. Help reactivate other workers via helpMaintainParallelism + * + * @param w the worker + * @param misses the number of scans by caller failing to find work + * (saturating at 2 to avoid wraparound) */ - final void doBlock(ManagedBlocker blocker, boolean maintainPar) - throws InterruptedException { - maintainPar &= maintainsParallelism; // override - boolean dec = false; - boolean done = false; + final void preStep(ForkJoinWorkerThread w, int misses) { + boolean active = w.active; + int pc = parallelism; for (;;) { - releaseWaiters(); - if (done = blocker.isReleasable()) - break; - ForkJoinWorkerThread spare = null; - for (ForkJoinWorkerThread w : workers) { - if (w != null && w.isSuspended()) { - spare = w; - break; - } - } - if (done = blocker.isReleasable()) - break; - if (spare != null && spare.tryUnsuspend()) { - if (dec) { - int c; - do {} while (!UNSAFE.compareAndSwapInt(this, - workerCountsOffset, - c = workerCounts, - c + ONE_RUNNING)); + int rs, wc, rc, ec; long h; + if (active && UNSAFE.compareAndSwapInt(this, runStateOffset, + rs = runState, rs - 1)) + active = w.active = false; + if (((int)((h = eventWaiters) & WAITER_ID_MASK)) != 0 && + (int)(h >>> EVENT_COUNT_SHIFT) != eventCount) { + releaseEventWaiters(); + if (misses > 1) + continue; // clear before sync below + } + if ((rc = ((wc = workerCounts) & RUNNING_COUNT_MASK)) > pc) { + if (!active && // must inactivate to suspend + workerCounts == wc && // try to suspend as spare + UNSAFE.compareAndSwapInt(this, workerCountsOffset, + wc, wc - ONE_RUNNING)) { + w.suspendAsSpare(); + if (!w.isRunning()) + break; // was killed while spare } - LockSupport.unpark(spare); - break; + continue; } - int wc = workerCounts; - if (!dec && (wc & RUNNING_COUNT_MASK) != 0) - dec = UNSAFE.compareAndSwapInt(this, workerCountsOffset, - wc, wc -= ONE_RUNNING); - if (dec) { - int tc = wc >>> TOTAL_COUNT_SHIFT; - int pc = parallelism; - int dc = pc - (wc & RUNNING_COUNT_MASK); - if ((dc < pc && (dc <= 0 || (dc * dc < (tc - pc) * pc) || - !maintainPar)) || - tc >= maxPoolSize) - break; - if (spare == null && - UNSAFE.compareAndSwapInt(this, workerCountsOffset, wc, - wc + (ONE_RUNNING|ONE_TOTAL))){ - addWorker(); - break; + if (misses > 0) { + if ((ec = eventCount) == w.lastEventCount && misses > 1) { + if (!active) { // must inactivate to sync + eventSync(w); + if (w.isRunning()) + misses = 1; // don't re-sync + else + break; // was killed while waiting + } + continue; } + w.lastEventCount = ec; } - } - - try { - if (!done) - do {} while (!blocker.isReleasable() && !blocker.block()); - } finally { - if (dec) { - int c; - do {} while (!UNSAFE.compareAndSwapInt(this, - workerCountsOffset, - c = workerCounts, - c + ONE_RUNNING)); - } + if (rc < pc) + helpMaintainParallelism(); + break; } } /** - * Unless there are not enough other running threads, adjusts - * counts for a a worker in performing helpJoin that cannot find - * any work, so that this worker can now block. + * Helps and/or blocks awaiting join of the given task. + * Alternates between helpJoinTask() and helpMaintainParallelism() + * as many times as there is a deficit in running count (or longer + * if running count would become zero), then blocks if task still + * not done. * - * @return true if worker may block + * @param joinMe the task to join */ - final boolean preBlockHelpingJoin(ForkJoinTask joinMe) { + final void awaitJoin(ForkJoinTask joinMe, ForkJoinWorkerThread worker) { + int threshold = parallelism; // descend blocking thresholds while (joinMe.status >= 0) { - releaseWaiters(); // help other threads progress - - // if a spare exists, resume it to maintain parallelism level - if ((workerCounts & RUNNING_COUNT_MASK) <= parallelism) { - ForkJoinWorkerThread spare = null; - for (ForkJoinWorkerThread w : workers) { - if (w != null && w.isSuspended()) { - spare = w; - break; - } - } - if (joinMe.status < 0) - break; - if (spare != null) { - if (spare.tryUnsuspend()) { - boolean canBlock = true; - if (joinMe.requestSignal() < 0) { - canBlock = false; // already done - int c; - do {} while (!UNSAFE.compareAndSwapInt - (this, workerCountsOffset, - c = workerCounts, c + ONE_RUNNING)); - } - LockSupport.unpark(spare); - return canBlock; - } - continue; // recheck -- another spare may exist - } - } - - int wc = workerCounts; // reread to shorten CAS window - int rc = wc & RUNNING_COUNT_MASK; - if (rc <= 2) // keep this and at most one other thread alive + boolean block; int wc; + worker.helpJoinTask(joinMe); + if (joinMe.status < 0) break; - - if (UNSAFE.compareAndSwapInt(this, workerCountsOffset, - wc, wc - ONE_RUNNING)) { - if (joinMe.requestSignal() >= 0) - return true; - int c; // back out + if (((wc = workerCounts) & RUNNING_COUNT_MASK) <= threshold) { + if (threshold > 0) + --threshold; + else + advanceEventCount(); // force release + block = false; + } + else + block = UNSAFE.compareAndSwapInt(this, workerCountsOffset, + wc, wc - ONE_RUNNING); + helpMaintainParallelism(); + if (block) { + int c; + joinMe.internalAwaitDone(); do {} while (!UNSAFE.compareAndSwapInt (this, workerCountsOffset, c = workerCounts, c + ONE_RUNNING)); break; } } - return false; + } + + /** + * Same idea as awaitJoin, but no helping + */ + final void awaitBlocker(ManagedBlocker blocker) + throws InterruptedException { + int threshold = parallelism; + while (!blocker.isReleasable()) { + boolean block; int wc; + if (((wc = workerCounts) & RUNNING_COUNT_MASK) <= threshold) { + if (threshold > 0) + --threshold; + else + advanceEventCount(); + block = false; + } + else + block = UNSAFE.compareAndSwapInt(this, workerCountsOffset, + wc, wc - ONE_RUNNING); + helpMaintainParallelism(); + if (block) { + try { + do {} while (!blocker.isReleasable() && !blocker.block()); + } finally { + int c; + do {} while (!UNSAFE.compareAndSwapInt + (this, workerCountsOffset, + c = workerCounts, c + ONE_RUNNING)); + } + break; + } + } } /** @@ -1056,16 +1173,51 @@ public class ForkJoinPool extends Abstra // Finish now if all threads terminated; else in some subsequent call if ((workerCounts >>> TOTAL_COUNT_SHIFT) == 0) { advanceRunLevel(TERMINATED); - terminationLatch.countDown(); + termination.arrive(); } return true; } /** * Actions on transition to TERMINATING + * + * Runs up to four passes through workers: (0) shutting down each + * (without waking up if parked) to quickly spread notifications + * without unnecessary bouncing around event queues etc (1) wake + * up and help cancel tasks (2) interrupt (3) mop up races with + * interrupted workers */ private void startTerminating() { - // Clear out and cancel submissions, ignoring exceptions + cancelSubmissions(); + for (int passes = 0; passes < 4 && workerCounts != 0; ++passes) { + advanceEventCount(); + eventWaiters = 0L; // clobber lists + spareWaiters = 0; + ForkJoinWorkerThread[] ws = workers; + int n = ws.length; + for (int i = 0; i < n; ++i) { + ForkJoinWorkerThread w = ws[i]; + if (w != null) { + w.shutdown(); + if (passes > 0 && !w.isTerminated()) { + w.cancelTasks(); + LockSupport.unpark(w); + if (passes > 1) { + try { + w.interrupt(); + } catch (SecurityException ignore) { + } + } + } + } + } + } + } + + /** + * Clear out and cancel submissions, ignoring exceptions + */ + private void cancelSubmissions() { ForkJoinTask task; while ((task = submissionQueue.poll()) != null) { try { @@ -1073,28 +1225,6 @@ public class ForkJoinPool extends Abstra } catch (Throwable ignore) { } } - // Propagate run level - for (ForkJoinWorkerThread w : workers) { - if (w != null) - w.shutdown(); // also resumes suspended workers - } - // Ensure no straggling local tasks - for (ForkJoinWorkerThread w : workers) { - if (w != null) - w.cancelTasks(); - } - // Wake up idle workers - advanceEventCount(); - releaseWaiters(); - // Unstick pending joins - for (ForkJoinWorkerThread w : workers) { - if (w != null && !w.isTerminated()) { - try { - w.interrupt(); - } catch (SecurityException ignore) { - } - } - } } // misc support for ForkJoinWorkerThread @@ -1107,17 +1237,21 @@ public class ForkJoinPool extends Abstra } /** - * Accumulates steal count from a worker, clearing - * the worker's value + * Tries to accumulates steal count from a worker, clearing + * the worker's value. + * + * @return true if worker steal count now zero */ - final void accumulateStealCount(ForkJoinWorkerThread w) { + final boolean tryAccumulateStealCount(ForkJoinWorkerThread w) { int sc = w.stealCount; - if (sc != 0) { - long c; - w.stealCount = 0; - do {} while (!UNSAFE.compareAndSwapLong(this, stealCountOffset, - c = stealCount, c + sc)); + long c = stealCount; + // CAS even if zero, for fence effects + if (UNSAFE.compareAndSwapLong(this, stealCountOffset, c, c + sc)) { + if (sc != 0) + w.stealCount = 0; + return true; } + return sc == 0; } /** @@ -1125,8 +1259,8 @@ public class ForkJoinPool extends Abstra * active thread. */ final int idlePerActive() { + int pc = parallelism; // use parallelism, not rc int ac = runState; // no mask -- artifically boosts during shutdown - int pc = parallelism; // use targeted parallelism, not rc // Use exact results for small values, saturate past 4 return pc <= ac? 0 : pc >>> 1 <= ac? 1 : pc >>> 2 <= ac? 3 : pc >>> 3; } @@ -1137,8 +1271,9 @@ public class ForkJoinPool extends Abstra /** * Creates a {@code ForkJoinPool} with parallelism equal to {@link - * java.lang.Runtime#availableProcessors}, and using the {@linkplain - * #defaultForkJoinWorkerThreadFactory default thread factory}. + * java.lang.Runtime#availableProcessors}, using the {@linkplain + * #defaultForkJoinWorkerThreadFactory default thread factory}, + * no UncaughtExceptionHandler, and non-async LIFO processing mode. * * @throws SecurityException if a security manager exists and * the caller is not permitted to modify threads @@ -1147,13 +1282,14 @@ public class ForkJoinPool extends Abstra */ public ForkJoinPool() { this(Runtime.getRuntime().availableProcessors(), - defaultForkJoinWorkerThreadFactory); + defaultForkJoinWorkerThreadFactory, null, false); } /** * Creates a {@code ForkJoinPool} with the indicated parallelism - * level and using the {@linkplain - * #defaultForkJoinWorkerThreadFactory default thread factory}. + * level, the {@linkplain + * #defaultForkJoinWorkerThreadFactory default thread factory}, + * no UncaughtExceptionHandler, and non-async LIFO processing mode. * * @param parallelism the parallelism level * @throws IllegalArgumentException if parallelism less than or @@ -1164,31 +1300,25 @@ public class ForkJoinPool extends Abstra * java.lang.RuntimePermission}{@code ("modifyThread")} */ public ForkJoinPool(int parallelism) { - this(parallelism, defaultForkJoinWorkerThreadFactory); + this(parallelism, defaultForkJoinWorkerThreadFactory, null, false); } /** - * Creates a {@code ForkJoinPool} with parallelism equal to {@link - * java.lang.Runtime#availableProcessors}, and using the given - * thread factory. + * Creates a {@code ForkJoinPool} with the given parameters. * - * @param factory the factory for creating new threads - * @throws NullPointerException if the factory is null - * @throws SecurityException if a security manager exists and - * the caller is not permitted to modify threads - * because it does not hold {@link - * java.lang.RuntimePermission}{@code ("modifyThread")} - */ - public ForkJoinPool(ForkJoinWorkerThreadFactory factory) { - this(Runtime.getRuntime().availableProcessors(), factory); - } - - /** - * Creates a {@code ForkJoinPool} with the given parallelism and - * thread factory. - * - * @param parallelism the parallelism level - * @param factory the factory for creating new threads + * @param parallelism the parallelism level. For default value, + * use {@link java.lang.Runtime#availableProcessors}. + * @param factory the factory for creating new threads. For default value, + * use {@link #defaultForkJoinWorkerThreadFactory}. + * @param handler the handler for internal worker threads that + * terminate due to unrecoverable errors encountered while executing + * tasks. For default value, use null. + * @param asyncMode if true, + * establishes local first-in-first-out scheduling mode for forked + * tasks that are never joined. This mode may be more appropriate + * than default locally stack-based mode in applications in which + * worker threads only process event-style asynchronous tasks. + * For default value, use false. * @throws IllegalArgumentException if parallelism less than or * equal to zero, or greater than implementation limit * @throws NullPointerException if the factory is null @@ -1197,25 +1327,25 @@ public class ForkJoinPool extends Abstra * because it does not hold {@link * java.lang.RuntimePermission}{@code ("modifyThread")} */ - public ForkJoinPool(int parallelism, ForkJoinWorkerThreadFactory factory) { + public ForkJoinPool(int parallelism, + ForkJoinWorkerThreadFactory factory, + Thread.UncaughtExceptionHandler handler, + boolean asyncMode) { checkPermission(); if (factory == null) throw new NullPointerException(); - if (parallelism <= 0 || parallelism > MAX_THREADS) + if (parallelism <= 0 || parallelism > MAX_WORKERS) throw new IllegalArgumentException(); - this.poolNumber = poolNumberGenerator.incrementAndGet(); - int arraySize = initialArraySizeFor(parallelism); this.parallelism = parallelism; this.factory = factory; - this.maxPoolSize = MAX_THREADS; - this.maintainsParallelism = true; + this.ueh = handler; + this.locallyFifo = asyncMode; + int arraySize = initialArraySizeFor(parallelism); this.workers = new ForkJoinWorkerThread[arraySize]; this.submissionQueue = new LinkedTransferQueue>(); this.workerLock = new ReentrantLock(); - this.terminationLatch = new CountDownLatch(1); - // Start first worker; remaining workers added upon first submission - workerCounts = ONE_RUNNING | ONE_TOTAL; - addWorker(); + this.termination = new Phaser(1); + this.poolNumber = poolNumberGenerator.incrementAndGet(); } /** @@ -1223,8 +1353,8 @@ public class ForkJoinPool extends Abstra * @param pc the initial parallelism level */ private static int initialArraySizeFor(int pc) { - // See Hackers Delight, sec 3.2. We know MAX_THREADS < (1 >>> 16) - int size = pc < MAX_THREADS ? pc + 1 : MAX_THREADS; + // See Hackers Delight, sec 3.2. We know MAX_WORKERS < (1 >>> 16) + int size = pc < MAX_WORKERS ? pc + 1 : MAX_WORKERS; size |= size >>> 1; size |= size >>> 2; size |= size >>> 4; @@ -1244,9 +1374,10 @@ public class ForkJoinPool extends Abstra throw new RejectedExecutionException(); submissionQueue.offer(task); advanceEventCount(); - releaseWaiters(); + if (eventWaiters != 0L) + releaseEventWaiters(); if ((workerCounts >>> TOTAL_COUNT_SHIFT) < parallelism) - ensureEnoughTotalWorkers(); + addWorkerIfBelowTarget(); } /** @@ -1292,6 +1423,20 @@ public class ForkJoinPool extends Abstra } /** + * Submits a ForkJoinTask for execution. + * + * @param task the task to submit + * @return the task + * @throws NullPointerException if the task is null + * @throws RejectedExecutionException if the task cannot be + * scheduled for execution + */ + public ForkJoinTask submit(ForkJoinTask task) { + doSubmit(task); + return task; + } + + /** * @throws NullPointerException if the task is null * @throws RejectedExecutionException if the task cannot be * scheduled for execution @@ -1329,20 +1474,6 @@ public class ForkJoinPool extends Abstra } /** - * Submits a ForkJoinTask for execution. - * - * @param task the task to submit - * @return the task - * @throws NullPointerException if the task is null - * @throws RejectedExecutionException if the task cannot be - * scheduled for execution - */ - public ForkJoinTask submit(ForkJoinTask task) { - doSubmit(task); - return task; - } - - /** * @throws NullPointerException {@inheritDoc} * @throws RejectedExecutionException {@inheritDoc} */ @@ -1384,80 +1515,15 @@ public class ForkJoinPool extends Abstra * @return the handler, or {@code null} if none */ public Thread.UncaughtExceptionHandler getUncaughtExceptionHandler() { - workerCountReadFence(); return ueh; } /** - * Sets the handler for internal worker threads that terminate due - * to unrecoverable errors encountered while executing tasks. - * Unless set, the current default or ThreadGroup handler is used - * as handler. - * - * @param h the new handler - * @return the old handler, or {@code null} if none - * @throws SecurityException if a security manager exists and - * the caller is not permitted to modify threads - * because it does not hold {@link - * java.lang.RuntimePermission}{@code ("modifyThread")} - */ - public Thread.UncaughtExceptionHandler - setUncaughtExceptionHandler(Thread.UncaughtExceptionHandler h) { - checkPermission(); - workerCountReadFence(); - Thread.UncaughtExceptionHandler old = ueh; - if (h != old) { - ueh = h; - workerCountWriteFence(); - for (ForkJoinWorkerThread w : workers) { - if (w != null) - w.setUncaughtExceptionHandler(h); - } - } - return old; - } - - /** - * Sets the target parallelism level of this pool. - * - * @param parallelism the target parallelism - * @throws IllegalArgumentException if parallelism less than or - * equal to zero or greater than maximum size bounds - * @throws SecurityException if a security manager exists and - * the caller is not permitted to modify threads - * because it does not hold {@link - * java.lang.RuntimePermission}{@code ("modifyThread")} - */ - public void setParallelism(int parallelism) { - checkPermission(); - if (parallelism <= 0 || parallelism > maxPoolSize) - throw new IllegalArgumentException(); - workerCountReadFence(); - int pc = this.parallelism; - if (pc != parallelism) { - this.parallelism = parallelism; - workerCountWriteFence(); - // Release spares. If too many, some will die after re-suspend - for (ForkJoinWorkerThread w : workers) { - if (w != null && w.tryUnsuspend()) { - updateRunningCount(1); - LockSupport.unpark(w); - } - } - ensureEnoughTotalWorkers(); - advanceEventCount(); - releaseWaiters(); // force config recheck by existing workers - } - } - - /** * Returns the targeted parallelism level of this pool. * * @return the targeted parallelism level of this pool */ public int getParallelism() { - // workerCountReadFence(); // inlined below - int ignore = workerCounts; return parallelism; } @@ -1474,99 +1540,12 @@ public class ForkJoinPool extends Abstra } /** - * Returns the maximum number of threads allowed to exist in the - * pool. Unless set using {@link #setMaximumPoolSize}, the - * maximum is an implementation-defined value designed only to - * prevent runaway growth. - * - * @return the maximum - */ - public int getMaximumPoolSize() { - workerCountReadFence(); - return maxPoolSize; - } - - /** - * Sets the maximum number of threads allowed to exist in the - * pool. The given value should normally be greater than or equal - * to the {@link #getParallelism parallelism} level. Setting this - * value has no effect on current pool size. It controls - * construction of new threads. The use of this method may cause - * tasks that intrinsically require extra threads for dependent - * computations to indefinitely stall. If you are instead trying - * to minimize internal thread creation, consider setting {@link - * #setMaintainsParallelism} as false. - * - * @throws IllegalArgumentException if negative or greater than - * internal implementation limit - */ - public void setMaximumPoolSize(int newMax) { - if (newMax < 0 || newMax > MAX_THREADS) - throw new IllegalArgumentException(); - maxPoolSize = newMax; - workerCountWriteFence(); - } - - /** - * Returns {@code true} if this pool dynamically maintains its - * target parallelism level. If false, new threads are added only - * to avoid possible starvation. This setting is by default true. - * - * @return {@code true} if maintains parallelism - */ - public boolean getMaintainsParallelism() { - workerCountReadFence(); - return maintainsParallelism; - } - - /** - * Sets whether this pool dynamically maintains its target - * parallelism level. If false, new threads are added only to - * avoid possible starvation. - * - * @param enable {@code true} to maintain parallelism - */ - public void setMaintainsParallelism(boolean enable) { - maintainsParallelism = enable; - workerCountWriteFence(); - } - - /** - * Establishes local first-in-first-out scheduling mode for forked - * tasks that are never joined. This mode may be more appropriate - * than default locally stack-based mode in applications in which - * worker threads only process asynchronous tasks. This method is - * designed to be invoked only when the pool is quiescent, and - * typically only before any tasks are submitted. The effects of - * invocations at other times may be unpredictable. - * - * @param async if {@code true}, use locally FIFO scheduling - * @return the previous mode - * @see #getAsyncMode - */ - public boolean setAsyncMode(boolean async) { - workerCountReadFence(); - boolean oldMode = locallyFifo; - if (oldMode != async) { - locallyFifo = async; - workerCountWriteFence(); - for (ForkJoinWorkerThread w : workers) { - if (w != null) - w.setAsyncMode(async); - } - } - return oldMode; - } - - /** * Returns {@code true} if this pool uses local first-in-first-out * scheduling mode for forked tasks that are never joined. * * @return {@code true} if this pool uses async mode - * @see #setAsyncMode */ public boolean getAsyncMode() { - workerCountReadFence(); return locallyFifo; } @@ -1635,7 +1614,10 @@ public class ForkJoinPool extends Abstra */ public long getQueuedTaskCount() { long count = 0; - for (ForkJoinWorkerThread w : workers) { + ForkJoinWorkerThread[] ws = workers; + int n = ws.length; + for (int i = 0; i < n; ++i) { + ForkJoinWorkerThread w = ws[i]; if (w != null) count += w.getQueueSize(); } @@ -1692,12 +1674,15 @@ public class ForkJoinPool extends Abstra * @return the number of elements transferred */ protected int drainTasksTo(Collection> c) { - int n = submissionQueue.drainTo(c); - for (ForkJoinWorkerThread w : workers) { + int count = submissionQueue.drainTo(c); + ForkJoinWorkerThread[] ws = workers; + int n = ws.length; + for (int i = 0; i < n; ++i) { + ForkJoinWorkerThread w = ws[i]; if (w != null) - n += w.drainTasksTo(c); + count += w.drainTasksTo(c); } - return n; + return count; } /** @@ -1821,18 +1806,28 @@ public class ForkJoinPool extends Abstra */ public boolean awaitTermination(long timeout, TimeUnit unit) throws InterruptedException { - return terminationLatch.await(timeout, unit); + try { + return termination.awaitAdvanceInterruptibly(0, timeout, unit) > 0; + } catch(TimeoutException ex) { + return false; + } } /** * Interface for extending managed parallelism for tasks running * in {@link ForkJoinPool}s. * - *

A {@code ManagedBlocker} provides two methods. - * Method {@code isReleasable} must return {@code true} if - * blocking is not necessary. Method {@code block} blocks the - * current thread if necessary (perhaps internally invoking - * {@code isReleasable} before actually blocking). + *

A {@code ManagedBlocker} provides two methods. Method + * {@code isReleasable} must return {@code true} if blocking is + * not necessary. Method {@code block} blocks the current thread + * if necessary (perhaps internally invoking {@code isReleasable} + * before actually blocking). The unusual methods in this API + * accommodate synchronizers that may, but don't usually, block + * for long periods. Similarly, they allow more efficient internal + * handling of cases in which additional workers may be, but + * usually are not, needed to ensure sufficient parallelism. + * Toward this end, implementations of method {@code isReleasable} + * must be amenable to repeated invocation. * *

For example, here is a ManagedBlocker based on a * ReentrantLock: @@ -1850,6 +1845,26 @@ public class ForkJoinPool extends Abstra * return hasLock || (hasLock = lock.tryLock()); * } * }} + * + *

Here is a class that possibly blocks waiting for an + * item on a given queue: + *

 {@code
+     * class QueueTaker implements ManagedBlocker {
+     *   final BlockingQueue queue;
+     *   volatile E item = null;
+     *   QueueTaker(BlockingQueue q) { this.queue = q; }
+     *   public boolean block() throws InterruptedException {
+     *     if (item == null)
+     *       item = queue.take
+     *     return true;
+     *   }
+     *   public boolean isReleasable() {
+     *     return item != null || (item = queue.poll) != null;
+     *   }
+     *   public E getItem() { // call after pool.managedBlock completes
+     *     return item;
+     *   }
+     * }}
*/ public static interface ManagedBlocker { /** @@ -1873,14 +1888,7 @@ public class ForkJoinPool extends Abstra * Blocks in accord with the given blocker. If the current thread * is a {@link ForkJoinWorkerThread}, this method possibly * arranges for a spare thread to be activated if necessary to - * ensure parallelism while the current thread is blocked. - * - *

If {@code maintainParallelism} is {@code true} and the pool - * supports it ({@link #getMaintainsParallelism}), this method - * attempts to maintain the pool's nominal parallelism. Otherwise - * it activates a thread only if necessary to avoid complete - * starvation. This option may be preferable when blockages use - * timeouts, or are almost always brief. + * ensure sufficient parallelism while the current thread is blocked. * *

If the caller is not a {@link ForkJoinTask}, this method is * behaviorally equivalent to @@ -1894,29 +1902,18 @@ public class ForkJoinPool extends Abstra * first be expanded to ensure parallelism, and later adjusted. * * @param blocker the blocker - * @param maintainParallelism if {@code true} and supported by - * this pool, attempt to maintain the pool's nominal parallelism; - * otherwise activate a thread only if necessary to avoid - * complete starvation. * @throws InterruptedException if blocker.block did so */ - public static void managedBlock(ManagedBlocker blocker, - boolean maintainParallelism) + public static void managedBlock(ManagedBlocker blocker) throws InterruptedException { Thread t = Thread.currentThread(); - if (t instanceof ForkJoinWorkerThread) - ((ForkJoinWorkerThread) t).pool. - doBlock(blocker, maintainParallelism); - else - awaitBlocker(blocker); - } - - /** - * Performs Non-FJ blocking - */ - private static void awaitBlocker(ManagedBlocker blocker) - throws InterruptedException { - do {} while (!blocker.isReleasable() && !blocker.block()); + if (t instanceof ForkJoinWorkerThread) { + ForkJoinWorkerThread w = (ForkJoinWorkerThread) t; + w.pool.awaitBlocker(blocker); + } + else { + do {} while (!blocker.isReleasable() && !blocker.block()); + } } // AbstractExecutorService overrides. These rely on undocumented @@ -1944,7 +1941,8 @@ public class ForkJoinPool extends Abstra objectFieldOffset("eventWaiters",ForkJoinPool.class); private static final long stealCountOffset = objectFieldOffset("stealCount",ForkJoinPool.class); - + private static final long spareWaitersOffset = + objectFieldOffset("spareWaiters",ForkJoinPool.class); private static long objectFieldOffset(String field, Class klazz) { try {