andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

The benchmark is not taking into account nanoTime granularity (#9033)

Browse Source 
Motivation:

Results are just wrong for small delays.

Modifications:

Switching to AvarageTime avoid to rely on OS nanoTime granularity.

Result:

Uncontended low delay results are not reliable
This commit is contained in:
Francesco Nigro 2019-04-15 15:14:36 +02:00 committed by Norman Maurer
parent 09faa72296
commit fb50847e39
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
microbench/src/main/java/io/netty/buffer/AbstractReferenceCountedByteBufBenchmark.java
View File

@ -49,7 +49,7 @@ public class AbstractReferenceCountedByteBufBenchmark extends AbstractMicrobench
}
@Benchmark
@BenchmarkMode(Mode.SampleTime)
@BenchmarkMode(Mode.AverageTime)
@OutputTimeUnit(TimeUnit.NANOSECONDS)
public boolean retainReleaseUncontended() {
buf.retain();
@ -58,9 +58,9 @@ public class AbstractReferenceCountedByteBufBenchmark extends AbstractMicrobench
}
@Benchmark
@BenchmarkMode(Mode.SampleTime)
@BenchmarkMode(Mode.AverageTime)
@OutputTimeUnit(TimeUnit.NANOSECONDS)
@GroupThreads(6)
@GroupThreads(4)
public boolean retainReleaseContended() {
buf.retain();
Blackhole.consumeCPU(delay);