summaryrefslogtreecommitdiffhomepage
path: root/pkg
diff options
context:
space:
mode:
authorFabricio Voznika <fvoznika@google.com>2019-05-30 12:01:41 -0700
committerShentubot <shentubot@google.com>2019-05-30 12:09:39 -0700
commit38de91b028639ef5f4a4c8874b3ee23503fd2f3a (patch)
tree8d58a49107c491688a89c805336690ca493bf008 /pkg
parente3c5fa3345139a38b380e14067640931ce885288 (diff)
Add build guard to files using go:linkname
Funcion signatures are not validated during compilation. Since they are not exported, they can change at any time. The guard ensures that they are verified at least on every version upgrade. PiperOrigin-RevId: 250733742
Diffstat (limited to 'pkg')
-rw-r--r--pkg/sentry/platform/kvm/bluepill_unsafe.go5
-rw-r--r--pkg/sentry/platform/kvm/machine_unsafe.go5
-rw-r--r--pkg/sentry/platform/ptrace/subprocess_unsafe.go5
-rw-r--r--pkg/sleep/sleep_unsafe.go4
-rw-r--r--pkg/tcpip/link/rawfile/blockingpoll_amd64_unsafe.go5
-rw-r--r--pkg/tcpip/time_unsafe.go4
6 files changed, 24 insertions, 4 deletions
diff --git a/pkg/sentry/platform/kvm/bluepill_unsafe.go b/pkg/sentry/platform/kvm/bluepill_unsafe.go
index 4184939e5..7e8e9f42a 100644
--- a/pkg/sentry/platform/kvm/bluepill_unsafe.go
+++ b/pkg/sentry/platform/kvm/bluepill_unsafe.go
@@ -12,6 +12,11 @@
// See the License for the specific language governing permissions and
// limitations under the License.
+// +build go1.12
+// +build !go1.14
+
+// Check go:linkname function signatures when updating Go version.
+
package kvm
import (
diff --git a/pkg/sentry/platform/kvm/machine_unsafe.go b/pkg/sentry/platform/kvm/machine_unsafe.go
index 452d88d7f..1d3c6d2d6 100644
--- a/pkg/sentry/platform/kvm/machine_unsafe.go
+++ b/pkg/sentry/platform/kvm/machine_unsafe.go
@@ -12,6 +12,11 @@
// See the License for the specific language governing permissions and
// limitations under the License.
+// +build go1.12
+// +build !go1.14
+
+// Check go:linkname function signatures when updating Go version.
+
package kvm
import (
diff --git a/pkg/sentry/platform/ptrace/subprocess_unsafe.go b/pkg/sentry/platform/ptrace/subprocess_unsafe.go
index 17736b05b..b80a3604d 100644
--- a/pkg/sentry/platform/ptrace/subprocess_unsafe.go
+++ b/pkg/sentry/platform/ptrace/subprocess_unsafe.go
@@ -12,6 +12,11 @@
// See the License for the specific language governing permissions and
// limitations under the License.
+// +build go1.12
+// +build !go1.14
+
+// Check go:linkname function signatures when updating Go version.
+
package ptrace
import (
diff --git a/pkg/sleep/sleep_unsafe.go b/pkg/sleep/sleep_unsafe.go
index 0526f52de..8f5e60a25 100644
--- a/pkg/sleep/sleep_unsafe.go
+++ b/pkg/sleep/sleep_unsafe.go
@@ -13,7 +13,9 @@
// limitations under the License.
// +build go1.11
-// +build !go1.13
+// +build !go1.14
+
+// Check go:linkname function signatures when updating Go version.
// Package sleep allows goroutines to efficiently sleep on multiple sources of
// notifications (wakers). It offers O(1) complexity, which is different from
diff --git a/pkg/tcpip/link/rawfile/blockingpoll_amd64_unsafe.go b/pkg/tcpip/link/rawfile/blockingpoll_amd64_unsafe.go
index 0b51982c6..c87268610 100644
--- a/pkg/tcpip/link/rawfile/blockingpoll_amd64_unsafe.go
+++ b/pkg/tcpip/link/rawfile/blockingpoll_amd64_unsafe.go
@@ -13,9 +13,10 @@
// limitations under the License.
// +build linux,amd64
-// +build !go1.13
+// +build go1.12
+// +build !go1.14
-// This must be validated with Go 1.13 and future releases.
+// Check go:linkname function signatures when updating Go version.
package rawfile
diff --git a/pkg/tcpip/time_unsafe.go b/pkg/tcpip/time_unsafe.go
index 1a307483b..a52262e87 100644
--- a/pkg/tcpip/time_unsafe.go
+++ b/pkg/tcpip/time_unsafe.go
@@ -13,7 +13,9 @@
// limitations under the License.
// +build go1.9
-// +build !go1.13
+// +build !go1.14
+
+// Check go:linkname function signatures when updating Go version.
package tcpip