savina icon indicating copy to clipboard operation
savina copied to clipboard

Published 20 hours ago • verified publisher icon shamsimam

PiPrecisionAkkaActorBenchmark uses AtomicInteger for numWorkersTerminated

Open smarr opened this issue 8 years ago • 1 comments

Not sure why this benchmark uses an atomic integer, seems to be overkill, no? I would expect this to be already safe: https://github.com/shamsmahmood/savina/blob/master/src/main/scala/edu/rice/habanero/benchmarks/piprecision/PiPrecisionAkkaActorBenchmark.scala#L53

smarr avatar Aug 05 '15 12:08 smarr

Similar issue are in the SleepingBarber and Philosopher benchmark. They use shared Atomics for communication.

smarr avatar Nov 24 '15 09:11 smarr