After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 664926 - display many cores on the sysinfo tab
display many cores on the sysinfo tab
Status: RESOLVED FIXED
Product: system-monitor
Classification: Core
Component: sysinfo
3.3.x
Other Linux
: Low minor
: ---
Assigned To: Robert Roth
System-monitor maintainers
Depends on:
Blocks:
 
 
Reported: 2011-11-27 19:29 UTC by Robert Roth
Modified: 2011-12-11 14:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (7.95 KB, patch)
2011-11-27 23:41 UTC, Robert Roth
none Details | Review

Description Robert Roth 2011-11-27 19:29:38 UTC
When a PC has many cores, the window gets too large to be displayed. See screenshot at [1]. Possible solutions would be:

1. Put the processors description in a scrollable view limited to e.g. 4 or 8 rows. Usually that should be enough without displaying the scrollbar, but with more cores, the window would still be ok.
2. Display the processor model in a single row with a multiplier showing the number of cores. E.g: "4x Intel(R) Core(TM)2 Quad CPU    Q8300  @ 2.50GHz" for a quad-core instead of 4 identical rows. There are some possible problems with this approach: * If there are different CPUs (I don't know if that's possible) they could be displayed in multiple rows, but I don't think that would cause trouble soon, as having more than 13 different CPUs shouldn't be the case soon.
* If the processor model already has a multiplier, it could be confusing (e.g. Turion x2 or other processors could have this issue)

What do you think? Which would be the best option?

[1] http://www.onemansanthology.com/images/fedora/fedora-f16x64-hp-dl385G7-3.0.0Kernel.png
Comment 1 Chris Kühl 2011-11-27 20:04:26 UTC
The GNOME Control Center displays CPUs in the 2nd way which is what makes sense to me. There is code there that can be mostly copy and pasted in.
Comment 2 Robert Roth 2011-11-27 23:41:25 UTC
Created attachment 202259 [details] [review]
Proposed patch

The patch is mostly the code from system monitor copied over, with formatting changes and fixing some warnings which have appeared.
Comment 3 Chris Kühl 2011-12-11 14:36:03 UTC
Thanks. Committed to master.