Revert "Revert "chore: Shrink allocator pool range""
This reverts commit 8f61b0e180
.
This commit is contained in:
parent
d773d335a2
commit
7efd692bbc
2 changed files with 32 additions and 57 deletions
|
@ -12,7 +12,7 @@ var defaultAllocator = NewAllocator()
|
|||
|
||||
// Allocator for incoming frames, optimized to prevent overwriting after zeroing
|
||||
type Allocator struct {
|
||||
buffers [17]sync.Pool
|
||||
buffers [11]sync.Pool
|
||||
}
|
||||
|
||||
// NewAllocator initiates a []byte allocator for frames less than 65536 bytes,
|
||||
|
@ -20,13 +20,7 @@ type Allocator struct {
|
|||
// no more than 50%.
|
||||
func NewAllocator() *Allocator {
|
||||
return &Allocator{
|
||||
buffers: [...]sync.Pool{ // 1B -> 64K
|
||||
{New: func() any { return new([1]byte) }},
|
||||
{New: func() any { return new([1 << 1]byte) }},
|
||||
{New: func() any { return new([1 << 2]byte) }},
|
||||
{New: func() any { return new([1 << 3]byte) }},
|
||||
{New: func() any { return new([1 << 4]byte) }},
|
||||
{New: func() any { return new([1 << 5]byte) }},
|
||||
buffers: [...]sync.Pool{ // 64B -> 64K
|
||||
{New: func() any { return new([1 << 6]byte) }},
|
||||
{New: func() any { return new([1 << 7]byte) }},
|
||||
{New: func() any { return new([1 << 8]byte) }},
|
||||
|
@ -52,46 +46,38 @@ func (alloc *Allocator) Get(size int) []byte {
|
|||
case size > 65536:
|
||||
return make([]byte, size)
|
||||
default:
|
||||
index := msb(size)
|
||||
var index uint16
|
||||
if size > 64 {
|
||||
index = msb(size)
|
||||
if size != 1<<index {
|
||||
index += 1
|
||||
}
|
||||
index -= 6
|
||||
}
|
||||
|
||||
buffer := alloc.buffers[index].Get()
|
||||
switch index {
|
||||
case 0:
|
||||
return buffer.(*[1]byte)[:size]
|
||||
case 1:
|
||||
return buffer.(*[1 << 1]byte)[:size]
|
||||
case 2:
|
||||
return buffer.(*[1 << 2]byte)[:size]
|
||||
case 3:
|
||||
return buffer.(*[1 << 3]byte)[:size]
|
||||
case 4:
|
||||
return buffer.(*[1 << 4]byte)[:size]
|
||||
case 5:
|
||||
return buffer.(*[1 << 5]byte)[:size]
|
||||
case 6:
|
||||
return buffer.(*[1 << 6]byte)[:size]
|
||||
case 7:
|
||||
case 1:
|
||||
return buffer.(*[1 << 7]byte)[:size]
|
||||
case 8:
|
||||
case 2:
|
||||
return buffer.(*[1 << 8]byte)[:size]
|
||||
case 9:
|
||||
case 3:
|
||||
return buffer.(*[1 << 9]byte)[:size]
|
||||
case 10:
|
||||
case 4:
|
||||
return buffer.(*[1 << 10]byte)[:size]
|
||||
case 11:
|
||||
case 5:
|
||||
return buffer.(*[1 << 11]byte)[:size]
|
||||
case 12:
|
||||
case 6:
|
||||
return buffer.(*[1 << 12]byte)[:size]
|
||||
case 13:
|
||||
case 7:
|
||||
return buffer.(*[1 << 13]byte)[:size]
|
||||
case 14:
|
||||
case 8:
|
||||
return buffer.(*[1 << 14]byte)[:size]
|
||||
case 15:
|
||||
case 9:
|
||||
return buffer.(*[1 << 15]byte)[:size]
|
||||
case 16:
|
||||
case 10:
|
||||
return buffer.(*[1 << 16]byte)[:size]
|
||||
default:
|
||||
panic("invalid pool index")
|
||||
|
@ -110,44 +96,33 @@ func (alloc *Allocator) Put(buf []byte) error {
|
|||
if cap(buf) != 1<<bits {
|
||||
return errors.New("allocator Put() incorrect buffer size")
|
||||
}
|
||||
bits -= 6
|
||||
buf = buf[:cap(buf)]
|
||||
|
||||
//nolint
|
||||
//lint:ignore SA6002 ignore temporarily
|
||||
switch bits {
|
||||
case 0:
|
||||
alloc.buffers[bits].Put((*[1]byte)(buf))
|
||||
case 1:
|
||||
alloc.buffers[bits].Put((*[1 << 1]byte)(buf))
|
||||
case 2:
|
||||
alloc.buffers[bits].Put((*[1 << 2]byte)(buf))
|
||||
case 3:
|
||||
alloc.buffers[bits].Put((*[1 << 3]byte)(buf))
|
||||
case 4:
|
||||
alloc.buffers[bits].Put((*[1 << 4]byte)(buf))
|
||||
case 5:
|
||||
alloc.buffers[bits].Put((*[1 << 5]byte)(buf))
|
||||
case 6:
|
||||
alloc.buffers[bits].Put((*[1 << 6]byte)(buf))
|
||||
case 7:
|
||||
case 1:
|
||||
alloc.buffers[bits].Put((*[1 << 7]byte)(buf))
|
||||
case 8:
|
||||
case 2:
|
||||
alloc.buffers[bits].Put((*[1 << 8]byte)(buf))
|
||||
case 9:
|
||||
case 3:
|
||||
alloc.buffers[bits].Put((*[1 << 9]byte)(buf))
|
||||
case 10:
|
||||
case 4:
|
||||
alloc.buffers[bits].Put((*[1 << 10]byte)(buf))
|
||||
case 11:
|
||||
case 5:
|
||||
alloc.buffers[bits].Put((*[1 << 11]byte)(buf))
|
||||
case 12:
|
||||
case 6:
|
||||
alloc.buffers[bits].Put((*[1 << 12]byte)(buf))
|
||||
case 13:
|
||||
case 7:
|
||||
alloc.buffers[bits].Put((*[1 << 13]byte)(buf))
|
||||
case 14:
|
||||
case 8:
|
||||
alloc.buffers[bits].Put((*[1 << 14]byte)(buf))
|
||||
case 15:
|
||||
case 9:
|
||||
alloc.buffers[bits].Put((*[1 << 15]byte)(buf))
|
||||
case 16:
|
||||
case 10:
|
||||
alloc.buffers[bits].Put((*[1 << 16]byte)(buf))
|
||||
default:
|
||||
panic("invalid pool index")
|
||||
|
|
|
@ -13,8 +13,8 @@ func TestAllocGet(t *testing.T) {
|
|||
assert.Equal(t, 1, len(alloc.Get(1)))
|
||||
assert.Equal(t, 2, len(alloc.Get(2)))
|
||||
assert.Equal(t, 3, len(alloc.Get(3)))
|
||||
assert.Equal(t, 4, cap(alloc.Get(3)))
|
||||
assert.Equal(t, 4, cap(alloc.Get(4)))
|
||||
assert.Equal(t, 64, cap(alloc.Get(3)))
|
||||
assert.Equal(t, 64, cap(alloc.Get(4)))
|
||||
assert.Equal(t, 1023, len(alloc.Get(1023)))
|
||||
assert.Equal(t, 1024, cap(alloc.Get(1023)))
|
||||
assert.Equal(t, 1024, len(alloc.Get(1024)))
|
||||
|
|
Loading…
Reference in a new issue