[PATCH] PowerKVM is a distro not a platform

Dinar valeev <[email protected]> Mon, 17 Aug 2015 18:14:29 +0200
Newsgroups gmane.linux.tools.diag.devel
Message-ID <[email protected]>
From: Dinar Valeev <[email protected]>

Don't mix platform name with distro name. PowerKVM is an IBM
distro. While platform name is OPAL.

Signed-off-by: Dinar Valeev <[email protected]>
---
 src/include/devicetreecollector.hpp              |  2 +-
 src/include/platformcollector.hpp                |  2 +-
 src/internal/sys_interface/platformcollector.cpp | 12 ++++++------
 src/internal/updater.cpp                         |  2 +-
 src/output/lscfg.cpp                             |  4 ++--
 src/output/lsmcode.cpp                           |  2 +-
 src/output/lsvio.cpp                             |  4 ++--
 src/output/lsvpd.cpp                             |  2 +-
 8 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/src/include/devicetreecollector.hpp b/src/include/devicetreecollector.hpp
index 3f833c4..a2fb76a 100755
--- a/src/include/devicetreecollector.hpp
+++ b/src/include/devicetreecollector.hpp
@@ -142,7 +142,7 @@ namespace lsvpd
 			/* Check whether we are on Opal based system */
 			inline bool isPlatformOPAL()
 			{
-				return (platForm == PF_POWERKVM_HOST);
+				return (platForm == PF_OPAL);
 			}
 
 			/**
diff --git a/src/include/platformcollector.hpp b/src/include/platformcollector.hpp
index d95a675..a08aa89 100644
--- a/src/include/platformcollector.hpp
+++ b/src/include/platformcollector.hpp
@@ -22,7 +22,7 @@ using namespace std;
 
 namespace lsvpd {
 
-	enum platform { PF_NULL, PF_POWERVM_LPAR, PF_POWERKVM_HOST , PF_POWERKVM_PSERIES_GUEST, PF_ERROR };
+	enum platform { PF_NULL, PF_POWERVM_LPAR, PF_OPAL , PF_PSERIES_KVM_GUEST, PF_ERROR };
 
 	class PlatformCollector {
 		public:
diff --git a/src/internal/sys_interface/platformcollector.cpp b/src/internal/sys_interface/platformcollector.cpp
index a45e98c..c14f5b2 100644
--- a/src/internal/sys_interface/platformcollector.cpp
+++ b/src/internal/sys_interface/platformcollector.cpp
@@ -72,10 +72,10 @@ error:
 
 		while (getline(ifs, buf)) {
 			if (strstr(buf.c_str(), "PowerNV")) {
-				platform_type = PF_POWERKVM_HOST;
+				platform_type = PF_OPAL;
 				break;
 			} else if (strstr(buf.c_str(), "pSeries (emulated by qemu)")) {
-				platform_type = PF_POWERKVM_PSERIES_GUEST;
+				platform_type = PF_PSERIES_KVM_GUEST;
 				break;
 			} else if (strstr(buf.c_str(), "pSeries")) {
 				platform_type = PF_POWERVM_LPAR;
@@ -94,12 +94,12 @@ error:
 	{
 		get_platform();
 		switch(platform_type) {
-		case PF_POWERKVM_HOST:
-			return "PowerKVM Host";
+		case PF_OPAL:
+			return "OPAL";
 		case PF_POWERVM_LPAR:
 			return "PowerVM pSeries LPAR";
-		case PF_POWERKVM_PSERIES_GUEST:
-			return "PowerKVM pSeries Guest";
+		case PF_PSERIES_KVM_GUEST:
+			return "pSeries KVM Guest";
 		case PF_NULL:
 		case PF_ERROR:
 			return "Unknown";
diff --git a/src/internal/updater.cpp b/src/internal/updater.cpp
index 869c3fd..ed4f305 100755
--- a/src/internal/updater.cpp
+++ b/src/internal/updater.cpp
@@ -87,7 +87,7 @@ int main( int argc, char** argv )
 	string platform = PlatformCollector::get_platform_name();
 
 	switch (PlatformCollector::platform_type) {
-	case PF_POWERKVM_PSERIES_GUEST:
+	case PF_PSERIES_KVM_GUEST:
 	case PF_ERROR:
 		cout<< "vpdupdate is not supported on the " << platform << endl;
 		return 1;
diff --git a/src/output/lscfg.cpp b/src/output/lscfg.cpp
index f9d4ae6..96b8854 100644
--- a/src/output/lscfg.cpp
+++ b/src/output/lscfg.cpp
@@ -657,7 +657,7 @@ int getCPUModelName(System *root, string &name)
 	 * On PowerNV platform we get model name in device tree.
 	 * On pSeries we have to rely on static file.
 	 */
-	if (platform ==  PF_POWERKVM_HOST)
+	if (platform ==  PF_OPAL)
 		return OpalgetCPUModelName(root, name);
 	else
 		return getCPUModelNameFromList(root, name);
@@ -770,7 +770,7 @@ int main( int argc, char** argv )
 	string platform = PlatformCollector::get_platform_name();
 
 	switch (PlatformCollector::platform_type) {
-	case PF_POWERKVM_PSERIES_GUEST:
+	case PF_PSERIES_KVM_GUEST:
 	case PF_ERROR:
 		cout<< argv[0] << " is not supported on the "
 			<< platform << endl;
diff --git a/src/output/lsmcode.cpp b/src/output/lsmcode.cpp
index ee7b5ae..24587f2 100644
--- a/src/output/lsmcode.cpp
+++ b/src/output/lsmcode.cpp
@@ -311,7 +311,7 @@ int main( int argc, char** argv )
 	string platform = PlatformCollector::get_platform_name();
 
 	switch (PlatformCollector::platform_type) {
-	case PF_POWERKVM_PSERIES_GUEST:
+	case PF_PSERIES_KVM_GUEST:
 	case PF_NULL:
 	case PF_ERROR:
 		cout<< "lsmcode is not supported on the " << platform << endl;
diff --git a/src/output/lsvio.cpp b/src/output/lsvio.cpp
index 358c15e..647ef78 100644
--- a/src/output/lsvio.cpp
+++ b/src/output/lsvio.cpp
@@ -218,8 +218,8 @@ int main( int argc, char** argv )
 	string platform = PlatformCollector::get_platform_name();
 
 	switch (PlatformCollector::platform_type) {
-	case PF_POWERKVM_PSERIES_GUEST:
-	case PF_POWERKVM_HOST:
+	case PF_PSERIES_KVM_GUEST:
+	case PF_OPAL:
 	case PF_NULL:
 	case PF_ERROR:
 		cout<< "lsvio is not supported on the " << platform << endl;
diff --git a/src/output/lsvpd.cpp b/src/output/lsvpd.cpp
index 9406d53..185be05 100644
--- a/src/output/lsvpd.cpp
+++ b/src/output/lsvpd.cpp
@@ -403,7 +403,7 @@ int main( int argc, char** argv )
 	string platform = PlatformCollector::get_platform_name();
 
 	switch (PlatformCollector::platform_type) {
-	case PF_POWERKVM_PSERIES_GUEST:
+	case PF_PSERIES_KVM_GUEST:
 	case PF_ERROR:
 		cout<< "lsvpd is not supported on the " << platform << " platform" << endl;
 		return 1;
-- 
2.1.4


------------------------------------------------------------------------------