From: Sebastien GODARD Date: Wed, 13 Dec 2017 07:47:49 +0000 (+0100) Subject: pr_stats.c: Don't use an array for tot_jiffies variable X-Git-Tag: v11.7.1~43 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=624374b9caea3a5c164c138662973b867f423995;p=sysstat pr_stats.c: Don't use an array for tot_jiffies variable We don't need 3 variables, only 2 are enough. Signed-off-by: Sebastien GODARD --- diff --git a/pr_stats.c b/pr_stats.c index 0e9f66e..bf45b95 100644 --- a/pr_stats.c +++ b/pr_stats.c @@ -126,7 +126,7 @@ __print_funct_t print_cpu_stats(struct activity *a, int prev, int curr, unsigned long long itv) { int i; - unsigned long long tot_jiffies[3]; + unsigned long long tot_jiffies_c, tot_jiffies_p; unsigned long long deltot_jiffies; struct stats_cpu *scc, *scp; @@ -163,17 +163,17 @@ __print_funct_t print_cpu_stats(struct activity *a, int prev, int curr, * NB: Don't add cpu_guest/cpu_guest_nice because cpu_user/cpu_nice * already include them. */ - tot_jiffies[curr] = scc->cpu_user + scc->cpu_nice + - scc->cpu_sys + scc->cpu_idle + - scc->cpu_iowait + scc->cpu_hardirq + - scc->cpu_steal + scc->cpu_softirq; - tot_jiffies[prev] = scp->cpu_user + scp->cpu_nice + - scp->cpu_sys + scp->cpu_idle + - scp->cpu_iowait + scp->cpu_hardirq + - scp->cpu_steal + scp->cpu_softirq; + tot_jiffies_c = scc->cpu_user + scc->cpu_nice + + scc->cpu_sys + scc->cpu_idle + + scc->cpu_iowait + scc->cpu_hardirq + + scc->cpu_steal + scc->cpu_softirq; + tot_jiffies_p = scp->cpu_user + scp->cpu_nice + + scp->cpu_sys + scp->cpu_idle + + scp->cpu_iowait + scp->cpu_hardirq + + scp->cpu_steal + scp->cpu_softirq; /* Total number of jiffies spent on the interval */ - deltot_jiffies = get_interval(tot_jiffies[prev], tot_jiffies[curr]); + deltot_jiffies = get_interval(tot_jiffies_p, tot_jiffies_c); printf("%-11s", timestamp[curr]); @@ -190,7 +190,7 @@ __print_funct_t print_cpu_stats(struct activity *a, int prev, int curr, * (Remember that guest/guest_nice times are already included in * user/nice modes.) */ - if (tot_jiffies[curr] == 0) { + if (tot_jiffies_c == 0) { /* * Set current struct fields (which have been set to zero) * to values from previous iteration. Hence their values won't