diff options
| author | Ting-Yu Wang <anivia@google.com> | 2020-09-22 17:54:37 -0700 |
|---|---|---|
| committer | gVisor bot <gvisor-bot@google.com> | 2020-09-22 17:56:40 -0700 |
| commit | c0f21bb19a0ff0fd4bc3bd1f0bed9171e43faf68 (patch) | |
| tree | a0c75508fce62d8b2b52eca971d523f77acd10af /pkg/buffer/safemem_test.go | |
| parent | cf3cef1171bdfb41a27d563eb368d4488e0b99f1 (diff) | |
pkg/buffer: Reorganize internal structure to allow dynamic sizes.
This change changes `buffer.data` into a `[]byte`, from `[bufferSize]byte`.
In exchange, each `buffer` is now grouped together to reduce the number of
allocation. Plus, `View` now holds an embeded list of `buffer` (via `pool`) to
support the happy path which the number of buffer is small. Expect no extra
allocation for the happy path.
It is to enable the use case for PacketBuffer, which
* each `View` is small (way less than `defaultBufferSize`), and
* needs to dynamically transfer ownership of `[]byte` to `View`.
(to allow gradual migration)
PiperOrigin-RevId: 333197252
Diffstat (limited to 'pkg/buffer/safemem_test.go')
| -rw-r--r-- | pkg/buffer/safemem_test.go | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/pkg/buffer/safemem_test.go b/pkg/buffer/safemem_test.go index 47f357e0c..721cc5934 100644 --- a/pkg/buffer/safemem_test.go +++ b/pkg/buffer/safemem_test.go @@ -23,6 +23,8 @@ import ( ) func TestSafemem(t *testing.T) { + const bufferSize = defaultBufferSize + testCases := []struct { name string input string |