Merge pull request #387 from zeeke/us/arm-vendor

Populate `Vendor` field for ARM cpus
jaypipes · Oct 24, 2024 · c1bfc6e · c1bfc6e
2 parents e050c5a + 37a20d2
commit c1bfc6e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/pkg/cpu/cpu_linux.go b/pkg/cpu/cpu_linux.go
@@ -108,6 +108,8 @@ func processorsGet(ctx *context.Context) []*Processor {
 				proc.Vendor = lp.Attrs["vendor_id"]
 			} else if len(lp.Attrs["isa"]) != 0 { // RISCV64
 				proc.Vendor = lp.Attrs["isa"]
+			} else if lp.Attrs["CPU implementer"] == "0x41" { // ARM
+				proc.Vendor = "ARM"
 			}
 			procs[procID] = proc
 		}

diff --git a/pkg/cpu/cpu_linux_test.go b/pkg/cpu/cpu_linux_test.go
@@ -49,6 +49,9 @@ func TestArmCPU(t *testing.T) {
 	}
 
 	for _, p := range info.Processors {
+		if p.Vendor == "" {
+			t.Fatalf("Expected not empty vendor field.")
+		}
 		if p.TotalCores == 0 {
 			t.Fatalf("Expected >0 cores but got 0.")
 		}