--- jsr166/src/jsr166y/ForkJoinTask.java 2009/08/04 12:41:27 1.35
+++ jsr166/src/jsr166y/ForkJoinTask.java 2010/05/27 16:46:48 1.48
@@ -56,9 +56,9 @@ import java.util.WeakHashMap;
* exceptions such as {@code IOExceptions} to be thrown. However,
* computations may still encounter unchecked exceptions, that are
* rethrown to callers attempting to join them. These exceptions may
- * additionally include RejectedExecutionExceptions stemming from
- * internal resource exhaustion such as failure to allocate internal
- * task queues.
+ * additionally include {@link RejectedExecutionException} stemming
+ * from internal resource exhaustion, such as failure to allocate
+ * internal task queues.
*
*
The primary method for awaiting completion and extracting
* results of a task is {@link #join}, but there are several variants:
@@ -80,16 +80,14 @@ import java.util.WeakHashMap;
*
The execution status of tasks may be queried at several levels
* of detail: {@link #isDone} is true if a task completed in any way
* (including the case where a task was cancelled without executing);
- * {@link #isCancelled} is true if completion was due to cancellation;
* {@link #isCompletedNormally} is true if a task completed without
- * cancellation or encountering an exception; {@link
- * #isCompletedExceptionally} is true if if the task encountered an
- * exception (in which case {@link #getException} returns the
- * exception); {@link #isCancelled} is true if the task was cancelled
- * (in which case {@link #getException} returns a {@link
- * java.util.concurrent.CancellationException}; and {@link
- * #isCompletedAbnormally} is true if a task was either cancelled or
- * encountered an exception.
+ * cancellation or encountering an exception; {@link #isCancelled} is
+ * true if the task was cancelled (in which case {@link #getException}
+ * returns a {@link java.util.concurrent.CancellationException}); and
+ * {@link #isCompletedAbnormally} is true if a task was either
+ * cancelled or encountered an exception, in which case {@link
+ * #getException} will return either the encountered exception or
+ * {@link java.util.concurrent.CancellationException}.
*
*
The ForkJoinTask class is not usually directly subclassed.
* Instead, you subclass one of the abstract classes that support a
@@ -125,11 +123,11 @@ import java.util.WeakHashMap;
* improve throughput. If too small, then memory and internal task
* maintenance overhead may overwhelm processing.
*
- *
This class provides {@code adapt} methods for {@link
- * java.lang.Runnable} and {@link java.util.concurrent.Callable}, that
- * may be of use when mixing execution of ForkJoinTasks with other
- * kinds of tasks. When all tasks are of this form, consider using a
- * pool in {@link ForkJoinPool#setAsyncMode async mode}.
+ *
This class provides {@code adapt} methods for {@link Runnable}
+ * and {@link Callable}, that may be of use when mixing execution of
+ * {@code ForkJoinTasks} with other kinds of tasks. When all tasks
+ * are of this form, consider using a pool in
+ * {@linkplain ForkJoinPool#setAsyncMode async mode}.
*
*
ForkJoinTasks are {@code Serializable}, which enables them to be
* used in extensions such as remote execution frameworks. It is
@@ -141,33 +139,45 @@ import java.util.WeakHashMap;
*/
public abstract class ForkJoinTask implements Future, Serializable {
+ /*
+ * See the internal documentation of class ForkJoinPool for a
+ * general implementation overview. ForkJoinTasks are mainly
+ * responsible for maintaining their "status" field amidst relays
+ * to methods in ForkJoinWorkerThread and ForkJoinPool. The
+ * methods of this class are more-or-less layered into (1) basic
+ * status maintenance (2) execution and awaiting completion (3)
+ * user-level methods that additionally report results. This is
+ * sometimes hard to see because this file orders exported methods
+ * in a way that flows well in javadocs.
+ */
+
/**
* Run control status bits packed into a single int to minimize
* footprint and to ensure atomicity (via CAS). Status is
* initially zero, and takes on nonnegative values until
* completed, upon which status holds COMPLETED. CANCELLED, or
* EXCEPTIONAL, which use the top 3 bits. Tasks undergoing
- * blocking waits by other threads have SIGNAL_MASK bits set --
- * bit 15 for external (nonFJ) waits, and the rest a count of
- * waiting FJ threads. (This representation relies on
- * ForkJoinPool max thread limits). Completion of a stolen task
- * with SIGNAL_MASK bits set awakens waiter via notifyAll. Even
- * though suboptimal for some purposes, we use basic builtin
- * wait/notify to take advantage of "monitor inflation" in JVMs
- * that we would otherwise need to emulate to avoid adding further
- * per-task bookkeeping overhead. Note that bits 16-28 are
- * currently unused. Also value 0x80000000 is available as spare
- * completion value.
+ * blocking waits by other threads have the SIGNAL bit set.
+ *
+ * Completion of a stolen task with SIGNAL set awakens any waiters
+ * via notifyAll. Even though suboptimal for some purposes, we use
+ * basic builtin wait/notify to take advantage of "monitor
+ * inflation" in JVMs that we would otherwise need to emulate to
+ * avoid adding further per-task bookkeeping overhead. We want
+ * these monitors to be "fat", i.e., not use biasing or thin-lock
+ * techniques, so use some odd coding idioms that tend to avoid
+ * them.
+ *
+ * Note that bits 1-28 are currently unused. Also value
+ * 0x80000000 is available as spare completion value.
*/
volatile int status; // accessed directly by pool and workers
- static final int COMPLETION_MASK = 0xe0000000;
- static final int NORMAL = 0xe0000000; // == mask
- static final int CANCELLED = 0xc0000000;
- static final int EXCEPTIONAL = 0xa0000000;
- static final int SIGNAL_MASK = 0x0000ffff;
- static final int INTERNAL_SIGNAL_MASK = 0x00007fff;
- static final int EXTERNAL_SIGNAL = 0x00008000; // top bit of low word
+ private static final int COMPLETION_MASK = 0xe0000000;
+ private static final int NORMAL = 0xe0000000; // == mask
+ private static final int CANCELLED = 0xc0000000;
+ private static final int EXCEPTIONAL = 0xa0000000;
+ private static final int SIGNAL = 0x00000001;
/**
* Table of exceptions thrown by tasks, to enable reporting by
@@ -181,328 +191,178 @@ public abstract class ForkJoinTask im
Collections.synchronizedMap
(new WeakHashMap, Throwable>());
- // within-package utilities
+ // Maintaining completion status
/**
- * Gets current worker thread, or null if not a worker thread.
- */
- static ForkJoinWorkerThread getWorker() {
- Thread t = Thread.currentThread();
- return ((t instanceof ForkJoinWorkerThread) ?
- (ForkJoinWorkerThread) t : null);
- }
-
- final boolean casStatus(int cmp, int val) {
- return UNSAFE.compareAndSwapInt(this, statusOffset, cmp, val);
- }
-
- /**
- * Workaround for not being able to rethrow unchecked exceptions.
- */
- static void rethrowException(Throwable ex) {
- if (ex != null)
- UNSAFE.throwException(ex);
- }
-
- // Setting completion status
-
- /**
- * Marks completion and wakes up threads waiting to join this task.
+ * Marks completion and wakes up threads waiting to join this task,
+ * also clearing signal request bits.
*
* @param completion one of NORMAL, CANCELLED, EXCEPTIONAL
+ * @return status on exit
*/
- final void setCompletion(int completion) {
- ForkJoinPool pool = getPool();
- if (pool != null) {
- int s; // Clear signal bits while setting completion status
- do {} while ((s = status) >= 0 && !casStatus(s, completion));
-
- if ((s & SIGNAL_MASK) != 0) {
- if ((s &= INTERNAL_SIGNAL_MASK) != 0)
- pool.updateRunningCount(s);
- synchronized (this) { notifyAll(); }
- }
- }
- else
- externallySetCompletion(completion);
- }
-
- /**
- * Version of setCompletion for non-FJ threads. Leaves signal
- * bits for unblocked threads to adjust, and always notifies.
- */
- private void externallySetCompletion(int completion) {
+ private int setCompletion(int completion) {
int s;
- do {} while ((s = status) >= 0 &&
- !casStatus(s, (s & SIGNAL_MASK) | completion));
- synchronized (this) { notifyAll(); }
- }
-
- /**
- * Sets status to indicate normal completion.
- */
- final void setNormalCompletion() {
- // Try typical fast case -- single CAS, no signal, not already done.
- // Manually expand casStatus to improve chances of inlining it
- if (!UNSAFE.compareAndSwapInt(this, statusOffset, 0, NORMAL))
- setCompletion(NORMAL);
- }
-
- // internal waiting and notification
-
- /**
- * Performs the actual monitor wait for awaitDone.
- */
- private void doAwaitDone() {
- // Minimize lock bias and in/de-flation effects by maximizing
- // chances of waiting inside sync
- try {
- while (status >= 0)
- synchronized (this) { if (status >= 0) wait(); }
- } catch (InterruptedException ie) {
- onInterruptedWait();
- }
- }
-
- /**
- * Performs the actual timed monitor wait for awaitDone.
- */
- private void doAwaitDone(long startTime, long nanos) {
- synchronized (this) {
- try {
- while (status >= 0) {
- long nt = nanos - (System.nanoTime() - startTime);
- if (nt <= 0)
- break;
- wait(nt / 1000000, (int) (nt % 1000000));
- }
- } catch (InterruptedException ie) {
- onInterruptedWait();
+ while ((s = status) >= 0) {
+ if (UNSAFE.compareAndSwapInt(this, statusOffset, s, completion)) {
+ if ((s & SIGNAL) != 0)
+ synchronized (this) { notifyAll(); }
+ return completion;
}
}
+ return s;
}
- // Awaiting completion
-
/**
- * Sets status to indicate there is joiner, then waits for join,
- * surrounded with pool notifications.
- *
- * @return status upon exit
+ * Record exception and set exceptional completion
+ * @return status on exit
*/
- private int awaitDone(ForkJoinWorkerThread w,
- boolean maintainParallelism) {
- ForkJoinPool pool = (w == null) ? null : w.pool;
- int s;
- while ((s = status) >= 0) {
- if (casStatus(s, (pool == null) ? s|EXTERNAL_SIGNAL : s+1)) {
- if (pool == null || !pool.preJoin(this, maintainParallelism))
- doAwaitDone();
- if (((s = status) & INTERNAL_SIGNAL_MASK) != 0)
- adjustPoolCountsOnUnblock(pool);
- break;
- }
- }
- return s;
+ private int setExceptionalCompletion(Throwable rex) {
+ exceptionMap.put(this, rex);
+ return setCompletion(EXCEPTIONAL);
}
/**
- * Timed version of awaitDone
- *
- * @return status upon exit
+ * Blocks a worker thread until completion. Called only by pool.
*/
- private int awaitDone(ForkJoinWorkerThread w, long nanos) {
- ForkJoinPool pool = (w == null) ? null : w.pool;
+ final void internalAwaitDone() {
int s;
while ((s = status) >= 0) {
- if (casStatus(s, (pool == null) ? s|EXTERNAL_SIGNAL : s+1)) {
- long startTime = System.nanoTime();
- if (pool == null || !pool.preJoin(this, false))
- doAwaitDone(startTime, nanos);
- if ((s = status) >= 0) {
- adjustPoolCountsOnCancelledWait(pool);
- s = status;
+ synchronized(this) {
+ if (UNSAFE.compareAndSwapInt(this, statusOffset, s, s|SIGNAL)){
+ do {
+ try {
+ wait();
+ } catch (InterruptedException ie) {
+ cancelIfTerminating();
+ }
+ } while (status >= 0);
+ break;
}
- if (s < 0 && (s & INTERNAL_SIGNAL_MASK) != 0)
- adjustPoolCountsOnUnblock(pool);
- break;
}
}
- return s;
}
/**
- * Notifies pool that thread is unblocked. Called by signalled
- * threads when woken by non-FJ threads (which is atypical).
+ * Blocks a non-worker-thread until completion.
+ * @return status on exit
*/
- private void adjustPoolCountsOnUnblock(ForkJoinPool pool) {
+ private int externalAwaitDone() {
int s;
- do {} while ((s = status) < 0 && !casStatus(s, s & COMPLETION_MASK));
- if (pool != null && (s &= INTERNAL_SIGNAL_MASK) != 0)
- pool.updateRunningCount(s);
- }
-
- /**
- * Notifies pool to adjust counts on cancelled or timed out wait.
- */
- private void adjustPoolCountsOnCancelledWait(ForkJoinPool pool) {
- if (pool != null) {
- int s;
- while ((s = status) >= 0 && (s & INTERNAL_SIGNAL_MASK) != 0) {
- if (casStatus(s, s - 1)) {
- pool.updateRunningCount(1);
+ while ((s = status) >= 0) {
+ synchronized(this) {
+ if (UNSAFE.compareAndSwapInt(this, statusOffset, s, s|SIGNAL)){
+ boolean interrupted = false;
+ do {
+ try {
+ wait();
+ } catch (InterruptedException ie) {
+ interrupted = true;
+ }
+ } while ((s = status) >= 0);
+ if (interrupted)
+ Thread.currentThread().interrupt();
break;
}
}
}
+ return s;
}
/**
- * Handles interruptions during waits.
- */
- private void onInterruptedWait() {
- ForkJoinWorkerThread w = getWorker();
- if (w == null)
- Thread.currentThread().interrupt(); // re-interrupt
- else if (w.isTerminating())
- cancelIgnoringExceptions();
- // else if FJworker, ignore interrupt
- }
-
- // Recording and reporting exceptions
-
- private void setDoneExceptionally(Throwable rex) {
- exceptionMap.put(this, rex);
- setCompletion(EXCEPTIONAL);
- }
-
- /**
- * Throws the exception associated with status s.
- *
- * @throws the exception
- */
- private void reportException(int s) {
- if ((s &= COMPLETION_MASK) < NORMAL) {
- if (s == CANCELLED)
- throw new CancellationException();
- else
- rethrowException(exceptionMap.get(this));
- }
- }
-
- /**
- * Returns result or throws exception using j.u.c.Future conventions.
- * Only call when {@code isDone} known to be true.
- */
- private V reportFutureResult()
- throws ExecutionException, InterruptedException {
- if (Thread.interrupted())
- throw new InterruptedException();
- int s = status & COMPLETION_MASK;
- if (s < NORMAL) {
- Throwable ex;
- if (s == CANCELLED)
- throw new CancellationException();
- if (s == EXCEPTIONAL && (ex = exceptionMap.get(this)) != null)
- throw new ExecutionException(ex);
- }
- return getRawResult();
- }
-
- /**
- * Returns result or throws exception using j.u.c.Future conventions
- * with timeouts.
- */
- private V reportTimedFutureResult()
- throws InterruptedException, ExecutionException, TimeoutException {
- if (Thread.interrupted())
- throw new InterruptedException();
- Throwable ex;
- int s = status & COMPLETION_MASK;
- if (s == NORMAL)
- return getRawResult();
- if (s == CANCELLED)
- throw new CancellationException();
- if (s == EXCEPTIONAL && (ex = exceptionMap.get(this)) != null)
- throw new ExecutionException(ex);
- throw new TimeoutException();
- }
-
- // internal execution methods
-
- /**
- * Calls exec, recording completion, and rethrowing exception if
- * encountered. Caller should normally check status before calling.
- *
- * @return true if completed normally
+ * Unless done, calls exec and records status if completed, but
+ * doesn't wait for completion otherwise.
*/
- private boolean tryExec() {
- try { // try block must contain only call to exec
- if (!exec())
- return false;
+ final void tryExec() {
+ try {
+ if (status < 0 || !exec())
+ return;
} catch (Throwable rex) {
- setDoneExceptionally(rex);
- rethrowException(rex);
- return false; // not reached
+ setExceptionalCompletion(rex);
+ return;
}
- setNormalCompletion();
- return true;
+ setCompletion(NORMAL); // must be outside try block
}
/**
- * Main execution method used by worker threads. Invokes
- * base computation unless already complete.
+ * If not done and this task is next in worker queue, runs it,
+ * else waits for it.
+ * @return status on exit
*/
- final void quietlyExec() {
- if (status >= 0) {
- try {
- if (!exec())
- return;
- } catch (Throwable rex) {
- setDoneExceptionally(rex);
- return;
+ private int waitingJoin() {
+ int s = status;
+ if (s < 0)
+ return s;
+ Thread t = Thread.currentThread();
+ if (t instanceof ForkJoinWorkerThread) {
+ ForkJoinWorkerThread w = (ForkJoinWorkerThread) t;
+ if (w.unpushTask(this)) {
+ boolean completed;
+ try {
+ completed = exec();
+ } catch (Throwable rex) {
+ return setExceptionalCompletion(rex);
+ }
+ if (completed)
+ return setCompletion(NORMAL);
}
- setNormalCompletion();
+ return w.pool.awaitJoin(this);
}
+ else
+ return externalAwaitDone();
}
/**
- * Calls exec(), recording but not rethrowing exception.
- * Caller should normally check status before calling.
- *
- * @return true if completed normally
+ * Unless done, calls exec and records status if completed, or
+ * waits for completion otherwise.
+ * @return status on exit
*/
- private boolean tryQuietlyInvoke() {
+ private int waitingInvoke() {
+ int s = status;
+ if (s < 0)
+ return s;
+ boolean completed;
try {
- if (!exec())
- return false;
+ completed = exec();
} catch (Throwable rex) {
- setDoneExceptionally(rex);
- return false;
+ return setExceptionalCompletion(rex);
}
- setNormalCompletion();
- return true;
+ if (completed)
+ return setCompletion(NORMAL);
+ return waitingJoin();
}
/**
- * Cancels, ignoring any exceptions it throws.
+ * If this task is next in worker queue, runs it, else processes other
+ * tasks until complete.
+ * @return status on exit
*/
- final void cancelIgnoringExceptions() {
- try {
- cancel(false);
- } catch (Throwable ignore) {
+ private int busyJoin() {
+ int s = status;
+ if (s < 0)
+ return s;
+ ForkJoinWorkerThread w = (ForkJoinWorkerThread) Thread.currentThread();
+ if (w.unpushTask(this)) {
+ boolean completed;
+ try {
+ completed = exec();
+ } catch (Throwable rex) {
+ return setExceptionalCompletion(rex);
+ }
+ if (completed)
+ return setCompletion(NORMAL);
}
+ return w.execWhileJoining(this);
}
/**
- * Main implementation of helpJoin
+ * Returns result or throws exception associated with given status.
+ * @param s the status
*/
- private int busyJoin(ForkJoinWorkerThread w) {
- int s;
- ForkJoinTask> t;
- while ((s = status) >= 0 && (t = w.scanWhileJoining(this)) != null)
- t.quietlyExec();
- return (s >= 0) ? awaitDone(w, false) : s; // block if no work
+ private V reportResult(int s) {
+ Throwable ex;
+ if (s < NORMAL && (ex = getException()) != null)
+ UNSAFE.throwException(ex);
+ return getRawResult();
}
// public methods
@@ -511,6 +371,11 @@ public abstract class ForkJoinTask im
* Arranges to asynchronously execute this task. While it is not
* necessarily enforced, it is a usage error to fork a task more
* than once unless it has completed and been reinitialized.
+ * Subsequent modifications to the state of this task or any data
+ * it operates on are not necessarily consistently observable by
+ * any thread other than the one executing it unless preceded by a
+ * call to {@link #join} or related methods, or a call to {@link
+ * #isDone} returning {@code true}.
*
* This method may be invoked only from within {@code
* ForkJoinTask} computations (as may be determined using method
@@ -527,7 +392,7 @@ public abstract class ForkJoinTask im
}
/**
- * Returns the result of the computation when it is ready.
+ * Returns the result of the computation when it {@link #isDone is done}.
* This method differs from {@link #get()} in that
* abnormal completion results in {@code RuntimeException} or
* {@code Error}, not {@code ExecutionException}.
@@ -535,10 +400,7 @@ public abstract class ForkJoinTask im
* @return the computed result
*/
public final V join() {
- ForkJoinWorkerThread w = getWorker();
- if (w == null || status < 0 || !w.unpushTask(this) || !tryExec())
- reportException(awaitDone(w, true));
- return getRawResult();
+ return reportResult(waitingJoin());
}
/**
@@ -549,18 +411,16 @@ public abstract class ForkJoinTask im
* @return the computed result
*/
public final V invoke() {
- if (status >= 0 && tryExec())
- return getRawResult();
- else
- return join();
+ return reportResult(waitingInvoke());
}
/**
* Forks the given tasks, returning when {@code isDone} holds for
* each task or an (unchecked) exception is encountered, in which
- * case the exception is rethrown. If more than one task
- * encounters an exception, then this method throws any one of
- * these exceptions. The individual status of each task may be
+ * case the exception is rethrown. If either task encounters an
+ * exception, the other one may be, but is not guaranteed to be,
+ * cancelled. If both tasks throw an exception, then this method
+ * throws one of them. The individual status of each task may be
* checked using {@link #getException()} and related methods.
*
* This method may be invoked only from within {@code
@@ -609,26 +469,20 @@ public abstract class ForkJoinTask im
}
else if (i != 0)
t.fork();
- else {
- t.quietlyInvoke();
- if (ex == null)
- ex = t.getException();
- }
+ else if (t.waitingInvoke() < NORMAL && ex == null)
+ ex = t.getException();
}
for (int i = 1; i <= last; ++i) {
ForkJoinTask> t = tasks[i];
if (t != null) {
if (ex != null)
t.cancel(false);
- else {
- t.quietlyJoin();
- if (ex == null)
- ex = t.getException();
- }
+ else if (t.waitingJoin() < NORMAL && ex == null)
+ ex = t.getException();
}
}
if (ex != null)
- rethrowException(ex);
+ UNSAFE.throwException(ex);
}
/**
@@ -671,26 +525,20 @@ public abstract class ForkJoinTask im
}
else if (i != 0)
t.fork();
- else {
- t.quietlyInvoke();
- if (ex == null)
- ex = t.getException();
- }
+ else if (t.waitingInvoke() < NORMAL && ex == null)
+ ex = t.getException();
}
for (int i = 1; i <= last; ++i) {
ForkJoinTask> t = ts.get(i);
if (t != null) {
if (ex != null)
t.cancel(false);
- else {
- t.quietlyJoin();
- if (ex == null)
- ex = t.getException();
- }
+ else if (t.waitingJoin() < NORMAL && ex == null)
+ ex = t.getException();
}
}
if (ex != null)
- rethrowException(ex);
+ UNSAFE.throwException(ex);
return tasks;
}
@@ -724,20 +572,34 @@ public abstract class ForkJoinTask im
}
/**
- * Returns {@code true} if the computation performed by this task
- * has completed (or has been cancelled).
- *
- * @return {@code true} if this computation has completed
+ * Cancels, ignoring any exceptions it throws. Used during worker
+ * and pool shutdown.
*/
- public final boolean isDone() {
- return status < 0;
+ final void cancelIgnoringExceptions() {
+ try {
+ cancel(false);
+ } catch (Throwable ignore) {
+ }
}
/**
- * Returns {@code true} if this task was cancelled.
- *
- * @return {@code true} if this task was cancelled
+ * Cancels ignoring exceptions if worker is terminating
*/
+ private void cancelIfTerminating() {
+ Thread t = Thread.currentThread();
+ if ((t instanceof ForkJoinWorkerThread) &&
+ ((ForkJoinWorkerThread) t).isTerminating()) {
+ try {
+ cancel(false);
+ } catch (Throwable ignore) {
+ }
+ }
+ }
+
+ public final boolean isDone() {
+ return status < 0;
+ }
+
public final boolean isCancelled() {
return (status & COMPLETION_MASK) == CANCELLED;
}
@@ -763,15 +625,6 @@ public abstract class ForkJoinTask im
}
/**
- * Returns {@code true} if this task threw an exception.
- *
- * @return {@code true} if this task threw an exception
- */
- public final boolean isCompletedExceptionally() {
- return (status & COMPLETION_MASK) == EXCEPTIONAL;
- }
-
- /**
* Returns the exception thrown by the base computation, or a
* {@code CancellationException} if cancelled, or {@code null} if
* none or if the method has not yet completed.
@@ -780,11 +633,9 @@ public abstract class ForkJoinTask im
*/
public final Throwable getException() {
int s = status & COMPLETION_MASK;
- if (s >= NORMAL)
- return null;
- if (s == CANCELLED)
- return new CancellationException();
- return exceptionMap.get(this);
+ return ((s >= NORMAL) ? null :
+ (s == CANCELLED) ? new CancellationException() :
+ exceptionMap.get(this));
}
/**
@@ -797,14 +648,14 @@ public abstract class ForkJoinTask im
* overridable, but overridden versions must invoke {@code super}
* implementation to maintain guarantees.
*
- * @param ex the exception to throw. If this exception is
- * not a RuntimeException or Error, the actual exception thrown
- * will be a RuntimeException with cause ex.
+ * @param ex the exception to throw. If this exception is not a
+ * {@code RuntimeException} or {@code Error}, the actual exception
+ * thrown will be a {@code RuntimeException} with cause {@code ex}.
*/
public void completeExceptionally(Throwable ex) {
- setDoneExceptionally((ex instanceof RuntimeException) ||
- (ex instanceof Error) ? ex :
- new RuntimeException(ex));
+ setExceptionalCompletion((ex instanceof RuntimeException) ||
+ (ex instanceof Error) ? ex :
+ new RuntimeException(ex));
}
/**
@@ -823,36 +674,109 @@ public abstract class ForkJoinTask im
try {
setRawResult(value);
} catch (Throwable rex) {
- setDoneExceptionally(rex);
+ setExceptionalCompletion(rex);
return;
}
- setNormalCompletion();
+ setCompletion(NORMAL);
}
public final V get() throws InterruptedException, ExecutionException {
- ForkJoinWorkerThread w = getWorker();
- if (w == null || status < 0 || !w.unpushTask(this) || !tryQuietlyInvoke())
- awaitDone(w, true);
- return reportFutureResult();
+ int s = waitingJoin() & COMPLETION_MASK;
+ if (Thread.interrupted())
+ throw new InterruptedException();
+ if (s < NORMAL) {
+ Throwable ex;
+ if (s == CANCELLED)
+ throw new CancellationException();
+ if (s == EXCEPTIONAL && (ex = exceptionMap.get(this)) != null)
+ throw new ExecutionException(ex);
+ }
+ return getRawResult();
}
public final V get(long timeout, TimeUnit unit)
throws InterruptedException, ExecutionException, TimeoutException {
- long nanos = unit.toNanos(timeout);
- ForkJoinWorkerThread w = getWorker();
- if (w == null || status < 0 || !w.unpushTask(this) || !tryQuietlyInvoke())
- awaitDone(w, nanos);
- return reportTimedFutureResult();
+ Thread t = Thread.currentThread();
+ ForkJoinPool pool;
+ if (t instanceof ForkJoinWorkerThread) {
+ ForkJoinWorkerThread w = (ForkJoinWorkerThread) t;
+ if (status >= 0 && w.unpushTask(this))
+ tryExec();
+ pool = w.pool;
+ }
+ else
+ pool = null;
+ /*
+ * Timed wait loop intermixes cases for fj (pool != null) and
+ * non FJ threads. For FJ, decrement pool count but don't try
+ * for replacement; increment count on completion. For non-FJ,
+ * deal with interrupts. This is messy, but a little less so
+ * than is splitting the FJ and nonFJ cases.
+ */
+ boolean interrupted = false;
+ boolean dec = false; // true if pool count decremented
+ for (;;) {
+ if (Thread.interrupted() && pool == null) {
+ interrupted = true;
+ break;
+ }
+ int s = status;
+ if (s < 0)
+ break;
+ if (UNSAFE.compareAndSwapInt(this, statusOffset,
+ s, s | SIGNAL)) {
+ long startTime = System.nanoTime();
+ long nanos = unit.toNanos(timeout);
+ long nt; // wait time
+ while (status >= 0 &&
+ (nt = nanos - (System.nanoTime() - startTime)) > 0) {
+ if (pool != null && !dec)
+ dec = pool.tryDecrementRunningCount();
+ else {
+ long ms = nt / 1000000;
+ int ns = (int) (nt % 1000000);
+ try {
+ synchronized(this) {
+ if (status >= 0)
+ wait(ms, ns);
+ }
+ } catch (InterruptedException ie) {
+ if (pool != null)
+ cancelIfTerminating();
+ else {
+ interrupted = true;
+ break;
+ }
+ }
+ }
+ }
+ break;
+ }
+ }
+ if (pool != null && dec)
+ pool.updateRunningCount(1);
+ if (interrupted)
+ throw new InterruptedException();
+ int es = status & COMPLETION_MASK;
+ if (es != NORMAL) {
+ Throwable ex;
+ if (es == CANCELLED)
+ throw new CancellationException();
+ if (es == EXCEPTIONAL && (ex = exceptionMap.get(this)) != null)
+ throw new ExecutionException(ex);
+ throw new TimeoutException();
+ }
+ return getRawResult();
}
/**
- * Possibly executes other tasks until this task is ready, then
- * returns the result of the computation. This method may be more
- * efficient than {@code join}, but is only applicable when
- * there are no potential dependencies between continuation of the
- * current task and that of any other task that might be executed
- * while helping. (This usually holds for pure divide-and-conquer
- * tasks).
+ * Possibly executes other tasks until this task {@link #isDone is
+ * done}, then returns the result of the computation. This method
+ * may be more efficient than {@code join}, but is only applicable
+ * when there are no potential dependencies between continuation
+ * of the current task and that of any other task that might be
+ * executed while helping. (This usually holds for pure
+ * divide-and-conquer tasks).
*
* This method may be invoked only from within {@code
* ForkJoinTask} computations (as may be determined using method
@@ -863,16 +787,14 @@ public abstract class ForkJoinTask im
* @return the computed result
*/
public final V helpJoin() {
- ForkJoinWorkerThread w = (ForkJoinWorkerThread) Thread.currentThread();
- if (status < 0 || !w.unpushTask(this) || !tryExec())
- reportException(busyJoin(w));
- return getRawResult();
+ return reportResult(busyJoin());
}
/**
- * Possibly executes other tasks until this task is ready. This
- * method may be useful when processing collections of tasks when
- * some have been cancelled or otherwise known to have aborted.
+ * Possibly executes other tasks until this task {@link #isDone is
+ * done}. This method may be useful when processing collections
+ * of tasks when some have been cancelled or otherwise known to
+ * have aborted.
*
* This method may be invoked only from within {@code
* ForkJoinTask} computations (as may be determined using method
@@ -881,12 +803,7 @@ public abstract class ForkJoinTask im
* ClassCastException}.
*/
public final void quietlyHelpJoin() {
- if (status >= 0) {
- ForkJoinWorkerThread w =
- (ForkJoinWorkerThread) Thread.currentThread();
- if (!w.unpushTask(this) || !tryQuietlyInvoke())
- busyJoin(w);
- }
+ busyJoin();
}
/**
@@ -896,11 +813,7 @@ public abstract class ForkJoinTask im
* known to have aborted.
*/
public final void quietlyJoin() {
- if (status >= 0) {
- ForkJoinWorkerThread w = getWorker();
- if (w == null || !w.unpushTask(this) || !tryQuietlyInvoke())
- awaitDone(w, true);
- }
+ waitingJoin();
}
/**
@@ -911,8 +824,7 @@ public abstract class ForkJoinTask im
* known to have aborted.
*/
public final void quietlyInvoke() {
- if (status >= 0 && !tryQuietlyInvoke())
- quietlyJoin();
+ waitingInvoke();
}
/**
@@ -1234,7 +1146,7 @@ public abstract class ForkJoinTask im
private static final long serialVersionUID = -7721805057305804111L;
/**
- * Save the state to a stream.
+ * Saves the state to a stream.
*
* @serialData the current run status and the exception thrown
* during execution, or {@code null} if none
@@ -1247,18 +1159,17 @@ public abstract class ForkJoinTask im
}
/**
- * Reconstitute the instance from a stream.
+ * Reconstitutes the instance from a stream.
*
* @param s the stream
*/
private void readObject(java.io.ObjectInputStream s)
throws java.io.IOException, ClassNotFoundException {
s.defaultReadObject();
- status &= ~INTERNAL_SIGNAL_MASK; // clear internal signal counts
- status |= EXTERNAL_SIGNAL; // conservatively set external signal
+ status |= SIGNAL; // conservatively set external signal
Object ex = s.readObject();
if (ex != null)
- setDoneExceptionally((Throwable) ex);
+ setExceptionalCompletion((Throwable) ex);
}
// Unsafe mechanics