[newlib-cygwin] fhandler_process.cc(format_process_stat): fix /proc/pid/stat issues
Corinna Vinschen
corinna@sourceware.org
Wed May 11 08:00:21 GMT 2022
https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=437d0a8f885036f77a6ae606bbe1ccf90fe79729
commit 437d0a8f885036f77a6ae606bbe1ccf90fe79729
Author: Brian Inglis <Brian.Inglis@SystematicSW.ab.ca>
Date: Tue May 10 08:44:42 2022 -0600
fhandler_process.cc(format_process_stat): fix /proc/pid/stat issues
fix tty_nr maj/min bits, vmmaxrss units, and x86 format mismatch:
ctty maj is 31:16, min is 15:0; tty_nr s/b maj 15:8, min 31:20, 7:0;
vmmaxrss s/b bytes not pages;
times all 64 bit - change formats of first two instances from %lu to %U;
realign sprintf formats and variables/values in more logical groups
Diff:
---
winsup/cygwin/fhandler_process.cc | 33 ++++++++++++++++++++-------------
1 file changed, 20 insertions(+), 13 deletions(-)
diff --git a/winsup/cygwin/fhandler_process.cc b/winsup/cygwin/fhandler_process.cc
index 4c42bc015..718945b9a 100644
--- a/winsup/cygwin/fhandler_process.cc
+++ b/winsup/cygwin/fhandler_process.cc
@@ -1092,6 +1092,11 @@ format_process_stat (void *data, char *&destbuf)
vmsize = 0UL, vmrss = 0UL, vmmaxrss = 0UL;
uint64_t utime = 0ULL, stime = 0ULL, start_time = 0ULL;
int nice = 0;
+/* ctty maj is 31:16, min is 15:0; tty_nr s/b maj 15:8, min 31:20, 7:0;
+ maj is 31:16 >> 16 & fff << 8; min is 15:0 >> 8 & ff << 20 | & ff */
+ int tty_nr = (((p->ctty >> 8) & 0xff) << 20)
+ | (((p->ctty >> 16) & 0xfff) << 8)
+ | (p->ctty & 0xff);
if (p->process_state & PID_EXITED)
strcpy (cmd, "<defunct>");
@@ -1171,23 +1176,25 @@ format_process_stat (void *data, char *&destbuf)
else
start_time = (p->start_time - to_time_t (&stodi.BootTime)) * CLOCKS_PER_SEC;
unsigned page_size = wincap.page_size ();
- vmsize = vmc.PagefileUsage;
- vmrss = vmc.WorkingSetSize / page_size;
- vmmaxrss = ql.MaximumWorkingSetSize / page_size;
+ vmsize = vmc.PagefileUsage; /* bytes */
+ vmrss = vmc.WorkingSetSize / page_size; /* pages */
+ vmmaxrss = ql.MaximumWorkingSetSize; /* bytes */
destbuf = (char *) crealloc_abort (destbuf, strlen (cmd) + 320);
return __small_sprintf (destbuf, "%d (%s) %c "
- "%d %d %d %d %d "
- "%u %lu %lu %u %u %lu %lu "
- "%U %U %d %d %d %d "
- "%U %lu "
- "%ld %lu\n",
+ "%d %d %d %d "
+ "%d %u %lu %lu %u %u "
+ "%U %U %U %U "
+ "%d %d %d %d "
+ "%U "
+ "%lu %ld %lu\n",
p->pid, cmd, state,
- p->ppid, p->pgid, p->sid, p->ctty, -1,
- 0, fault_count, fault_count, 0, 0, utime, stime,
- utime, stime, NZERO + nice, nice, 0, 0,
- start_time, vmsize,
- vmrss, vmmaxrss
+ p->ppid, p->pgid, p->sid, tty_nr,
+ -1, 0, fault_count, fault_count, 0, 0,
+ utime, stime, utime, stime,
+ NZERO + nice, nice, 0, 0,
+ start_time,
+ vmsize, vmrss, vmmaxrss
);
}
More information about the Cygwin-cvs
mailing list