| Summary: | sys-process/procps: top: memory based fields goof on >1G | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Raymond Jennings <shentino> |
| Component: | Current packages | Assignee: | Gentoo's Team for Core System packages <base-system> |
| Status: | UNCONFIRMED --- | ||
| Severity: | normal | Keywords: | PATCH |
| Priority: | Normal | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Attachments: |
emerge --info
Screenshot of top, with an orange circle highlighting the problem. patch update |
||
|
Description
Raymond Jennings
2016-08-20 21:46:09 UTC
That's short for terabytes. Try pressing `e' (for task memory scale) and `E' (for summary memory scale) a few times. Also, please post your `emerge --info sys-process/procps' output in a comment. Created attachment 443826 [details]
emerge --info
Pressing "e" does change the scale, but for some reason if I run a program with >10G in RSS, it comes up as 0.010t something instead of 10.XXXg. The VIRT field however shows up fine. It also displays fine if its <10G, so RSS of 9.XXXg shows up as expected, it just switches to terabytes if it goes over <10G, but it should only do that for over <1024G, as 10.00g instead of 0.010t my guess is that its a reaction to the field width being unexpectedly narrower than needed. Pressing 'e' doesn't really help much. As a test, run a program that allocates more than 10G in RSS, and top will show it as 0.010t instead of 10.000g. Created attachment 443834 [details]
Screenshot of top, with an orange circle highlighting the problem.
This shows the 18g vs 0.018t thing I was talking about.
Wrote a quick program to allocate and touch 18g of memory.
This seems like the sort of thing you should report to the upstream developers, but to the Gentoo maintainer. (In reply to Mike Gilbert from comment #6) Sorry, that should say "not to the Gentoo maintainer". Created attachment 444056 [details, diff]
patch
In the meantime, how do you guys like this patch?
Created attachment 444058 [details, diff]
update
DId you send it upstream? |
