Remove the new guest metrics feature switch, permanently enabling the

collection of new guest metrics for vROps..

diff --git a/open-vm-tools/lib/include/guestStats.h b/open-vm-tools/lib/include/guestStats.h
index c59b97c884e139d6cf4955f1f76d10c5b271859c..4f5fa6d8813c6cd4bb7c8e3dda4c2b6b7df8cd62 100644 (file)
--- a/open-vm-tools/lib/include/guestStats.h
+++ b/open-vm-tools/lib/include/guestStats.h
@@ -34,6 +34,7 @@
 #include "vm_basic_types.h"
 
 #define PUBLISH_EXPERIMENTAL_STATS   0
+#define ADD_NEW_STATS                0
 
 /*
  * Version 1: Legacy data

diff --git a/open-vm-tools/services/plugins/guestInfo/perfMonLinux.c b/open-vm-tools/services/plugins/guestInfo/perfMonLinux.c
index e6c551e4298192a5f827f722188aa5fccb20cad2..b1323b7db41daca854f0b439b9ffa7b142756573 100644 (file)
--- a/open-vm-tools/services/plugins/guestInfo/perfMonLinux.c
+++ b/open-vm-tools/services/plugins/guestInfo/perfMonLinux.c
@@ -58,7 +58,9 @@ static Bool gInternal = FALSE;
 #if PUBLISH_EXPERIMENTAL_STATS
 static Bool gExperimental = PUBLISH_EXPERIMENTAL_STATS;
 #endif
+#if ADD_NEW_STATS
 static Bool gUnstable = FALSE;
+#endif
 
 #define DECLARE_STAT(publish, file, isRegExp, locatorString, reportID, units, dataType) \
    { file, publish, isRegExp, locatorString, reportID, units, dataType }
@@ -128,9 +130,9 @@ GuestInfoQuery guestInfoQuerySpecTable[] = {
    DECLARE_STAT(&gExperimental,  STAT_FILE,       FALSE, "processes",       GuestStatID_ProcessCreationRate,       GuestUnitsNumberPerSecond, GuestTypeDouble),
 #endif
 
-   DECLARE_STAT(&gUnstable,  STAT_FILE,       FALSE, "procs_running",   GuestStatID_Linux_CpuRunQueue,         GuestUnitsNumber,          GuestTypeUint64),
-   DECLARE_STAT(&gUnstable,  NULL,            FALSE, NULL,              GuestStatID_Linux_DiskRequestQueue,    GuestUnitsNumber,          GuestTypeUint64),
-   DECLARE_STAT(&gUnstable,  NULL,            FALSE, NULL,              GuestStatID_Linux_DiskRequestQueueAvg, GuestUnitsNumber,          GuestTypeDouble),
+   DECLARE_STAT(&gReleased,  STAT_FILE,       FALSE, "procs_running",   GuestStatID_Linux_CpuRunQueue,         GuestUnitsNumber,          GuestTypeUint64),
+   DECLARE_STAT(&gReleased,  NULL,            FALSE, NULL,              GuestStatID_Linux_DiskRequestQueue,    GuestUnitsNumber,          GuestTypeUint64),
+   DECLARE_STAT(&gReleased,  NULL,            FALSE, NULL,              GuestStatID_Linux_DiskRequestQueueAvg, GuestUnitsNumber,          GuestTypeDouble),
 };
 
 #define N_QUERIES (sizeof guestInfoQuerySpecTable / sizeof(GuestInfoQuery))
@@ -869,10 +871,8 @@ GuestInfoCollect(GuestInfoCollector *collector)  // IN/OUT:
    }
 
    GuestInfoDeriveMemNeeded(collector);
-   if (gUnstable) {
-      GuestInfoDecreaseCpuRunQueueByOne(collector);
-      GuestInfoProcDiskStatsData(collector);
-   }
+   GuestInfoDecreaseCpuRunQueueByOne(collector);
+   GuestInfoProcDiskStatsData(collector);
 }
 
 
@@ -1528,10 +1528,12 @@ GuestInfo_StatProviderPoll(gpointer data)
 
    g_debug("Entered guest info stats gather.\n");
 
+#if ADD_NEW_STATS
    gUnstable = g_key_file_get_boolean(ctx->config,
                                       CONFGROUPNAME_GUESTINFO,
                                       "enable-unstable-stats",
                                       NULL);
+#endif
 
    /* Send the vmstats to the VMX. */
    DynBuf_Init(&stats);