Merge pull request #4953 from ImJasonH/cpuinfo

Derive cpuinfo as needed, instead of at init-time

platforms/cpuinfo.go

@@ -21,21 +21,26 @@ import (
 	"os"
 	"runtime"
 	"strings"
+	"sync"
 
 	"github.com/containerd/containerd/errdefs"
 	"github.com/containerd/containerd/log"
 	"github.com/pkg/errors"
 )
 
 // Present the ARM instruction set architecture, eg: v7, v8
-var cpuVariant string
+// Don't use this value directly; call cpuVariant() instead.
+var cpuVariantValue string
 
-func init() {
-	if isArmArch(runtime.GOARCH) {
-		cpuVariant = getCPUVariant()
-	} else {
-		cpuVariant = ""
-	}
+var cpuVariantOnce sync.Once
+
+func cpuVariant() string {
+	cpuVariantOnce.Do(func() {
+		if isArmArch(runtime.GOARCH) {
+			cpuVariantValue = getCPUVariant()
+		}
+	})
+	return cpuVariantValue
 }
 
 // For Linux, the kernel has already detected the ABI, ISA and Features.

platforms/defaults.go

@@ -33,6 +33,6 @@ func DefaultSpec() specs.Platform {
 		OS:           runtime.GOOS,
 		Architecture: runtime.GOARCH,
 		// The Variant field will be empty if arch != ARM.
-		Variant: cpuVariant,
+		Variant: cpuVariant(),
 	}
 }

platforms/defaults_test.go

@@ -28,7 +28,7 @@ func TestDefault(t *testing.T) {
 	expected := specs.Platform{
 		OS:           runtime.GOOS,
 		Architecture: runtime.GOARCH,
-		Variant:      cpuVariant,
+		Variant:      cpuVariant(),
 	}
 	p := DefaultSpec()
 	if !reflect.DeepEqual(p, expected) {

platforms/platforms.go

@@ -189,8 +189,8 @@ func Parse(specifier string) (specs.Platform, error) {
 		if isKnownOS(p.OS) {
 			// picks a default architecture
 			p.Architecture = runtime.GOARCH
-			if p.Architecture == "arm" && cpuVariant != "v7" {
-				p.Variant = cpuVariant
+			if p.Architecture == "arm" && cpuVariant() != "v7" {
+				p.Variant = cpuVariant()
 			}
 
 			return p, nil

platforms/platforms_test.go

@@ -31,8 +31,8 @@ func TestParseSelector(t *testing.T) {
 		defaultVariant = ""
 	)
 
-	if defaultArch == "arm" && cpuVariant != "v7" {
-		defaultVariant = cpuVariant
+	if defaultArch == "arm" && cpuVariant() != "v7" {
+		defaultVariant = cpuVariant()
 	}
 
 	for _, testcase := range []struct {