]> Cypherpunks.ru repositories - gostls13.git/commit
runtime: implement Pinner API for object pinning
authorSven Anderson <sven@anderson.de>
Sun, 28 Nov 2021 04:05:16 +0000 (13:05 +0900)
committerGopher Robot <gobot@golang.org>
Fri, 19 May 2023 14:59:14 +0000 (14:59 +0000)
commit251daf46fb6531220145603ff3d977f9146a43f6
tree048fe946bad67521e84b12f43baf885c7e05c6b8
parent3b9f99ebaa1eb9209f3bab07f09cf27f06ccce0d
runtime: implement Pinner API for object pinning

Some C APIs require the use or structures that contain pointers to
buffers (iovec, io_uring, ...).  The pointer passing rules would
require that these buffers are allocated in C memory and to process
this data with Go libraries it would need to be copied.

In order to provide a zero-copy way to use these C APIs, this CL
implements a Pinner API that allows to pin Go objects, which
guarantees that the garbage collector does not move these objects
while pinned.  This allows to relax the pointer passing rules so that
pinned pointers can be stored in C allocated memory or can be
contained in Go memory that is passed to C functions.

The Pin() method accepts pointers to objects of any type and
unsafe.Pointer.  Slices and arrays can be pinned by calling Pin()
with the pointer to the first element.  Pinning of maps is not
supported.

If the GC collects unreachable Pinner holding pinned objects it
panics.  If Pin() is called with the other non-pointer types it
panics as well.

Performance considerations: This change has no impact on execution
time on existing code, because checks are only done in code paths,
that would panic otherwise.  The memory footprint on existing code is
one pointer per memory span.

Fixes: #46787
Signed-off-by: Sven Anderson <sven@anderson.de>
Change-Id: I110031fe789b92277ae45a9455624687bd1c54f2
Reviewed-on: https://go-review.googlesource.com/c/go/+/367296
Auto-Submit: Michael Knyszek <mknyszek@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Than McIntosh <thanm@google.com>
Run-TryBot: Michael Knyszek <mknyszek@google.com>
12 files changed:
api/next/46787.txt [new file with mode: 0644]
src/cmd/cgo/internal/testerrors/ptr_test.go
src/runtime/cgocall.go
src/runtime/cgocheck.go
src/runtime/export_test.go
src/runtime/mbitmap.go
src/runtime/mfinal.go
src/runtime/mheap.go
src/runtime/mksizeclasses.go
src/runtime/pinner.go [new file with mode: 0644]
src/runtime/pinner_test.go [new file with mode: 0644]
src/runtime/sizeclasses.go