--- jsr166/src/jsr166y/ForkJoinTask.java 2009/07/20 22:26:03 1.9 +++ jsr166/src/jsr166y/ForkJoinTask.java 2009/07/24 22:05:22 1.15 @@ -81,8 +81,10 @@ import java.lang.reflect.*; * class. While these methods have {@code public} access (to allow * instances of different task subclasses to call each others * methods), some of them may only be called from within other - * ForkJoinTasks. Attempts to invoke them in other contexts result in - * exceptions or errors possibly including ClassCastException. + * ForkJoinTasks (as may be determined using method {@link + * #inForkJoinPool}). Attempts to invoke them in other contexts + * result in exceptions or errors, possibly including + * ClassCastException. * *

Most base support methods are {@code final} because their * implementations are intrinsically tied to the underlying @@ -107,6 +109,9 @@ import java.lang.reflect.*; * in general sensible to serialize tasks only before or after, but * not during execution. Serialization is not relied on during * execution itself. + * + * @since 1.7 + * @author Doug Lea */ public abstract class ForkJoinTask implements Future, Serializable { @@ -141,10 +146,10 @@ public abstract class ForkJoinTask im /** * Table of exceptions thrown by tasks, to enable reporting by * callers. Because exceptions are rare, we don't directly keep - * them with task objects, but instead us a weak ref table. Note + * them with task objects, but instead use a weak ref table. Note * that cancellation exceptions don't appear in the table, but are * instead recorded as status values. - * Todo: Use ConcurrentReferenceHashMap + * TODO: Use ConcurrentReferenceHashMap */ static final Map, Throwable> exceptionMap = Collections.synchronizedMap @@ -153,16 +158,16 @@ public abstract class ForkJoinTask im // within-package utilities /** - * Get current worker thread, or null if not a worker thread + * 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); + return ((t instanceof ForkJoinWorkerThread) ? + (ForkJoinWorkerThread) t : null); } final boolean casStatus(int cmp, int val) { - return _unsafe.compareAndSwapInt(this, statusOffset, cmp, val); + return UNSAFE.compareAndSwapInt(this, statusOffset, cmp, val); } /** @@ -170,25 +175,26 @@ public abstract class ForkJoinTask im */ static void rethrowException(Throwable ex) { if (ex != null) - _unsafe.throwException(ex); + UNSAFE.throwException(ex); } // Setting completion status /** - * Mark completion and wake up threads waiting to join this task. + * Marks completion and wakes up threads waiting to join this task. + * * @param completion one of NORMAL, CANCELLED, EXCEPTIONAL */ 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)); + 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(); } + synchronized (this) { notifyAll(); } } } else @@ -201,48 +207,48 @@ public abstract class ForkJoinTask im */ private void externallySetCompletion(int completion) { int s; - do;while ((s = status) >= 0 && - !casStatus(s, (s & SIGNAL_MASK) | completion)); - synchronized(this) { notifyAll(); } + do {} while ((s = status) >= 0 && + !casStatus(s, (s & SIGNAL_MASK) | completion)); + synchronized (this) { notifyAll(); } } /** - * Sets status to indicate normal completion + * 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)) + if (!UNSAFE.compareAndSwapInt(this, statusOffset, 0, NORMAL)) setCompletion(NORMAL); } // internal waiting and notification /** - * Performs the actual monitor wait for awaitDone + * 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(); } + synchronized (this) { if (status >= 0) wait(); } } catch (InterruptedException ie) { onInterruptedWait(); } } /** - * Performs the actual monitor wait for awaitDone + * Performs the actual timed monitor wait for awaitDone. */ private void doAwaitDone(long startTime, long nanos) { - synchronized(this) { + synchronized (this) { try { while (status >= 0) { long nt = nanos - System.nanoTime() - startTime; if (nt <= 0) break; - wait(nt / 1000000, (int)(nt % 1000000)); + wait(nt / 1000000, (int) (nt % 1000000)); } } catch (InterruptedException ie) { onInterruptedWait(); @@ -255,13 +261,15 @@ public abstract class ForkJoinTask im /** * Sets status to indicate there is joiner, then waits for join, * surrounded with pool notifications. + * * @return status upon exit */ - private int awaitDone(ForkJoinWorkerThread w, boolean maintainParallelism) { - ForkJoinPool pool = w == null? null : w.pool; + 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 (casStatus(s, (pool == null) ? s|EXTERNAL_SIGNAL : s+1)) { if (pool == null || !pool.preJoin(this, maintainParallelism)) doAwaitDone(); if (((s = status) & INTERNAL_SIGNAL_MASK) != 0) @@ -274,13 +282,14 @@ public abstract class ForkJoinTask im /** * Timed version of awaitDone + * * @return status upon exit */ private int awaitDone(ForkJoinWorkerThread w, long nanos) { - ForkJoinPool pool = w == null? null : w.pool; + ForkJoinPool pool = (w == null) ? null : w.pool; int s; while ((s = status) >= 0) { - if (casStatus(s, pool == null? s|EXTERNAL_SIGNAL : s+1)) { + if (casStatus(s, (pool == null) ? s|EXTERNAL_SIGNAL : s+1)) { long startTime = System.nanoTime(); if (pool == null || !pool.preJoin(this, false)) doAwaitDone(startTime, nanos); @@ -297,18 +306,18 @@ public abstract class ForkJoinTask im } /** - * Notify pool that thread is unblocked. Called by signalled + * Notifies pool that thread is unblocked. Called by signalled * threads when woken by non-FJ threads (which is atypical). */ private void adjustPoolCountsOnUnblock(ForkJoinPool pool) { int s; - do;while ((s = status) < 0 && !casStatus(s, s & COMPLETION_MASK)); + do {} while ((s = status) < 0 && !casStatus(s, s & COMPLETION_MASK)); if (pool != null && (s &= INTERNAL_SIGNAL_MASK) != 0) pool.updateRunningCount(s); } /** - * Notify pool to adjust counts on cancelled or timed out wait + * Notifies pool to adjust counts on cancelled or timed out wait. */ private void adjustPoolCountsOnCancelledWait(ForkJoinPool pool) { if (pool != null) { @@ -323,7 +332,7 @@ public abstract class ForkJoinTask im } /** - * Handle interruptions during waits. + * Handles interruptions during waits. */ private void onInterruptedWait() { ForkJoinWorkerThread w = getWorker(); @@ -342,7 +351,8 @@ public abstract class ForkJoinTask im } /** - * Throws the exception associated with status s; + * Throws the exception associated with status s. + * * @throws the exception */ private void reportException(int s) { @@ -355,8 +365,8 @@ public abstract class ForkJoinTask im } /** - * Returns result or throws exception using j.u.c.Future conventions - * Only call when isDone known to be true. + * 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 { @@ -375,7 +385,7 @@ public abstract class ForkJoinTask im /** * Returns result or throws exception using j.u.c.Future conventions - * with timeouts + * with timeouts. */ private V reportTimedFutureResult() throws InterruptedException, ExecutionException, TimeoutException { @@ -396,7 +406,8 @@ public abstract class ForkJoinTask im /** * Calls exec, recording completion, and rethrowing exception if - * encountered. Caller should normally check status before calling + * encountered. Caller should normally check status before calling. + * * @return true if completed normally */ private boolean tryExec() { @@ -414,14 +425,14 @@ public abstract class ForkJoinTask im /** * Main execution method used by worker threads. Invokes - * base computation unless already complete + * base computation unless already complete. */ final void quietlyExec() { if (status >= 0) { try { if (!exec()) return; - } catch(Throwable rex) { + } catch (Throwable rex) { setDoneExceptionally(rex); return; } @@ -430,8 +441,9 @@ public abstract class ForkJoinTask im } /** - * Calls exec, recording but not rethrowing exception - * Caller should normally check status before calling + * Calls exec(), recording but not rethrowing exception. + * Caller should normally check status before calling. + * * @return true if completed normally */ private boolean tryQuietlyInvoke() { @@ -447,12 +459,12 @@ public abstract class ForkJoinTask im } /** - * Cancel, ignoring any exceptions it throws + * Cancels, ignoring any exceptions it throws. */ final void cancelIgnoringExceptions() { try { cancel(false); - } catch(Throwable ignore) { + } catch (Throwable ignore) { } } @@ -464,7 +476,7 @@ public abstract class ForkJoinTask im ForkJoinTask t; while ((s = status) >= 0 && (t = w.scanWhileJoining(this)) != null) t.quietlyExec(); - return (s >= 0)? awaitDone(w, false) : s; // block if no work + return (s >= 0) ? awaitDone(w, false) : s; // block if no work } // public methods @@ -474,11 +486,13 @@ public abstract class ForkJoinTask im * necessarily enforced, it is a usage error to fork a task more * than once unless it has completed and been reinitialized. This * method may be invoked only from within ForkJoinTask - * computations. Attempts to invoke in other contexts result in - * exceptions or errors possibly including ClassCastException. + * computations (as may be determined using method {@link + * #inForkJoinPool}). Attempts to invoke in other contexts result + * in exceptions or errors, possibly including ClassCastException. */ public final void fork() { - ((ForkJoinWorkerThread)(Thread.currentThread())).pushTask(this); + ((ForkJoinWorkerThread) Thread.currentThread()) + .pushTask(this); } /** @@ -499,8 +513,9 @@ public abstract class ForkJoinTask im /** * Commences performing this task, awaits its completion if * necessary, and return its result. + * * @throws Throwable (a RuntimeException, Error, or unchecked - * exception) if the underlying computation did so. + * exception) if the underlying computation did so * @return the computed result */ public final V invoke() { @@ -513,13 +528,15 @@ public abstract class ForkJoinTask im /** * Forks both tasks, returning when {@code isDone} holds for * both of them or an exception is encountered. This method may be - * invoked only from within ForkJoinTask computations. Attempts to - * invoke in other contexts result in exceptions or errors + * invoked only from within ForkJoinTask computations (as may be + * determined using method {@link #inForkJoinPool}). Attempts to + * invoke in other contexts result in exceptions or errors, * possibly including ClassCastException. + * * @param t1 one task * @param t2 the other task * @throws NullPointerException if t1 or t2 are null - * @throws RuntimeException or Error if either task did so. + * @throws RuntimeException or Error if either task did so */ public static void invokeAll(ForkJoinTaskt1, ForkJoinTask t2) { t2.fork(); @@ -531,11 +548,14 @@ public abstract class ForkJoinTask im * Forks the given tasks, returning when {@code isDone} holds * for all of them. If any task encounters an exception, others * may be cancelled. This method may be invoked only from within - * ForkJoinTask computations. Attempts to invoke in other contexts - * result in exceptions or errors possibly including ClassCastException. + * ForkJoinTask computations (as may be determined using method + * {@link #inForkJoinPool}). Attempts to invoke in other contexts + * result in exceptions or errors, possibly including + * ClassCastException. + * * @param tasks the array of tasks - * @throws NullPointerException if tasks or any element are null. - * @throws RuntimeException or Error if any task did so. + * @throws NullPointerException if tasks or any element are null + * @throws RuntimeException or Error if any task did so */ public static void invokeAll(ForkJoinTask... tasks) { Throwable ex = null; @@ -574,20 +594,23 @@ public abstract class ForkJoinTask im * Forks all tasks in the collection, returning when * {@code isDone} holds for all of them. If any task * encounters an exception, others may be cancelled. This method - * may be invoked only from within ForkJoinTask - * computations. Attempts to invoke in other contexts result in - * exceptions or errors possibly including ClassCastException. + * may be invoked only from within ForkJoinTask computations (as + * may be determined using method {@link + * #inForkJoinPool}). Attempts to invoke in other contexts result + * in exceptions or errors, possibly including ClassCastException. + * * @param tasks the collection of tasks - * @throws NullPointerException if tasks or any element are null. - * @throws RuntimeException or Error if any task did so. + * @throws NullPointerException if tasks or any element are null + * @throws RuntimeException or Error if any task did so */ public static void invokeAll(Collection> tasks) { - if (!(tasks instanceof List)) { - invokeAll(tasks.toArray(new ForkJoinTask[tasks.size()])); + if (!(tasks instanceof List)) { + invokeAll(tasks.toArray(new ForkJoinTask[tasks.size()])); return; } + @SuppressWarnings("unchecked") List> ts = - (List>)tasks; + (List>) tasks; Throwable ex = null; int last = ts.size() - 1; for (int i = last; i >= 0; --i) { @@ -623,6 +646,7 @@ public abstract class ForkJoinTask im /** * Returns true if the computation performed by this task has * completed (or has been cancelled). + * * @return true if this computation has completed */ public final boolean isDone() { @@ -631,6 +655,7 @@ public abstract class ForkJoinTask im /** * Returns true if this task was cancelled. + * * @return true if this task was cancelled */ public final boolean isCancelled() { @@ -660,7 +685,7 @@ public abstract class ForkJoinTask im * * @param mayInterruptIfRunning this value is ignored in the * default implementation because tasks are not in general - * cancelled via interruption. + * cancelled via interruption * * @return true if this task is now cancelled */ @@ -670,7 +695,8 @@ public abstract class ForkJoinTask im } /** - * Returns true if this task threw an exception or was cancelled + * Returns true if this task threw an exception or was cancelled. + * * @return true if this task threw an exception or was cancelled */ public final boolean isCompletedAbnormally() { @@ -681,6 +707,7 @@ public abstract class ForkJoinTask im * Returns the exception thrown by the base computation, or a * CancellationException if cancelled, or null if none or if the * method has not yet completed. + * * @return the exception, or null if none */ public final Throwable getException() { @@ -708,7 +735,7 @@ public abstract class ForkJoinTask im */ public void completeExceptionally(Throwable ex) { setDoneExceptionally((ex instanceof RuntimeException) || - (ex instanceof Error)? ex : + (ex instanceof Error) ? ex : new RuntimeException(ex)); } @@ -722,12 +749,12 @@ public abstract class ForkJoinTask im * overridable, but overridden versions must invoke {@code super} * implementation to maintain guarantees. * - * @param value the result value for this task. + * @param value the result value for this task */ public void complete(V value) { try { setRawResult(value); - } catch(Throwable rex) { + } catch (Throwable rex) { setDoneExceptionally(rex); return; } @@ -757,12 +784,15 @@ public abstract class ForkJoinTask im * 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 - * ForkJoinTask computations. Attempts to invoke in other contexts - * result in exceptions or errors possibly including ClassCastException. + * ForkJoinTask computations (as may be determined using method + * {@link #inForkJoinPool}). Attempts to invoke in other contexts + * result in exceptions or errors, possibly including + * ClassCastException. + * * @return the computed result */ public final V helpJoin() { - ForkJoinWorkerThread w = (ForkJoinWorkerThread)(Thread.currentThread()); + ForkJoinWorkerThread w = (ForkJoinWorkerThread) Thread.currentThread(); if (status < 0 || !w.unpushTask(this) || !tryExec()) reportException(busyJoin(w)); return getRawResult(); @@ -771,13 +801,14 @@ public abstract class ForkJoinTask im /** * Possibly executes other tasks until this task is ready. This * method may be invoked only from within ForkJoinTask - * computations. Attempts to invoke in other contexts result in - * exceptions or errors possibly including ClassCastException. + * computations (as may be determined using method {@link + * #inForkJoinPool}). Attempts to invoke in other contexts result + * in exceptions or errors, possibly including ClassCastException. */ public final void quietlyHelpJoin() { if (status >= 0) { ForkJoinWorkerThread w = - (ForkJoinWorkerThread)(Thread.currentThread()); + (ForkJoinWorkerThread) Thread.currentThread(); if (!w.unpushTask(this) || !tryQuietlyInvoke()) busyJoin(w); } @@ -816,8 +847,8 @@ public abstract class ForkJoinTask im * joined, instead executing them until all are processed. */ public static void helpQuiesce() { - ((ForkJoinWorkerThread)(Thread.currentThread())). - helpQuiescePool(); + ((ForkJoinWorkerThread) Thread.currentThread()) + .helpQuiescePool(); } /** @@ -838,13 +869,25 @@ public abstract class ForkJoinTask im /** * Returns the pool hosting the current task execution, or null - * if this task is executing outside of any pool. - * @return the pool, or null if none. + * if this task is executing outside of any ForkJoinPool. + * + * @return the pool, or null if none */ public static ForkJoinPool getPool() { Thread t = Thread.currentThread(); - return ((t instanceof ForkJoinWorkerThread)? - ((ForkJoinWorkerThread)t).pool : null); + return (t instanceof ForkJoinWorkerThread) ? + ((ForkJoinWorkerThread) t).pool : null; + } + + /** + * Returns {@code true} if the current thread is executing as a + * ForkJoinPool computation. + * + * @return {@code true} if the current thread is executing as a + * ForkJoinPool computation, or false otherwise + */ + public static boolean inForkJoinPool() { + return Thread.currentThread() instanceof ForkJoinWorkerThread; } /** @@ -854,12 +897,16 @@ public abstract class ForkJoinTask im * another thread. This method may be useful when arranging * alternative local processing of tasks that could have been, but * were not, stolen. This method may be invoked only from within - * ForkJoinTask computations. Attempts to invoke in other contexts - * result in exceptions or errors possibly including ClassCastException. + * ForkJoinTask computations (as may be determined using method + * {@link #inForkJoinPool}). Attempts to invoke in other contexts + * result in exceptions or errors, possibly including + * ClassCastException. + * * @return true if unforked */ public boolean tryUnfork() { - return ((ForkJoinWorkerThread)(Thread.currentThread())).unpushTask(this); + return ((ForkJoinWorkerThread) Thread.currentThread()) + .unpushTask(this); } /** @@ -867,15 +914,16 @@ public abstract class ForkJoinTask im * forked by the current worker thread but not yet executed. This * value may be useful for heuristic decisions about whether to * fork other tasks. + * * @return the number of tasks */ public static int getQueuedTaskCount() { - return ((ForkJoinWorkerThread)(Thread.currentThread())). - getQueueSize(); + return ((ForkJoinWorkerThread) Thread.currentThread()) + .getQueueSize(); } /** - * Returns a estimate of how many more locally queued tasks are + * Returns an estimate of how many more locally queued tasks are * held by the current worker thread than there are other worker * threads that might steal them. This value may be useful for * heuristic decisions about whether to fork other tasks. In many @@ -883,10 +931,11 @@ public abstract class ForkJoinTask im * aim to maintain a small constant surplus (for example, 3) of * tasks, and to process computations locally if this threshold is * exceeded. + * * @return the surplus number of tasks, which may be negative */ public static int getSurplusQueuedTaskCount() { - return ((ForkJoinWorkerThread)(Thread.currentThread())) + return ((ForkJoinWorkerThread) Thread.currentThread()) .getEstimatedSurplusTaskCount(); } @@ -899,7 +948,7 @@ public abstract class ForkJoinTask im * aid debugging, as well as to support extensions. Its use in any * other context is discouraged. * - * @return the result, or null if not completed. + * @return the result, or null if not completed */ public abstract V getRawResult(); @@ -920,6 +969,7 @@ public abstract class ForkJoinTask im * asynchronous actions that require explicit invocations of * {@code complete} to become joinable. It may throw exceptions * to indicate abnormal exit. + * * @return true if completed normally * @throws Error or RuntimeException if encountered during computation */ @@ -932,13 +982,15 @@ public abstract class ForkJoinTask im * be polled or executed next. This method is designed primarily * to support extensions, and is unlikely to be useful otherwise. * This method may be invoked only from within ForkJoinTask - * computations. Attempts to invoke in other contexts result in - * exceptions or errors possibly including ClassCastException. + * computations (as may be determined using method {@link + * #inForkJoinPool}). Attempts to invoke in other contexts result + * in exceptions or errors, possibly including ClassCastException. * * @return the next task, or null if none are available */ protected static ForkJoinTask peekNextLocalTask() { - return ((ForkJoinWorkerThread)(Thread.currentThread())).peekTask(); + return ((ForkJoinWorkerThread) Thread.currentThread()) + .peekTask(); } /** @@ -946,14 +998,16 @@ public abstract class ForkJoinTask im * queued by the current thread but not yet executed. This method * is designed primarily to support extensions, and is unlikely to * be useful otherwise. This method may be invoked only from - * within ForkJoinTask computations. Attempts to invoke in other - * contexts result in exceptions or errors possibly including + * within ForkJoinTask computations (as may be determined using + * method {@link #inForkJoinPool}). Attempts to invoke in other + * contexts result in exceptions or errors, possibly including * ClassCastException. * * @return the next task, or null if none are available */ protected static ForkJoinTask pollNextLocalTask() { - return ((ForkJoinWorkerThread)(Thread.currentThread())).pollLocalTask(); + return ((ForkJoinWorkerThread) Thread.currentThread()) + .pollLocalTask(); } /** @@ -965,15 +1019,16 @@ public abstract class ForkJoinTask im * of the pool this task is operating in. This method is designed * primarily to support extensions, and is unlikely to be useful * otherwise. This method may be invoked only from within - * ForkJoinTask computations. Attempts to invoke in other contexts - * result in exceptions or errors possibly including + * ForkJoinTask computations (as may be determined using method + * {@link #inForkJoinPool}). Attempts to invoke in other contexts + * result in exceptions or errors, possibly including * ClassCastException. * * @return a task, or null if none are available */ protected static ForkJoinTask pollTask() { - return ((ForkJoinWorkerThread)(Thread.currentThread())). - pollTask(); + return ((ForkJoinWorkerThread) Thread.currentThread()) + .pollTask(); } // Serialization support @@ -984,7 +1039,7 @@ public abstract class ForkJoinTask im * Save the state to a stream. * * @serialData the current run status and the exception thrown - * during execution, or null if none. + * during execution, or null if none * @param s the stream */ private void writeObject(java.io.ObjectOutputStream s) @@ -995,6 +1050,7 @@ public abstract class ForkJoinTask im /** * Reconstitute the instance from a stream. + * * @param s the stream */ private void readObject(java.io.ObjectInputStream s) @@ -1004,7 +1060,7 @@ public abstract class ForkJoinTask im status |= EXTERNAL_SIGNAL; // conservatively set external signal Object ex = s.readObject(); if (ex != null) - setDoneExceptionally((Throwable)ex); + setDoneExceptionally((Throwable) ex); } // Temporary Unsafe mechanics for preliminary release @@ -1033,16 +1089,16 @@ public abstract class ForkJoinTask im private static long fieldOffset(String fieldName) throws NoSuchFieldException { - return _unsafe.objectFieldOffset + return UNSAFE.objectFieldOffset (ForkJoinTask.class.getDeclaredField(fieldName)); } - static final Unsafe _unsafe; + static final Unsafe UNSAFE; static final long statusOffset; static { try { - _unsafe = getUnsafe(); + UNSAFE = getUnsafe(); statusOffset = fieldOffset("status"); } catch (Throwable e) { throw new RuntimeException("Could not initialize intrinsics", e);