]> granicus.if.org Git - procps-ng/commitdiff
top: whack a label & branch in 'summary_show' function
authorJim Warner <james.warner@comcast.net>
Wed, 21 Jul 2021 05:00:00 +0000 (00:00 -0500)
committerCraig Small <csmall@dropbear.xyz>
Tue, 27 Jul 2021 11:17:18 +0000 (21:17 +1000)
If there are zero numa nodes then the View_CPUNOD flag
cannot possibly be set. So it was unnecessary to check
both the numa total and View_CPUNOD in 'summary_show'.

Signed-off-by: Jim Warner <james.warner@comcast.net>
top/top.c

index c69db6b57cc543cf881e5dc2abb3060be9005e74..68fb23afd60f2d9822cf46efa2b73c8c2345693f 100644 (file)
--- a/top/top.c
+++ b/top/top.c
@@ -5574,7 +5574,6 @@ static void summary_show (void) {
 
       cpus_refresh();
 
-      if (!Numa_node_tot) goto numa_nope;
       if (CHKw(w, View_CPUNOD)) {
          if (Numa_node_sel < 0) {
 numa_oops:
@@ -5612,12 +5611,11 @@ numa_oops:
                }
             }
          }
-      } else
 
-numa_nope:
-      if (CHKw(w, View_CPUSUM)) {
+      } else if (CHKw(w, View_CPUSUM)) {
          // display just the 1st /proc/stat line
          Msg_row += cpu_tics(Stat_reap->summary, N_txt(WORD_allcpus_txt), 1);
+
       } else {
          // display each cpu's states separately, screen height permitting...
 #ifdef PRETEND48CPU