[haiku-bugs] [Haiku] #6541: ActivityMonitor only using 4 colors on 8 CPUs.
- From: "jstressman" <trac@xxxxxxxxxxxx>
- Date: Thu, 02 Sep 2010 04:53:21 -0000
#6541: ActivityMonitor only using 4 colors on 8 CPUs.
--------------------------------------------+-------------------------------
Reporter: jstressman | Owner: axeld
Type: enhancement | Status: new
Priority: normal | Milestone: R1
Component: Applications/ActivityMonitor | Version: R1/Development
Keywords: activitymonitor colors | Blocked By:
Has a Patch: 0 | Platform: All
Blocking: |
--------------------------------------------+-------------------------------
I noticed this on a fresh install of r38478 but confirmed that it's been
happening for many months (by looking at an old screenshot from 3/19/2010
and confirming it in that as well).
We discussed this on IRC a bit and the discussion seems to hold some other
possibly interesting information as well from people much more
knowledgeable than I.
(I've tried to trim out the extraneous chatter)
{{{
(11:08:03 AM) phoudoin: the fix will be in
src/apps/activitymonitor/DataSource.cpp, line 878
(11:08:12 AM) phoudoin: only 4 colors are defined.
(11:09:49 AM) largo: phoudoin: thanks. I'll try some mock-ups. how many
colors do you think should be defined?
(11:09:55 AM) largo: just 8 for now until after R1?
(11:10:23 AM) phoudoin: MAX_CPU_COUNT
(11:11:44 AM) phoudoin: B_MAX_CPU_COUNT is set to 8 for R1, yes
(11:11:48 AM) OmniMancer: largo look here on line 880 http://dev.haiku-
os.org/browser/haiku/trunk/src/apps/activitymonitor/DataSource.cpp
(11:12:27 AM) OmniMancer: yes otherwise the struct that uses an embedded
array of that size is no longer the same size and abi compatability breaks
(11:12:49 AM) phoudoin: OmniMancer: I know that's an ABI issue indeed.
(11:19:25 AM) idefix_xifedi: largo: it is strange that your screenshot
shows the middle window with 4 cpu's, two of them having the same colour
(11:19:39 AM) idefix_xifedi: they all should have gotten a different
colour...
(11:20:17 AM) OmniMancer: it probably begind picking colours for each
column of labels
(11:22:36 AM) idefix_xifedi: but if I understand the code in
DataSource.cpp correctly, the colour is directly related to the CPU number
(variable 'cpu')
(11:23:49 AM) OmniMancer: oh yes that could still do it
(11:24:08 AM) OmniMancer: it doesn't necessarily have cpu's from 0..number
of cores you have
(11:24:54 AM) idefix_xifedi: but then the text should't say CPU 0..number
of cores
(11:25:35 AM) idefix_xifedi: as I understand the code, the text "CPU 0"
always implies a colour of 200,0,200
(11:26:34 AM) idefix_xifedi: but I have no way to test this, as I only
have one signle core CPU in my computer (and a slow one at that)...
(11:34:15 AM) phoudoin: I think that's a bug, idefix_xifedi. A
CPUUsageDataSource (implicit CPU #0) is added, but as it report being
PerCpu(), other are added by copy from 0 to # CPU. Should have been from 0
to # CPU *except* fro the first data source's CPU.
(11:36:55 AM) phoudoin: which means we've two data source sharing the same
data history *and* color.
(11:41:53 AM) idefix_xifedi: not sure if I understood you correctly, but
shouldn't the first CPUUsageDataSource not be CPU #0, but the average of
all (8) CPU's?
(11:42:40 AM) phoudoin: Not, that's the object of
CombinedCPUUsageDataSource...
(11:43:07 AM) idefix_xifedi: yeah sorry, just saw that one in the source
code
(11:48:17 AM) phoudoin: Could be related to the previous launch colors
stored in settings: maybe "CPU 3 usage color" in 8 CPUs mode was cyan two
(12:12:26 PM) largo: idefix_xifedi: I tested this on the latest nightly
build a few hours ago. the problem with the colors is still there.
(12:12:36 PM) largo: I only looked at the old screenshot to confirm
whether or not it was a new issue.
(12:13:52 PM) idefix_xifedi: and if you removed the settings file before
launching?
(12:18:28 PM) largo: idefix_xifedi: it was on a fresh install on a newly
formatted partition.
(12:19:05 PM) idefix_xifedi: then you should open a bug-report about that
:)
(12:19:38 PM) largo: will do.
}}}
While I had seen in those older shots only 3 colors used for 4 CPUs, I
can't seem to replicate that behavior now, so that may very well have been
fixed already.
But the 4 colors for 8 CPUs issue is definitely still there. (as only 4
colors are defined, they are reused for any # of CPU over 4)
--
Ticket URL: <http://dev.haiku-os.org/ticket/6541>
Haiku <http://dev.haiku-os.org>
Haiku - the operating system.
Other related posts:
