--- jsr166/src/jsr166y/ForkJoinPool.java 2010/07/23 13:07:43 1.58 +++ jsr166/src/jsr166y/ForkJoinPool.java 2010/08/17 18:30:32 1.64 @@ -52,7 +52,7 @@ import java.util.concurrent.CountDownLat * convenient form for informal monitoring. * *
As is the case with other ExecutorServices, there are three - * main task execution methods summarized in the follwoing + * 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 @@ -60,7 +60,7 @@ import java.util.concurrent.CountDownLat * 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. + * within-computation forms listed in the table. * *
{@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 @@ -110,7 +110,7 @@ import java.util.concurrent.CountDownLat * *
This implementation rejects submitted tasks (that is, by throwing
* {@link RejectedExecutionException}) only when the pool is shut down
- * or internal resources have been exhuasted.
+ * or internal resources have been exhausted.
*
* @since 1.7
* @author Doug Lea
@@ -138,43 +138,41 @@ public class ForkJoinPool extends Abstra
* cache pollution effects.)
*
* Beyond work-stealing support and essential bookkeeping, the
- * main responsibility of this framework is to arrange tactics for
- * 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:
+ * 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:
*
- * 1. Arranging for the joiner to execute some task that it
+ * 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.
*
- * 2. Unless there are already enough live threads, creating or
- * or re-activating a spare thread to compensate for the
- * (blocked) joiner until it unblocks. Spares then suspend
- * at their next opportunity or eventually die if unused for
- * too long. See below and the internal documentation
- * for tryAwaitJoin for more details about compensation
- * rules.
+ * 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, joins (in
- * ForkJoinWorkerThread.joinTask) interleave these options until
- * successful. Creating a new spare always succeeds, but also
- * increases application footprint, so we try to avoid it, within
- * reason.
+ * 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 option (1) so uses a
- * special version of (2) in method awaitBlocker.
+ * 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
@@ -207,6 +205,19 @@ public class ForkJoinPool extends Abstra
* 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
* aim to approximately maintain the given level of parallelism.
* When some workers are known to be blocked (on joins or via
@@ -248,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
@@ -259,18 +270,13 @@ 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.
*
@@ -285,44 +291,49 @@ public class ForkJoinPool extends Abstra
* 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 tryAwaitJoin and awaitBlocker
- * look 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 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 awaitJoin and awaitBlocker. We always need to create
- * one when the number of running threads would become zero and
- * all workers are busy. However, this is not easy to detect
- * reliably in the presence of transients so we use retries and
- * allow slack (in tryAwaitJoin) to reduce false alarms. These
- * effectively reduce 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 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.
+ * so, suspends itself. Method helpMaintainParallelism looks for
+ * suspended threads to resume before considering creating a new
+ * 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.
+ *
+ * 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.
*
* Beware that there is a lot of representation-level coupling
* among classes ForkJoinPool, ForkJoinWorkerThread, and
@@ -335,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 other
- * coding oddities that help some methods perform reasonably even
- * when interpreted (not compiled), at the expense of messiness.
+ * 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)
@@ -407,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 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_THREADS = 0x7fff;
+ private static final int MAX_WORKERS = 0x7fff;
/**
* Array holding all worker threads in the pool. Array size must
@@ -452,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_ID_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
@@ -479,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;
@@ -488,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)
@@ -497,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;
@@ -530,16 +567,17 @@ public class ForkJoinPool extends Abstra
*/
private final int poolNumber;
- // Utilities for CASing fields. Note that several of these
- // are manually inlined by callers
+
+ // Utilities for CASing fields. Note that most of these
+ // are usually manually inlined by callers
/**
- * Increments running count. Also used by ForkJoinTask.
+ * Increments running count part of workerCounts
*/
final void incrementRunningCount() {
int c;
do {} while (!UNSAFE.compareAndSwapInt(this, workerCountsOffset,
- c = workerCounts,
+ c = workerCounts,
c + ONE_RUNNING));
}
@@ -555,12 +593,34 @@ public class ForkJoinPool extends Abstra
}
/**
- * Tries to increment running count
+ * 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
*/
- final boolean tryIncrementRunningCount() {
- int wc;
- return UNSAFE.compareAndSwapInt(this, workerCountsOffset,
- wc = workerCounts, wc + ONE_RUNNING);
+ 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));
}
/**
@@ -572,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);
}
/**
@@ -582,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);
}
/**
@@ -611,12 +671,12 @@ public class ForkJoinPool extends Abstra
lock.lock();
try {
ForkJoinWorkerThread[] ws = workers;
- int nws = ws.length;
- if (k < 0 || k >= nws || ws[k] != null) {
- for (k = 0; k < nws && 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 == nws)
- ws = Arrays.copyOf(ws, nws << 1);
+ if (k == n)
+ ws = Arrays.copyOf(ws, n << 1);
}
ws[k] = w;
workers = ws; // volatile array write ensures slot visibility
@@ -650,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;
@@ -658,447 +718,439 @@ 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), ueh);
+ if (w != null) {
+ w.start(recordWorker(w), ueh);
+ advanceEventCount();
+ }
return w;
}
/**
- * Adjusts counts upon failure to create worker
+ * Final callback from terminating worker. Removes record of
+ * worker from array, and adjusts counts. If pool is shutting
+ * down, tries to complete terminatation.
+ *
+ * @param w the worker
*/
- private void onWorkerCreationFailure() {
- for (;;) {
- int wc = workerCounts;
- if ((wc >>> TOTAL_COUNT_SHIFT) == 0)
- Thread.yield(); // wait for other counts to settle
- else if (UNSAFE.compareAndSwapInt(this, workerCountsOffset, wc,
- wc - (ONE_RUNNING|ONE_TOTAL)))
+ final void workerTerminated(ForkJoinWorkerThread w) {
+ forgetWorker(w);
+ decrementWorkerCounts(w.isTrimmed()? 0 : ONE_RUNNING, ONE_TOTAL);
+ while (w.stealCount != 0) // collect final count
+ tryAccumulateStealCount(w);
+ tryTerminate(false);
+ }
+
+ // Waiting for and signalling events
+
+ /**
+ * 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;
}
- tryTerminate(false); // in case of failure during shutdown
}
/**
- * Creates and/or resumes enough workers to establish target
- * parallelism, giving up if terminating or addWorker fails
+ * Tries to advance eventCount and releases waiters. Called only
+ * from workers.
+ */
+ 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();
+ }
+
+ /**
+ * Adds the given worker to event queue and blocks until
+ * terminating or event count advances from the workers
+ * lastEventCount value
*
- * TODO: recast this to support lazier creation and automated
- * parallelism maintenance
+ * @param w the calling worker thread
*/
- private void ensureEnoughWorkers() {
- for (;;) {
- int pc = parallelism;
- int wc = workerCounts;
- int rc = wc & RUNNING_COUNT_MASK;
- int tc = wc >>> TOTAL_COUNT_SHIFT;
- if (tc < pc) {
- if (runState == TERMINATING ||
- (UNSAFE.compareAndSwapInt
- (this, workerCountsOffset,
- wc, wc + (ONE_RUNNING|ONE_TOTAL)) &&
- addWorker() == null))
- break;
- }
- else if (tc > pc && rc < pc &&
- tc > (runState & ACTIVE_COUNT_MASK)) {
- ForkJoinWorkerThread spare = null;
- ForkJoinWorkerThread[] ws = workers;
- int nws = ws.length;
- for (int i = 0; i < nws; ++i) {
- ForkJoinWorkerThread w = ws[i];
- if (w != null && w.isSuspended()) {
- if ((workerCounts & RUNNING_COUNT_MASK) > pc ||
- runState == TERMINATING)
- return;
- if (w.tryResumeSpare())
- incrementRunningCount();
- break;
- }
- }
- }
- else
+ 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;
+ }
}
}
/**
- * 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.
+ * 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 worker
+ * @param w the calling worker thread
+ * @param ec the count
*/
- final void workerTerminated(ForkJoinWorkerThread w) {
- if (w.active) { // force inactive
- w.active = false;
- do {} while (!tryDecrementActiveCount());
+ 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);
+ }
+ }
}
- forgetWorker(w);
+ }
- // Decrement total count, and if was running, running count
- // Spin (waiting for other updates) if either would be negative
- int nr = w.isTrimmed() ? 0 : ONE_RUNNING;
- int unit = ONE_TOTAL + nr;
- for (;;) {
- int wc = workerCounts;
- int rc = wc & RUNNING_COUNT_MASK;
- if (rc - nr < 0 || (wc >>> TOTAL_COUNT_SHIFT) == 0)
- Thread.yield(); // back off if waiting for other updates
- else if (UNSAFE.compareAndSwapInt(this, workerCountsOffset,
- wc, wc - unit))
- break;
+ /**
+ * 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;
}
-
- accumulateStealCount(w); // collect final count
- if (!tryTerminate(false))
- ensureEnoughWorkers();
+ 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();
}
- // Waiting for and signalling events
+ // Maintaining spares
/**
- * Releases workers blocked on a count not equal to current count.
- * @return true if any released
+ * Pushes worker onto the spare stack
*/
- private void releaseWaiters() {
- long top;
- while ((top = eventWaiters) != 0L) {
- ForkJoinWorkerThread[] ws = workers;
- int n = ws.length;
- for (;;) {
- int i = ((int)(top & WAITER_ID_MASK)) - 1;
- if (i < 0 || (int)(top >>> EVENT_COUNT_SHIFT) == eventCount)
- return;
- ForkJoinWorkerThread w;
- if (i < n && (w = ws[i]) != null &&
- UNSAFE.compareAndSwapLong(this, eventWaitersOffset,
- top, w.nextWaiter)) {
- LockSupport.unpark(w);
- top = eventWaiters;
- }
- else
- break; // possibly stale; reread
- }
- }
+ 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));
}
/**
- * Ensures eventCount on exit is different (mod 2^32) than on
- * entry and wakes up all waiters
+ * Callback from oldest spare occasionally waking up. Tries
+ * (once) to shutdown a spare. Same idea as tryShutdownWaiter.
*/
- private void signalEvent() {
- int c;
- do {} while (!UNSAFE.compareAndSwapInt(this, eventCountOffset,
- c = eventCount, c+1));
- releaseWaiters();
+ 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();
+ }
}
/**
- * Advances eventCount and releases waiters until interference by
- * other releasing threads is detected.
+ * Tries (once) to resume a spare if worker counts match
+ * the given count.
+ *
+ * @param wc workerCounts value on invocation of this method
*/
- final void signalWork() {
- int c;
- UNSAFE.compareAndSwapInt(this, eventCountOffset, c=eventCount, c+1);
- long top;
- while ((top = eventWaiters) != 0L) {
- int ec = eventCount;
- ForkJoinWorkerThread[] ws = workers;
- int n = ws.length;
- for (;;) {
- int i = ((int)(top & WAITER_ID_MASK)) - 1;
- if (i < 0 || (int)(top >>> EVENT_COUNT_SHIFT) == ec)
- return;
- ForkJoinWorkerThread w;
- if (i < n && (w = ws[i]) != null &&
- UNSAFE.compareAndSwapLong(this, eventWaitersOffset,
- top, top = w.nextWaiter)) {
+ 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);
- if (top != eventWaiters) // let someone else take over
- return;
- }
- else
- break; // possibly stale; reread
+ else // back out if w was shutdown
+ decrementWorkerCounts(ONE_RUNNING, 0);
}
}
}
+ // adding workers on demand
+
/**
- * If worker is inactive, blocks until terminating or event count
- * advances from last value held by worker; in any case helps
- * release others.
- *
- * @param w the calling worker thread
- * @param retries the number of scans by caller failing to find work
- * @return false if now too many threads running
+ * Adds one or more workers if needed to establish target parallelism.
+ * Retries upon contention.
*/
- private boolean eventSync(ForkJoinWorkerThread w, int retries) {
- int wec = w.lastEventCount;
- if (retries > 1) { // can only block after 2nd miss
- long nextTop = (((long)wec << EVENT_COUNT_SHIFT) |
- ((long)(w.poolIndex + 1)));
- long top;
- while ((runState < SHUTDOWN || !tryTerminate(false)) &&
- (((int)(top = eventWaiters) & WAITER_ID_MASK) == 0 ||
- (int)(top >>> EVENT_COUNT_SHIFT) == wec) &&
- eventCount == wec) {
- if (UNSAFE.compareAndSwapLong(this, eventWaitersOffset,
- w.nextWaiter = top, nextTop)) {
- accumulateStealCount(w); // transfer steals while idle
- Thread.interrupted(); // clear/ignore interrupt
- while (eventCount == wec)
- w.doPark();
+ 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;
- }
}
- wec = eventCount;
}
- releaseWaiters();
- int wc = workerCounts;
- if ((wc & RUNNING_COUNT_MASK) <= parallelism) {
- w.lastEventCount = wec;
- return true;
+ }
+
+ /**
+ * 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();
+ }
+ }
+
+ /**
+ * 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);
}
- if (wec != w.lastEventCount) // back up if may re-wait
- w.lastEventCount = wec - (wc >>> TOTAL_COUNT_SHIFT);
- return false;
}
/**
* 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:
+ * stealing a task or taking a submission and running it).
+ * Performs one or more 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.
+ * 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 retries the number of scans by caller failing to find work
- * find any (in which case it may block waiting for work).
+ * @param misses the number of scans by caller failing to find work
+ * (saturating at 2 to avoid wraparound)
*/
- final void preStep(ForkJoinWorkerThread w, int retries) {
+ final void preStep(ForkJoinWorkerThread w, int misses) {
boolean active = w.active;
- boolean inactivate = active && retries != 0;
+ int pc = parallelism;
for (;;) {
- int rs, wc;
- if (inactivate &&
- UNSAFE.compareAndSwapInt(this, runStateOffset,
- rs = runState, rs - ONE_ACTIVE))
- inactivate = active = w.active = false;
- if (((wc = workerCounts) & RUNNING_COUNT_MASK) <= parallelism) {
- if (active || eventSync(w, retries))
- break;
+ 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
}
- else if (!(inactivate |= active) && // must inactivate to suspend
- UNSAFE.compareAndSwapInt(this, workerCountsOffset,
- wc, wc - ONE_RUNNING) &&
- !w.suspendAsSpare()) // false if trimmed
- break;
+ 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
+ }
+ continue;
+ }
+ 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;
+ }
+ if (rc < pc)
+ helpMaintainParallelism();
+ break;
}
}
/**
- * Awaits join of the given task if enough threads, or can resume
- * or create a spare. Fails (in which case the given task might
- * not be done) upon contention or lack of decision about
- * blocking. Returns void because caller must check
- * task status on return anyway.
- *
- * We allow blocking if:
- *
- * 1. There would still be at least as many running threads as
- * parallelism level if this thread blocks.
- *
- * 2. A spare is resumed to replace this worker. We tolerate
- * slop in the decision to replace if a spare is found without
- * first decrementing run count. This may release too many,
- * but if so, the superfluous ones will re-suspend via
- * preStep().
- *
- * 3. After #spares repeated checks, there are no fewer than #spare
- * threads not running. We allow this slack to avoid hysteresis
- * and as a hedge against lag/uncertainty of running count
- * estimates when signalling or unblocking stalls.
- *
- * 4. All existing workers are busy (as rechecked via repeated
- * retries by caller) and a new spare is created.
- *
- * If none of the above hold, we try to escape out by
- * re-incrementing count and returning to caller, which can retry
- * later.
+ * 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.
*
* @param joinMe the task to join
- * @param retries if negative, then serve only as a precheck
- * that the thread can be replaced by a spare. Otherwise,
- * the number of repeated calls to this method returning busy
- * @return true if the call must be retried because there
- * none of the blocking checks hold
*/
- final boolean tryAwaitJoin(ForkJoinTask> joinMe, int retries) {
- if (joinMe.status < 0) // precheck to prime loop
- return false;
- int pc = parallelism;
- boolean running = true; // false when running count decremented
- outer:for (;;) {
- int wc = workerCounts;
- int rc = wc & RUNNING_COUNT_MASK;
- int tc = wc >>> TOTAL_COUNT_SHIFT;
- if (running) { // replace with spare or decrement count
- if (rc <= pc && tc > pc &&
- (retries > 0 || tc > (runState & ACTIVE_COUNT_MASK))) {
- ForkJoinWorkerThread[] ws = workers;
- int nws = ws.length;
- for (int i = 0; i < nws; ++i) { // search for spare
- ForkJoinWorkerThread w = ws[i];
- if (w != null) {
- if (joinMe.status < 0)
- return false;
- if (w.isSuspended()) {
- if ((workerCounts & RUNNING_COUNT_MASK)>=pc &&
- w.tryResumeSpare()) {
- running = false;
- break outer;
- }
- continue outer; // rescan
- }
- }
- }
- }
- if (retries < 0 || // < 0 means replacement check only
- rc == 0 || joinMe.status < 0 || workerCounts != wc ||
- !UNSAFE.compareAndSwapInt(this, workerCountsOffset,
- wc, wc - ONE_RUNNING))
- return false; // done or inconsistent or contended
- running = false;
- if (rc > pc)
- break;
+ final void awaitJoin(ForkJoinTask> joinMe, ForkJoinWorkerThread worker) {
+ int threshold = parallelism; // descend blocking thresholds
+ while (joinMe.status >= 0) {
+ boolean block; int wc;
+ worker.helpJoinTask(joinMe);
+ if (joinMe.status < 0)
+ break;
+ if (((wc = workerCounts) & RUNNING_COUNT_MASK) <= threshold) {
+ if (threshold > 0)
+ --threshold;
+ else
+ advanceEventCount(); // force release
+ block = false;
}
- else { // allow blocking if enough threads
- if (rc >= pc || joinMe.status < 0)
- break;
- int sc = tc - pc + 1; // = spare threads, plus the one to add
- if (retries > sc) {
- if (rc > 0 && rc >= pc - sc) // allow slack
- break;
- if (tc < MAX_THREADS &&
- tc == (runState & ACTIVE_COUNT_MASK) &&
- workerCounts == wc &&
- UNSAFE.compareAndSwapInt(this, workerCountsOffset, wc,
- wc+(ONE_RUNNING|ONE_TOTAL))) {
- addWorker();
- break;
- }
- }
- if (workerCounts == wc && // back out to allow rescan
- UNSAFE.compareAndSwapInt (this, workerCountsOffset,
- wc, wc + ONE_RUNNING)) {
- releaseWaiters(); // help others progress
- return true; // let caller retry
- }
+ 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;
}
}
- // arrive here if can block
- joinMe.internalAwaitDone();
- int c; // to inline incrementRunningCount
- do {} while (!UNSAFE.compareAndSwapInt
- (this, workerCountsOffset,
- c = workerCounts, c + ONE_RUNNING));
- return false;
}
/**
- * Same idea as (and shares many code snippets with) tryAwaitJoin,
- * but self-contained because there are no caller retries.
- * TODO: Rework to use simpler API.
+ * Same idea as awaitJoin, but no helping
*/
final void awaitBlocker(ManagedBlocker blocker)
throws InterruptedException {
- boolean done;
- if (done = blocker.isReleasable())
- return;
- int pc = parallelism;
- int retries = 0;
- boolean running = true; // false when running count decremented
- outer:for (;;) {
- int wc = workerCounts;
- int rc = wc & RUNNING_COUNT_MASK;
- int tc = wc >>> TOTAL_COUNT_SHIFT;
- if (running) {
- if (rc <= pc && tc > pc &&
- (retries > 0 || tc > (runState & ACTIVE_COUNT_MASK))) {
- ForkJoinWorkerThread[] ws = workers;
- int nws = ws.length;
- for (int i = 0; i < nws; ++i) {
- ForkJoinWorkerThread w = ws[i];
- if (w != null) {
- if (done = blocker.isReleasable())
- return;
- if (w.isSuspended()) {
- if ((workerCounts & RUNNING_COUNT_MASK)>=pc &&
- w.tryResumeSpare()) {
- running = false;
- break outer;
- }
- continue outer; // rescan
- }
- }
- }
- }
- if (done = blocker.isReleasable())
- return;
- if (rc == 0 || workerCounts != wc ||
- !UNSAFE.compareAndSwapInt(this, workerCountsOffset,
- wc, wc - ONE_RUNNING))
- continue;
- running = false;
- if (rc > pc)
- break;
+ 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 {
- if (rc >= pc || (done = blocker.isReleasable()))
- break;
- int sc = tc - pc + 1;
- if (retries++ > sc) {
- if (rc > 0 && rc >= pc - sc)
- break;
- if (tc < MAX_THREADS &&
- tc == (runState & ACTIVE_COUNT_MASK) &&
- workerCounts == wc &&
- UNSAFE.compareAndSwapInt(this, workerCountsOffset, wc,
- wc+(ONE_RUNNING|ONE_TOTAL))) {
- addWorker();
- break;
- }
+ 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));
}
- Thread.yield();
- }
- }
-
- try {
- if (!done)
- do {} while (!blocker.isReleasable() && !blocker.block());
- } finally {
- if (!running) {
- int c;
- do {} while (!UNSAFE.compareAndSwapInt
- (this, workerCountsOffset,
- c = workerCounts, c + ONE_RUNNING));
+ break;
}
}
- }
+ }
/**
* Possibly initiates and/or completes termination.
@@ -1128,14 +1180,37 @@ public class ForkJoinPool extends Abstra
/**
* 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() {
- for (int i = 0; i < 2; ++i) { // twice to mop up newly created workers
- cancelSubmissions();
- shutdownWorkers();
- cancelWorkerTasks();
- signalEvent();
- interruptWorkers();
+ 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) {
+ }
+ }
+ }
+ }
+ }
}
}
@@ -1152,50 +1227,6 @@ public class ForkJoinPool extends Abstra
}
}
- /**
- * Sets all worker run states to at least shutdown,
- * also resuming suspended workers
- */
- private void shutdownWorkers() {
- ForkJoinWorkerThread[] ws = workers;
- int nws = ws.length;
- for (int i = 0; i < nws; ++i) {
- ForkJoinWorkerThread w = ws[i];
- if (w != null)
- w.shutdown();
- }
- }
-
- /**
- * Clears out and cancels all locally queued tasks
- */
- private void cancelWorkerTasks() {
- ForkJoinWorkerThread[] ws = workers;
- int nws = ws.length;
- for (int i = 0; i < nws; ++i) {
- ForkJoinWorkerThread w = ws[i];
- if (w != null)
- w.cancelTasks();
- }
- }
-
- /**
- * Unsticks all workers blocked on joins etc
- */
- private void interruptWorkers() {
- ForkJoinWorkerThread[] ws = workers;
- int nws = ws.length;
- for (int i = 0; i < nws; ++i) {
- ForkJoinWorkerThread w = ws[i];
- if (w != null && !w.isTerminated()) {
- try {
- w.interrupt();
- } catch (SecurityException ignore) {
- }
- }
- }
- }
-
// misc support for ForkJoinWorkerThread
/**
@@ -1206,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;
}
/**
@@ -1275,10 +1310,10 @@ public class ForkJoinPool extends Abstra
* 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
+ * @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,
+ * @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
@@ -1292,14 +1327,14 @@ public class ForkJoinPool extends Abstra
* because it does not hold {@link
* java.lang.RuntimePermission}{@code ("modifyThread")}
*/
- public ForkJoinPool(int parallelism,
+ 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.parallelism = parallelism;
this.factory = factory;
@@ -1318,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;
@@ -1338,15 +1373,15 @@ public class ForkJoinPool extends Abstra
if (runState >= SHUTDOWN)
throw new RejectedExecutionException();
submissionQueue.offer(task);
- signalEvent();
- ensureEnoughWorkers();
+ advanceEventCount();
+ if (eventWaiters != 0L)
+ releaseEventWaiters();
+ if ((workerCounts >>> TOTAL_COUNT_SHIFT) < parallelism)
+ addWorkerIfBelowTarget();
}
/**
* Performs the given task, returning its result upon completion.
- * If the caller is already engaged in a fork/join computation in
- * the current pool, this method is equivalent in effect to
- * {@link ForkJoinTask#invoke}.
*
* @param task the task
* @return the task's result
@@ -1361,9 +1396,6 @@ public class ForkJoinPool extends Abstra
/**
* Arranges for (asynchronous) execution of the given task.
- * If the caller is already engaged in a fork/join computation in
- * the current pool, this method is equivalent in effect to
- * {@link ForkJoinTask#fork}.
*
* @param task the task
* @throws NullPointerException if the task is null
@@ -1392,9 +1424,6 @@ public class ForkJoinPool extends Abstra
/**
* Submits a ForkJoinTask for execution.
- * If the caller is already engaged in a fork/join computation in
- * the current pool, this method is equivalent in effect to
- * {@link ForkJoinTask#fork}.
*
* @param task the task to submit
* @return the task
@@ -1586,8 +1615,8 @@ public class ForkJoinPool extends Abstra
public long getQueuedTaskCount() {
long count = 0;
ForkJoinWorkerThread[] ws = workers;
- int nws = ws.length;
- for (int i = 0; i < nws; ++i) {
+ int n = ws.length;
+ for (int i = 0; i < n; ++i) {
ForkJoinWorkerThread w = ws[i];
if (w != null)
count += w.getQueueSize();
@@ -1645,29 +1674,13 @@ public class ForkJoinPool extends Abstra
* @return the number of elements transferred
*/
protected int drainTasksTo(Collection super ForkJoinTask>> c) {
- int n = submissionQueue.drainTo(c);
- ForkJoinWorkerThread[] ws = workers;
- int nws = ws.length;
- for (int i = 0; i < nws; ++i) {
- ForkJoinWorkerThread w = ws[i];
- if (w != null)
- n += w.drainTasksTo(c);
- }
- return n;
- }
-
- /**
- * Returns count of total parks by existing workers.
- * Used during development only since not meaningful to users.
- */
- private int collectParkCount() {
- int count = 0;
+ int count = submissionQueue.drainTo(c);
ForkJoinWorkerThread[] ws = workers;
- int nws = ws.length;
- for (int i = 0; i < nws; ++i) {
+ int n = ws.length;
+ for (int i = 0; i < n; ++i) {
ForkJoinWorkerThread w = ws[i];
if (w != null)
- count += w.parkCount;
+ count += w.drainTasksTo(c);
}
return count;
}
@@ -1689,7 +1702,6 @@ public class ForkJoinPool extends Abstra
int pc = parallelism;
int rs = runState;
int ac = rs & ACTIVE_COUNT_MASK;
- // int pk = collectParkCount();
return super.toString() +
"[" + runLevelToString(rs) +
", parallelism = " + pc +
@@ -1699,7 +1711,6 @@ public class ForkJoinPool extends Abstra
", steals = " + st +
", tasks = " + qt +
", submissions = " + qs +
- // ", parks = " + pk +
"]";
}
@@ -1806,11 +1817,17 @@ public class ForkJoinPool extends Abstra
* 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: @@ -1828,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*/ public static interface ManagedBlocker { /** @@ -1870,8 +1907,10 @@ public class ForkJoinPool extends Abstra public static void managedBlock(ManagedBlocker blocker) throws InterruptedException { Thread t = Thread.currentThread(); - if (t instanceof ForkJoinWorkerThread) - ((ForkJoinWorkerThread) t).pool.awaitBlocker(blocker); + if (t instanceof ForkJoinWorkerThread) { + ForkJoinWorkerThread w = (ForkJoinWorkerThread) t; + w.pool.awaitBlocker(blocker); + } else { do {} while (!blocker.isReleasable() && !blocker.block()); } @@ -1902,6 +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 {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; + * } + * }}