tools/xentop: Display '-' when stats are not available.

Displaying 0 is misleading.

Signed-off-by: Pritha Srivastava <pritha.srivastava@citrix.com>
Signed-off-by: Ronan Abhamon <ronan.abhamon@vates.fr>
Acked-by: Wei Liu <wei.liu2@citrix.com>

diff --git a/tools/xenstat/libxenstat/src/xenstat.c b/tools/xenstat/libxenstat/src/xenstat.c
index fbe44f3c56fc6da4a4c9fe159c8e6fffc5ddfb62..8b856b32a3ae5d68a50b125de578d7da23fa3ce5 100644 (file)
--- a/tools/xenstat/libxenstat/src/xenstat.c
+++ b/tools/xenstat/libxenstat/src/xenstat.c
@@ -729,6 +729,12 @@ unsigned long long xenstat_vbd_wr_sects(xenstat_vbd * vbd)
        return vbd->wr_sects;
 }
 
+/* Returns error while getting stats (1 if error happened, 0 otherwise) */
+bool xenstat_vbd_error(xenstat_vbd * vbd)
+{
+       return vbd->error;
+}
+
 /*
  * Tmem functions
  */

diff --git a/tools/xenstat/libxenstat/src/xenstat.h b/tools/xenstat/libxenstat/src/xenstat.h
index 47ec60e14da854624610f818118cd246a454fbc4..9f5053dd92c08c75ef5d16c7e9794f9f922c08e1 100644 (file)
--- a/tools/xenstat/libxenstat/src/xenstat.h
+++ b/tools/xenstat/libxenstat/src/xenstat.h
@@ -20,6 +20,8 @@
 #ifndef XENSTAT_H
 #define XENSTAT_H
 
+#include <stdbool.h>
+
 /* Opaque handles */
 typedef struct xenstat_handle xenstat_handle;
 typedef struct xenstat_domain xenstat_domain;
@@ -193,6 +195,9 @@ unsigned long long xenstat_vbd_wr_reqs(xenstat_vbd * vbd);
 unsigned long long xenstat_vbd_rd_sects(xenstat_vbd * vbd);
 unsigned long long xenstat_vbd_wr_sects(xenstat_vbd * vbd);
 
+/* Returns error while getting stats (1 if error happened, 0 otherwise) */
+bool xenstat_vbd_error(xenstat_vbd * vbd);
+
 /*
  * Tmem functions - extract tmem information
  */

diff --git a/tools/xenstat/libxenstat/src/xenstat_linux.c b/tools/xenstat/libxenstat/src/xenstat_linux.c
index 7cdd3bf91fa034f0cdf48fe4e65612dabf967298..9421ca43c81082d6ce14fa0321129320934a060d 100644 (file)
--- a/tools/xenstat/libxenstat/src/xenstat_linux.c
+++ b/tools/xenstat/libxenstat/src/xenstat_linux.c
@@ -436,13 +436,15 @@ int xenstat_collect_vbds(xenstat_node * node)
                ret = sscanf(dp->d_name, "%3s-%u-%u", buf, &domid, &vbd.dev);
                if (ret != 3)
                        continue;
+               if (!(strstr(buf, "vbd")) && !(strstr(buf, "tap")))
+                       continue;
 
                if (strcmp(buf,"vbd") == 0)
                        vbd.back_type = 1;
                else if (strcmp(buf,"tap") == 0)
                        vbd.back_type = 2;
                else
-                       continue;
+                       vbd.back_type = 0;
 
                domain = xenstat_node_domain(node, domid);
                if (domain == NULL) {
@@ -453,36 +455,29 @@ int xenstat_collect_vbds(xenstat_node * node)
                        continue;
                }
 
-               if((read_attributes_vbd(dp->d_name, "statistics/oo_req", buf, 256)<=0)
-                  || ((ret = sscanf(buf, "%llu", &vbd.oo_reqs)) != 1))
-               {
-                       continue;
-               }
-
-               if((read_attributes_vbd(dp->d_name, "statistics/rd_req", buf, 256)<=0)
-                  || ((ret = sscanf(buf, "%llu", &vbd.rd_reqs)) != 1))
+               if (vbd.back_type == 1 || vbd.back_type == 2)
                {
-                       continue;
-               }
 
-               if((read_attributes_vbd(dp->d_name, "statistics/wr_req", buf, 256)<=0)
-                  || ((ret = sscanf(buf, "%llu", &vbd.wr_reqs)) != 1))
-               {
-                       continue;
-               }
-
-               if((read_attributes_vbd(dp->d_name, "statistics/rd_sect", buf, 256)<=0)
-                  || ((ret = sscanf(buf, "%llu", &vbd.rd_sects)) != 1))
-               {
-                       continue;
+                       vbd.error = 0;
+
+                       if ((read_attributes_vbd(dp->d_name, "statistics/oo_req", buf, 256)<=0) ||
+                               ((ret = sscanf(buf, "%llu", &vbd.oo_reqs)) != 1) ||
+                               (read_attributes_vbd(dp->d_name, "statistics/rd_req", buf, 256)<=0) ||
+                               ((ret = sscanf(buf, "%llu", &vbd.rd_reqs)) != 1) ||
+                               (read_attributes_vbd(dp->d_name, "statistics/wr_req", buf, 256)<=0) ||
+                               ((ret = sscanf(buf, "%llu", &vbd.wr_reqs)) != 1) ||
+                               (read_attributes_vbd(dp->d_name, "statistics/rd_sect", buf, 256)<=0) ||
+                               ((ret = sscanf(buf, "%llu", &vbd.wr_sects)) != 1) ||
+                               (read_attributes_vbd(dp->d_name, "statistics/wr_sect", buf, 256)<=0) ||
+                               ((ret = sscanf(buf, "%llu", &vbd.wr_sects)) != 1))
+                       {
+                               vbd.error = 1;
+                       }
                }
-
-               if((read_attributes_vbd(dp->d_name, "statistics/wr_sect", buf, 256)<=0)
-                  || ((ret = sscanf(buf, "%llu", &vbd.wr_sects)) != 1))
+               else
                {
-                       continue;
+                       vbd.error = 1;
                }
-
                if ((xenstat_save_vbd(domain, &vbd)) == NULL) {
                        perror("Allocation error");
                        return 0;

diff --git a/tools/xenstat/libxenstat/src/xenstat_priv.h b/tools/xenstat/libxenstat/src/xenstat_priv.h
index 74e0774a5e37395e405f81e063b283116afc0c08..ebfcd0fff63f55eeddf17e7d17005293d53a12ad 100644 (file)
--- a/tools/xenstat/libxenstat/src/xenstat_priv.h
+++ b/tools/xenstat/libxenstat/src/xenstat_priv.h
@@ -98,6 +98,7 @@ struct xenstat_network {
 struct xenstat_vbd {
        unsigned int back_type;
        unsigned int dev;
+       unsigned int error;
        unsigned long long oo_reqs;
        unsigned long long rd_reqs;
        unsigned long long wr_reqs;

diff --git a/tools/xenstat/xentop/xentop.c b/tools/xenstat/xentop/xentop.c
index c46581062bf1fe7c52cba28ee09a640f0e439998..31ea0e65a67df1186c1fcc567e2e8adf5e17e6ae 100644 (file)
--- a/tools/xenstat/xentop/xentop.c
+++ b/tools/xenstat/xentop/xentop.c
@@ -87,7 +87,7 @@ static int handle_key(int);
 static int compare(unsigned long long, unsigned long long);
 static int compare_domains(xenstat_domain **, xenstat_domain **);
 static unsigned long long tot_net_bytes( xenstat_domain *, int);
-static unsigned long long tot_vbd_reqs( xenstat_domain *, int);
+static bool tot_vbd_reqs(xenstat_domain *, int, unsigned long long *);
 
 /* Field functions */
 static int compare_state(xenstat_domain *domain1, xenstat_domain *domain2);
@@ -685,91 +685,164 @@ static void print_vbds(xenstat_domain *domain)
    returning -1,0,1 * for <,=,> */
 static int compare_vbd_oo(xenstat_domain *domain1, xenstat_domain *domain2)
 {
-  return -compare(tot_vbd_reqs(domain1, FIELD_VBD_OO),
-                 tot_vbd_reqs(domain2, FIELD_VBD_OO));
+       unsigned long long dom1_vbd_oo = 0, dom2_vbd_oo = 0;
+
+       tot_vbd_reqs(domain1, FIELD_VBD_OO, &dom1_vbd_oo);
+       tot_vbd_reqs(domain1, FIELD_VBD_OO, &dom2_vbd_oo);
+
+       return -compare(dom1_vbd_oo, dom2_vbd_oo);
 }
 
 /* Prints number of total VBD OO requests statistic */
 static void print_vbd_oo(xenstat_domain *domain)
 {
-       print("%8llu", tot_vbd_reqs(domain, FIELD_VBD_OO));
+       unsigned long long vbd_oo;
+
+       if (tot_vbd_reqs(domain, FIELD_VBD_OO, &vbd_oo))
+       {
+               print("%8llu", vbd_oo);
+       }
+       else
+       {
+               print("%8c", '-');
+       }
 }
 
 /* Compares number of total VBD READ requests of two domains,
    returning -1,0,1 * for <,=,> */
 static int compare_vbd_rd(xenstat_domain *domain1, xenstat_domain *domain2)
 {
-       return -compare(tot_vbd_reqs(domain1, FIELD_VBD_RD),
-                       tot_vbd_reqs(domain2, FIELD_VBD_RD));
+       unsigned long long dom1_vbd_rd = 0, dom2_vbd_rd = 0;
+
+       tot_vbd_reqs(domain1, FIELD_VBD_RD, &dom1_vbd_rd);
+       tot_vbd_reqs(domain1, FIELD_VBD_RD, &dom2_vbd_rd);
+
+       return -compare(dom1_vbd_rd, dom1_vbd_rd);
 }
 
 /* Prints number of total VBD READ requests statistic */
 static void print_vbd_rd(xenstat_domain *domain)
 {
-       print("%*llu", fields[FIELD_VBD_RD-1].default_width, tot_vbd_reqs(domain, FIELD_VBD_RD));
+       unsigned long long vbd_rd;
+
+       if (tot_vbd_reqs(domain, FIELD_VBD_RD, &vbd_rd))
+       {
+               print("%*llu", fields[FIELD_VBD_RD-1].default_width, vbd_rd);
+       }
+       else
+       {
+               print("%*c", fields[FIELD_VBD_RD-1].default_width, '-');
+       }
 }
 
 /* Compares number of total VBD WRITE requests of two domains,
    returning -1,0,1 * for <,=,> */
 static int compare_vbd_wr(xenstat_domain *domain1, xenstat_domain *domain2)
 {
-       return -compare(tot_vbd_reqs(domain1, FIELD_VBD_WR),
-                       tot_vbd_reqs(domain2, FIELD_VBD_WR));
+       unsigned long long dom1_vbd_wr = 0, dom2_vbd_wr = 0;
+
+       tot_vbd_reqs(domain1, FIELD_VBD_WR, &dom1_vbd_wr);
+       tot_vbd_reqs(domain1, FIELD_VBD_WR, &dom2_vbd_wr);
+
+       return -compare(dom1_vbd_wr, dom2_vbd_wr);
 }
 
 /* Prints number of total VBD WRITE requests statistic */
 static void print_vbd_wr(xenstat_domain *domain)
 {
-       print("%*llu", fields[FIELD_VBD_WR-1].default_width, tot_vbd_reqs(domain, FIELD_VBD_WR));
+       unsigned long long vbd_wr;
+
+       if (tot_vbd_reqs(domain, FIELD_VBD_WR, &vbd_wr))
+       {
+               print("%*llu", fields[FIELD_VBD_WR-1].default_width, vbd_wr);
+       }
+       else
+       {
+               print("%*c", fields[FIELD_VBD_WR-1].default_width, '-');
+       }
 }
 
 /* Compares number of total VBD READ sectors of two domains,
    returning -1,0,1 * for <,=,> */
 static int compare_vbd_rsect(xenstat_domain *domain1, xenstat_domain *domain2)
 {
-       return -compare(tot_vbd_reqs(domain1, FIELD_VBD_RSECT),
-                       tot_vbd_reqs(domain2, FIELD_VBD_RSECT));
+       unsigned long long dom1_vbd_rsect = 0, dom2_vbd_rsect = 0;
+
+       tot_vbd_reqs(domain1, FIELD_VBD_RSECT, &dom1_vbd_rsect);
+       tot_vbd_reqs(domain1, FIELD_VBD_RSECT, &dom2_vbd_rsect);
+
+       return -compare(dom1_vbd_rsect, dom2_vbd_rsect);
 }
 
 /* Prints number of total VBD READ sectors statistic */
 static void print_vbd_rsect(xenstat_domain *domain)
 {
-       print("%*llu", fields[FIELD_VBD_RSECT-1].default_width, tot_vbd_reqs(domain, FIELD_VBD_RSECT));
+       unsigned long long vbd_rsect;
+
+       if (tot_vbd_reqs(domain, FIELD_VBD_RSECT, &vbd_rsect))
+       {
+               print("%*llu", fields[FIELD_VBD_RSECT-1].default_width, vbd_rsect);
+       }
+       else
+       {
+               print("%*c", fields[FIELD_VBD_RSECT-1].default_width, '-');
+       }
 }
 
 /* Compares number of total VBD WRITE sectors of two domains,
    returning -1,0,1 * for <,=,> */
 static int compare_vbd_wsect(xenstat_domain *domain1, xenstat_domain *domain2)
 {
-       return -compare(tot_vbd_reqs(domain1, FIELD_VBD_WSECT),
-                       tot_vbd_reqs(domain2, FIELD_VBD_WSECT));
+       unsigned long long dom1_vbd_wsect = 0, dom2_vbd_wsect = 0;
+
+       tot_vbd_reqs(domain1, FIELD_VBD_WSECT, &dom1_vbd_wsect);
+       tot_vbd_reqs(domain2, FIELD_VBD_WSECT, &dom2_vbd_wsect);
+
+       return -compare(dom1_vbd_wsect, dom2_vbd_wsect);
 }
 
 /* Prints number of total VBD WRITE sectors statistic */
 static void print_vbd_wsect(xenstat_domain *domain)
 {
-       print("%*llu", fields[FIELD_VBD_WSECT-1].default_width, tot_vbd_reqs(domain, FIELD_VBD_WSECT));
+       unsigned long long vbd_wsect;
+
+       if (tot_vbd_reqs(domain, FIELD_VBD_WSECT, &vbd_wsect))
+       {
+               print("%*llu", fields[FIELD_VBD_WSECT-1].default_width, vbd_wsect);
+       }
+       else
+       {
+               print("%*c", fields[FIELD_VBD_WSECT-1].default_width, '-');
+       }
 }
 
 
-/* Gets number of total VBD requests statistic, 
+/* Gets number of total VBD requests statistic,
  *   if flag is FIELD_VBD_OO, then OO requests,
  *   if flag is FIELD_VBD_RD, then READ requests,
  *   if flag is FIELD_VBD_WR, then WRITE requests,
  *   if flag is FIELD_VBD_RSECT, then READ sectors,
  *   if flag is FIELD_VBD_WSECT, then WRITE sectors.
  */
-static unsigned long long tot_vbd_reqs(xenstat_domain *domain, int flag)
+static bool tot_vbd_reqs(xenstat_domain *domain, int flag, unsigned long long *res)
 {
        int i = 0;
        xenstat_vbd *vbd;
        unsigned num_vbds = 0;
        unsigned long long total = 0;
-       
+       bool show_stats = false;
+
        num_vbds = xenstat_domain_num_vbds(domain);
-       
+       if (!num_vbds)
+               show_stats = true;
+
        for ( i=0 ; i < num_vbds ; i++) {
                vbd = xenstat_domain_vbd(domain,i);
+               if (xenstat_vbd_error(vbd))
+                       continue;
+               else
+                       show_stats = true;
+
                switch(flag) {
                case FIELD_VBD_OO:
                        total += xenstat_vbd_oo_reqs(vbd);
@@ -790,8 +863,10 @@ static unsigned long long tot_vbd_reqs(xenstat_domain *domain, int flag)
                        break;
                }
        }
-       
-       return total;
+
+       *res = total;
+
+       return show_stats;
 }
 
 /* Compares security id (ssid) of two domains, returning -1,0,1 for <,=,> */
@@ -822,6 +897,7 @@ void reset_field_widths(void)
 void adjust_field_widths(xenstat_domain *domain)
 {
        unsigned int length;
+       unsigned long long vbd_rd, vbd_wr, vbd_rsect, vbd_wsect;
 
        if (show_full_name) {
                length = strlen(xenstat_domain_name(domain));
@@ -837,19 +913,23 @@ void adjust_field_widths(xenstat_domain *domain)
        if (length > fields[FIELD_NET_RX-1].default_width)
                fields[FIELD_NET_RX-1].default_width = length;
 
-       length = INT_FIELD_WIDTH((tot_vbd_reqs(domain, FIELD_VBD_RD)) + 1);
+       tot_vbd_reqs(domain, FIELD_VBD_RD, &vbd_rd);
+       length = INT_FIELD_WIDTH(vbd_rd + 1);
        if (length > fields[FIELD_VBD_RD-1].default_width)
                fields[FIELD_VBD_RD-1].default_width = length;
 
-       length = INT_FIELD_WIDTH((tot_vbd_reqs(domain, FIELD_VBD_WR)) + 1);
+       tot_vbd_reqs(domain, FIELD_VBD_WR, &vbd_wr);
+       length = INT_FIELD_WIDTH(vbd_wr + 1);
        if (length > fields[FIELD_VBD_WR-1].default_width)
                fields[FIELD_VBD_WR-1].default_width = length;
 
-       length = INT_FIELD_WIDTH((tot_vbd_reqs(domain, FIELD_VBD_RSECT)) + 1);
+       tot_vbd_reqs(domain, FIELD_VBD_RSECT, &vbd_rsect);
+       length = INT_FIELD_WIDTH(vbd_rsect + 1);
        if (length > fields[FIELD_VBD_RSECT-1].default_width)
                fields[FIELD_VBD_RSECT-1].default_width = length;
 
-       length = INT_FIELD_WIDTH((tot_vbd_reqs(domain, FIELD_VBD_WSECT)) + 1);
+       tot_vbd_reqs(domain, FIELD_VBD_WSECT, &vbd_wsect);
+       length = INT_FIELD_WIDTH(vbd_wsect + 1);
        if (length > fields[FIELD_VBD_WSECT-1].default_width)
                fields[FIELD_VBD_WSECT-1].default_width = length;
 }
@@ -1009,7 +1089,7 @@ void do_vcpu(xenstat_domain *domain)
                if (xenstat_vcpu_online(vcpu) > 0) {
                        if (i != 0 && (i%5)==0)
                                print("\n        ");
-                       print(" %2u: %10llus", i, 
+                       print(" %2u: %10llus", i,
                                        xenstat_vcpu_ns(vcpu)/1000000000);
                }
        }
@@ -1074,15 +1154,24 @@ void do_vbd(xenstat_domain *domain)
                         MAJOR(xenstat_vbd_dev(vbd)),
                         MINOR(xenstat_vbd_dev(vbd)));
 #endif
-
-               print("VBD %s %4d %s OO: %8llu   RD: %8llu   WR: %8llu   RSECT: %10llu   WSECT: %10llu\n",
-                     vbd_type[xenstat_vbd_type(vbd)],
-                     xenstat_vbd_dev(vbd), details,
-                     xenstat_vbd_oo_reqs(vbd),
-                     xenstat_vbd_rd_reqs(vbd),
-                     xenstat_vbd_wr_reqs(vbd),
-                     xenstat_vbd_rd_sects(vbd),
-                     xenstat_vbd_wr_sects(vbd));
+               if (xenstat_vbd_error(vbd))
+               {
+                       print("VBD %s %4d %s OO: %8c   RD: %8c   WR: %8c   RSECT: %10c   WSECT: %10c\n",
+                               vbd_type[xenstat_vbd_type(vbd)],
+                               xenstat_vbd_dev(vbd), details,
+                               '-', '-', '-', '-','-');
+               }
+               else
+               {
+                       print("VBD %s %4d %s OO: %8llu   RD: %8llu   WR: %8llu   RSECT: %10llu   WSECT: %10llu\n",
+                               vbd_type[xenstat_vbd_type(vbd)],
+                               xenstat_vbd_dev(vbd), details,
+                               xenstat_vbd_oo_reqs(vbd),
+                               xenstat_vbd_rd_reqs(vbd),
+                               xenstat_vbd_wr_reqs(vbd),
+                               xenstat_vbd_rd_sects(vbd),
+                               xenstat_vbd_wr_sects(vbd));
+               }
        }
 }