summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJim Warner <james.warner@comcast.net>2020-06-24 00:00:00 -0500
committerCraig Small <csmall@dropbear.xyz>2020-06-25 20:08:22 +1000
commit65ec527e4c4aa1a654b26ab18c8116b23411387f (patch)
treeb631fa13f7d598a411999125d318fddfed1e31c6
parentdf8fb11a47350224240c1c673d69faa7cf723021 (diff)
downloadprocps-ng-65ec527e4c4aa1a654b26ab18c8116b23411387f.tar.gz
top: response to revised numa stuff in that <stat> api
[ actually, this patch goes a little beyond what was ] [ strictly required. some messages were expanded for ] [ clarity and should an inactive node be selected by ] [ the '3' command, a second '2' no longer is needed. ] Signed-off-by: Jim Warner <james.warner@comcast.net>
-rw-r--r--top/top.c33
-rw-r--r--top/top_nls.c4
2 files changed, 21 insertions, 16 deletions
diff --git a/top/top.c b/top/top.c
index 626b5b8..f54ca4c 100644
--- a/top/top.c
+++ b/top/top.c
@@ -249,7 +249,7 @@ enum Rel_statitems {
stat_SUM_TOT };
// cpu/node stack results extractor macros, where e=rel enum, x=index
#define CPU_VAL(e,x) STAT_VAL(e, s_int, Stat_reap->cpus->stacks[x], Stat_ctx)
-#define NOD_VAL(e,x) STAT_VAL(e, s_int, Stat_reap->nodes->stacks[x], Stat_ctx)
+#define NOD_VAL(e,x) STAT_VAL(e, s_int, Stat_reap->numa->stacks[x], Stat_ctx)
#define TIC_VAL(e,s) STAT_VAL(e, sl_int, s, Stat_ctx)
/*
* --- <proc/meminfo.h> ----------------------------------------------- */
@@ -355,7 +355,7 @@ static void bye_bye (const char *str) {
"\n\tProgram"
"\n\t %s"
"\n\t Hertz = %u (%u bytes, %u-bit time)"
- "\n\t Stat_reap->cpus->total = %d, Stat_reap->nodes->total = %d"
+ "\n\t Stat_reap->cpus->total = %d, Stat_reap->numa->total = %d"
"\n\t Pids_itms_tot = %d, sizeof(struct pids_result) = %d, pids stack size = %d"
"\n\t SCREENMAX = %d, ROWMINSIZ = %d, ROWMAXSIZ = %d"
"\n\t PACKAGE = '%s', LOCALEDIR = '%s'"
@@ -383,7 +383,7 @@ static void bye_bye (const char *str) {
, __func__
, PACKAGE_STRING
, (unsigned)Hertz, (unsigned)sizeof(Hertz), (unsigned)sizeof(Hertz) * 8
- , Stat_reap->cpus->total, Stat_reap->nodes->total
+ , Stat_reap->cpus->total, Stat_reap->numa->total
, Pids_itms_tot, (int)sizeof(struct pids_result), (int)(sizeof(struct pids_result) * Pids_itms_tot)
, (int)SCREENMAX, (int)ROWMINSIZ, (int)ROWMAXSIZ
, PACKAGE, LOCALEDIR
@@ -2332,14 +2332,14 @@ static void cpus_refresh (void) {
which = STAT_REAP_CPUS_ONLY;
if (CHKw(Curwin, View_CPUNOD))
- which = STAT_REAP_CPUS_AND_NODES;
+ which = STAT_REAP_NUMA_NODES_TOO;
Stat_reap = procps_stat_reap(Stat_ctx, which, Stat_items, MAXTBL(Stat_items));
if (!Stat_reap)
error_exit(fmtmk(N_fmt(LIB_errorcpu_fmt),__LINE__, strerror(errno)));
// adapt to changes in total numa nodes (assuming it's even possible)
- if (Stat_reap->nodes->total && Stat_reap->nodes->total != Numa_node_tot) {
- Numa_node_tot = Stat_reap->nodes->total;
+ if (Stat_reap->numa->total && Stat_reap->numa->total != Numa_node_tot) {
+ Numa_node_tot = Stat_reap->numa->total;
Numa_node_sel = -1;
}
if (Stat_reap->cpus->total && Stat_reap->cpus->total != Cpu_cnt) {
@@ -3205,7 +3205,7 @@ static void before (char *me) {
struct sigaction sa;
int i, rc;
int linux_version_code = procps_linux_version();
- enum stat_reap_type which = STAT_REAP_CPUS_AND_NODES;
+ enum stat_reap_type which = STAT_REAP_NUMA_NODES_TOO;
atexit(close_stdout);
@@ -3241,7 +3241,7 @@ static void before (char *me) {
error_exit(fmtmk(N_fmt(LIB_errorcpu_fmt),__LINE__, strerror(-rc)));
if (!(Stat_reap = procps_stat_reap(Stat_ctx, which, Stat_items, MAXTBL(Stat_items))))
error_exit(fmtmk(N_fmt(LIB_errorcpu_fmt),__LINE__, strerror(errno)));
- Numa_node_tot = Stat_reap->nodes->total;
+ Numa_node_tot = Stat_reap->numa->total;
Cpu_cnt = Stat_reap->cpus->total;
#ifdef PRETEND48CPU
Cpu_cnt = 48;
@@ -5546,19 +5546,24 @@ numa_oops:
Msg_row += cpu_tics(Stat_reap->summary, N_txt(WORD_allcpus_txt), 1);
// display each cpu node's states
for (i = 0; i < Numa_node_tot; i++) {
- struct stat_stack *nod_ptr = Stat_reap->nodes->stacks[i];
- if (NOD_VAL(stat_ID, i) == STAT_NODE_INVALID) continue;
+ struct stat_stack *nod_ptr = Stat_reap->numa->stacks[i];
+ if (NOD_VAL(stat_NU, i) == STAT_NODE_INVALID) continue;
if (!isROOM(anyFLG, 1)) break;
snprintf(tmp, sizeof(tmp), N_fmt(NUMA_nodenam_fmt), NOD_VAL(stat_ID, i));
Msg_row += cpu_tics(nod_ptr, tmp, 1);
}
} else {
// display the node summary, then the associated cpus (if room)
- for (i = 0; i < Numa_node_tot; i++)
- if (Numa_node_sel == NOD_VAL(stat_ID, i)) break;
- if (i == Numa_node_tot) goto numa_oops;
+ for (i = 0; i < Numa_node_tot; i++) {
+ if (Numa_node_sel == NOD_VAL(stat_ID, i)
+ && (NOD_VAL(stat_NU, i) != STAT_NODE_INVALID)) break;
+ }
+ if (i == Numa_node_tot) {
+ Numa_node_sel = -1;
+ goto numa_oops;
+ }
snprintf(tmp, sizeof(tmp), N_fmt(NUMA_nodenam_fmt), Numa_node_sel);
- Msg_row += cpu_tics(Stat_reap->nodes->stacks[Numa_node_sel], tmp, 1);
+ Msg_row += cpu_tics(Stat_reap->numa->stacks[Numa_node_sel], tmp, 1);
#ifdef PRETEND48CPU
for (i = 0; i < Stat_reap->cpus->total; i++) {
#else
diff --git a/top/top_nls.c b/top/top_nls.c
index cfb9032..990af84 100644
--- a/top/top_nls.c
+++ b/top/top_nls.c
@@ -493,8 +493,8 @@ static void build_norm_nlstab (void) {
/* Translation Hint: The following word 'Node' should be exactly
4 characters, excluding leading %%, fmt chars & colon) */
Norm_nlstab[NUMA_nodenam_fmt] = _("%%Node%-2d:");
- Norm_nlstab[NUMA_nodeget_fmt] = _("expand which node (0-%d)");
- Norm_nlstab[NUMA_nodebad_txt] = _("invalid node");
+ Norm_nlstab[NUMA_nodeget_fmt] = _("expand which numa node (0-%d)");
+ Norm_nlstab[NUMA_nodebad_txt] = _("invalid numa node");
Norm_nlstab[NUMA_nodenot_txt] = _("sorry, NUMA extensions unavailable");
/* Translation Hint: 'Mem ' is an abbreviation for physical memory/ram
. 'Swap' represents the linux swap file --