docs: Alter descriptions of perf cpu_cycles

https://bugzilla.redhat.com/show_bug.cgi?id=1381714

Alter the descriptions to match what the cpu_cycles actually is

diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
index 1266e9d164b54fb33ab5a385882178800f64366c..b989c8f9a4596655aa548cbd261211a75ab27189 100644 (file)
--- a/docs/formatdomain.html.in
+++ b/docs/formatdomain.html.in
@@ -1954,7 +1954,7 @@
     </tr>
     <tr>
       <td><code>cpu_cycles</code></td>
-      <td>the number of cpu cycles one instruction needs</td>
+      <td>the count of cpu cycles (total/elapsed)</td>
       <td><code>perf.cpu_cycles</code></td>
     </tr>
     <tr>

diff --git a/include/libvirt/libvirt-domain.h b/include/libvirt/libvirt-domain.h
index ec94cdfd226f32c2b9aeaf8dadbfa2ac9376a6ca..2342820e83f8fc2f87c72ac7a249a0ac1fbbb3b6 100644 (file)
--- a/include/libvirt/libvirt-domain.h
+++ b/include/libvirt/libvirt-domain.h
@@ -2008,7 +2008,8 @@ void virDomainStatsRecordListFree(virDomainStatsRecordPtr *stats);
  * VIR_PERF_PARAM_CPU_CYCLES:
  *
  * Macro for typed parameter name that represents cpu_cycles perf event
- * which can be used to measure how many cpu cycles one instruction needs.
+ * describing the total/elapsed cpu cycles. This can be used to measure
+ * how many cpu cycles one instruction needs.
  * It corresponds to the "perf.cpu_cycles" field in the *Stats APIs.
  */
 # define VIR_PERF_PARAM_CPU_CYCLES "cpu_cycles"

diff --git a/src/libvirt-domain.c b/src/libvirt-domain.c
index 9ba9f49a2d52c6b9feb534a4e6dc371bcd4bb8aa..f9f5a46964af2b58219887d377e3dc2846b0bbf6 100644 (file)
--- a/src/libvirt-domain.c
+++ b/src/libvirt-domain.c
@@ -11482,7 +11482,7 @@ virConnectGetDomainCapabilities(virConnectPtr conn,
  *                           It is produced by cache_references perf event.
  * "perf.instructions"     - The count of instructions as unsigned long long.
  *                           It is produced by instructions perf event.
- * "perf.cpu_cycles"       - The number of cpu cycles one instruction needs as
+ * "perf.cpu_cycles"       - The count of cpu cycles (total/elapsed) as an
  *                           unsigned long long. It is produced by cpu_cycles
  *                           perf event.
  *

diff --git a/src/util/virperf.h b/src/util/virperf.h
index 41be8781c67a52d9272be6dee701ef87828c883e..3fca2d0352d7bfc76f64419cb80d20add93ae6ea 100644 (file)
--- a/src/util/virperf.h
+++ b/src/util/virperf.h
@@ -32,7 +32,7 @@ typedef enum {
     VIR_PERF_EVENT_MBMT,   /* Memory Bandwidth Monitoring Total */
     VIR_PERF_EVENT_MBML,   /* Memory Bandwidth Monitor Limit for controller */
 
-    VIR_PERF_EVENT_CPU_CYCLES,       /* CPU Cycles per instruction */
+    VIR_PERF_EVENT_CPU_CYCLES,       /* Count of CPU Cycles (total/elapsed) */
     VIR_PERF_EVENT_INSTRUCTIONS,     /* Count of instructions for application */
     VIR_PERF_EVENT_CACHE_REFERENCES, /* Cache hits by applications */
     VIR_PERF_EVENT_CACHE_MISSES,     /* Cache misses by applications */

diff --git a/tools/virsh.pod b/tools/virsh.pod
index 227c9b22185604039eb0262cbcb471f8be5ad63f..85992debc9f693797d90c7f44239caee6355c240 100644 (file)
--- a/tools/virsh.pod
+++ b/tools/virsh.pod
@@ -934,7 +934,7 @@ I<--perf> returns the statistics of all enabled perf events:
 "perf.cmt" - the cache usage in Byte currently used
 "perf.mbmt" - total system bandwidth from one level of cache
 "perf.mbml" - bandwidth of memory traffic for a memory controller
-"perf.cpu_cycles" - the number of cpu cycles one instruction needs
+"perf.cpu_cycles" - the count of cpu cycles (total/elapsed)
 "perf.instructions" - the count of instructions
 "perf.cache_references" - the count of cache hits
 "perf.cache_misses" - the count of caches misses
@@ -2247,9 +2247,10 @@ B<Valid perf event names>
                      applications running on th e platform.
   instructions     - Provides the count of instructions executed
                      by applications running on the platform.
-  cpu_cycles       - Provides the number of cpu_cycles for one
-                     instruction. May be used with instructions
-                     in order to get a cycles per instruction.
+  cpu_cycles       - Provides the count of cpu cycles
+                     (total/elapsed). May be used with
+                     instructions in order to get a cycles
+                     per instruction.
 
 B<Note>: The statistics can be retrieved using the B<domstats> command using
 the I<--perf> flag.