Hi Martin,
On 11/17/2015 12:39 AM, Martin Buchholz wrote:
> http://cr.openjdk.java.net/~martin/webrevs/openjdk9/jsr166-jdk9-integration/
CancelledLockLoops/: OK
CheckedMapBash/: OK
CompletableFuture/: OK
ConcurrentHashMap/: OK
ReentrantReadWriteLock/: OK
atomic/: OK!
jtregTests/: Glanced over it, seem OK
miscellaneous/: OK
ForkJoinWorkerThread.newThread/:
* I think this one requires CCC, because it changes public API in
newThread.
PhaserBasic/:
* Are those tracing statements ("// trace("barrier action",
startTime);") kept for a reason? Should we conditionalize them with
"static final boolean"-s?
TimeoutLockLoops/:
* Should we use Runtime.availableProcessors() instead of 8?
> The primary focus is making jtreg tests more robust and faster.
> It also contains the changes to j.u.c.atomic that Aleksey is waiting for.
Yes! Thank you. If you push everything in one batch, please split that
one ("atomic/") out to ease backporting to 8u.
Cheers,
-Aleksey
