Remove unnecessary "valid" variable in cpu_perc

[slstatus.git] / components / cpu.c

diff --git a/components/cpu.c b/components/cpu.c
index 02e7671..53af71d 100644 (file)
--- a/components/cpu.c
+++ b/components/cpu.c
@@ -6,20 +6,26 @@
 #include "../util.h"
 
 #if defined(__linux__)
 #include "../util.h"
 
 #if defined(__linux__)

+       #include <inttypes.h>
+       #include <stdint.h>
+

        const char *
        cpu_freq(void)
        {
        const char *
        cpu_freq(void)
        {

-               int freq;
+               uint64_t freq;
+
+               /* in kHz */
+               if (pscanf("/sys/devices/system/cpu/cpu0/cpufreq/scaling_cur_freq",
+                           "%"SCNu64, &freq) != 1) {
+                       return NULL;
+               }

 
 

-               return (pscanf("/sys/devices/system/cpu/cpu0/cpufreq/scaling_cur_freq",
-                              "%i", &freq) == 1) ?
-                      bprintf("%d", (freq + 500) / 1000) : NULL;
+               return fmt_human_10(freq * 1000, "Hz");

        }
 
        const char *
        cpu_perc(void)
        {
        }
 
        const char *
        cpu_perc(void)
        {

-               static int valid;

                static long double a[7];
                long double b[7];
 
                static long double a[7];
                long double b[7];
 


@@ -29,12 +35,11 @@
                           &a[0], &a[1], &a[2], &a[3], &a[4], &a[5], &a[6]) != 7) {
                        return NULL;
                }
                           &a[0], &a[1], &a[2], &a[3], &a[4], &a[5], &a[6]) != 7) {
                        return NULL;
                }

-               if (!valid) {
-                       valid = 1;
+               if (b[0] == 0) {

                        return NULL;
                }
 
                        return NULL;
                }
 

-               return bprintf("%d", (int)(100 *
+               return bprintf("%d%%", (int)(100 *

                               ((b[0] + b[1] + b[2] + b[5] + b[6]) -
                                (a[0] + a[1] + a[2] + a[5] + a[6])) /
                               ((b[0] + b[1] + b[2] + b[3] + b[4] + b[5] + b[6]) -
                               ((b[0] + b[1] + b[2] + b[5] + b[6]) -
                                (a[0] + a[1] + a[2] + a[5] + a[6])) /
                               ((b[0] + b[1] + b[2] + b[3] + b[4] + b[5] + b[6]) -


@@ -56,19 +61,19 @@
 
                size = sizeof(freq);
 
 
                size = sizeof(freq);
 

+               /* in MHz */

                if (sysctl(mib, 2, &freq, &size, NULL, 0) < 0) {
                if (sysctl(mib, 2, &freq, &size, NULL, 0) < 0) {

-                       fprintf(stderr, "sysctl 'HW_CPUSPEED': %s\n", strerror(errno));
+                       warn("sysctl 'HW_CPUSPEED':");

                        return NULL;
                }
 
                        return NULL;
                }
 

-               return bprintf("%d", freq);
+               return fmt_human_10((size_t)freq * 1000 * 1000, "Hz");

        }
 
        const char *
        cpu_perc(void)
        {
                int mib[2];
        }
 
        const char *
        cpu_perc(void)
        {
                int mib[2];

-               static int valid;

                static long int a[CPUSTATES];
                long int b[CPUSTATES];
                size_t size;
                static long int a[CPUSTATES];
                long int b[CPUSTATES];
                size_t size;


@@ -80,15 +85,14 @@
 
                memcpy(b, a, sizeof(b));
                if (sysctl(mib, 2, &a, &size, NULL, 0) < 0) {
 
                memcpy(b, a, sizeof(b));
                if (sysctl(mib, 2, &a, &size, NULL, 0) < 0) {

-                       fprintf(stderr, "sysctl 'KERN_CPTIME': %s\n", strerror(errno));
+                       warn("sysctl 'KERN_CPTIME':");

                        return NULL;
                }
                        return NULL;
                }

-               if (!valid) {
-                       valid = 1;
+               if (b[0] == 0) {

                        return NULL;
                }
 
                        return NULL;
                }
 

-               return bprintf("%d", 100 *
+               return bprintf("%d%%", 100 *

                               ((a[CP_USER] + a[CP_NICE] + a[CP_SYS] + a[CP_INTR]) -
                                (b[CP_USER] + b[CP_NICE] + b[CP_SYS] + b[CP_INTR])) /
                               ((a[CP_USER] + a[CP_NICE] + a[CP_SYS] + a[CP_INTR] +
                               ((a[CP_USER] + a[CP_NICE] + a[CP_SYS] + a[CP_INTR]) -
                                (b[CP_USER] + b[CP_NICE] + b[CP_SYS] + b[CP_INTR])) /
                               ((a[CP_USER] + a[CP_NICE] + a[CP_SYS] + a[CP_INTR] +