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

Avoid auto boxing in PoolChunk#removeAvailRun (#10769)

Browse Source 
Motivation:
PoolChunk maintains multiple PriorityQueue<Long> collections. The usage
of PoolChunk#removeAvailRun unboxes the Long values to long, and then
this method uses queue.remove(..) which will auto box the value back to
Long. This creates unnecessary allocations via Long.valueOf(long).

Modifications:
- Adjust method signature and usage of PoolChunk#removeAvailRun to avoid
boxing

Result:
Less allocations as a result of PoolChunk#removeAvailRun.
This commit is contained in:
Scott Mitchell 2020-11-03 12:08:43 -08:00 committed by GitHub
parent c5077a3d87
commit 7e1147ea4f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
buffer/src/main/java/io/netty/buffer/PoolChunk.java
View File

@ -250,13 +250,13 @@ final class PoolChunk<T> implements PoolChunkMetric {
assert pre == null;
}
private void removeAvailRun(long handle) {
private void removeAvailRun(Long handle) {
int pageIdxFloor = arena.pages2pageIdxFloor(runPages(handle));
PriorityQueue<Long> queue = runsAvail[pageIdxFloor];
removeAvailRun(queue, handle);
}
private void removeAvailRun(PriorityQueue<Long> queue, long handle) {
private void removeAvailRun(PriorityQueue<Long> queue, Long handle) {
queue.remove(handle);
int runOffset = runOffset(handle);
@ -335,9 +335,9 @@ final class PoolChunk<T> implements PoolChunkMetric {
//get run with min offset in this queue
PriorityQueue<Long> queue = runsAvail[queueIdx];
long handle = queue.poll();
Long handle = queue.poll();
assert !isUsed(handle);
assert handle != null && !isUsed(handle);
removeAvailRun(queue, handle);