From 68959e7f929b011446cdf5a5e36740b0c54e4e14 Mon Sep 17 00:00:00 2001
From: Chris Gardner <chris_g_g@hotmail.com>
Date: Wed, 29 May 2013 18:54:43 +0100
Subject: [PATCH] Use 'implementation' if 'brand' not available (Solaris 9).
 Add CPU socket/core counting.

---
 library/system/setup | 32 ++++++++++++++++++++++++++------
 1 file changed, 26 insertions(+), 6 deletions(-)

diff --git a/library/system/setup b/library/system/setup
index 2127071d3f9..27c1651bf92 100644
--- a/library/system/setup
+++ b/library/system/setup
@@ -655,6 +655,8 @@ class SunOSHardware(Hardware):
         return self.facts
 
     def get_cpu_facts(self):
+        physid = 0
+        sockets = {}
         rc, out, err = module.run_command("/usr/bin/kstat cpu_info")
         self.facts['processor'] = []
         for line in out.split('\n'):
@@ -662,15 +664,33 @@ class SunOSHardware(Hardware):
                 continue
             data = line.split(None, 1)
             key = data[0].strip()
-            # key "brand" works on Solaris 10
-            if key == 'brand':
+            # "brand" works on Solaris 10 & 11. "implementation" for Solaris 9.
+            if key == 'module':
+                brand = ''
+            elif key == 'brand':
+                brand = data[1].strip()
+            elif key == 'implementation':
+                processor = brand or data[1].strip()
                 if 'processor' not in self.facts:
                     self.facts['processor'] = []
-                self.facts['processor'].append(data[1].strip())
-        # Counting cores on Solaris can be complicated. Leave as-is for now.
+                self.facts['processor'].append(processor)
+            elif key == 'chip_id':
+                physid = data[1].strip()
+                if physid not in sockets:
+                    sockets[physid] = 1
+                else:
+                    sockets[physid] += 1
+        # Counting cores on Solaris can be complicated.
         # https://blogs.oracle.com/mandalika/entry/solaris_show_me_the_cpu
-        self.facts['processor_cores'] = 'NA'
-        self.facts['processor_count'] = len(self.facts['processor'])
+        # Treat 'processor_count' as physical sockets and 'processor_cores' as
+        # virtual CPUs visisble to Solaris. Not a true count of cores for modern SPARC as
+        # these processors have: sockets -> cores -> threads/virtual CPU.
+        if len(sockets) > 0:
+            self.facts['processor_count'] = len(sockets)
+            self.facts['processor_cores'] = reduce(lambda x, y: x + y, sockets.values())
+        else:
+            self.facts['processor_cores'] = 'NA'
+            self.facts['processor_count'] = len(self.facts['processor'])
 
     def get_memory_facts(self):
         rc, out, err = module.run_command(["/usr/sbin/prtconf"])