Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 321252 - [LTTng] Bad statistics in StatisticsView
Summary: [LTTng] Bad statistics in StatisticsView
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: LinuxTools (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Francois Chouinard CLA
QA Contact: Francois Chouinard CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-29 13:38 EDT by Francois Chouinard CLA
Modified: 2022-01-13 14:53 EST (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Francois Chouinard CLA 2010-07-29 13:38:46 EDT 
Some stat counters are not correct (e.g. CPU Time).
Comment 1 Francois Chouinard CLA 2010-07-29 13:43:25 EDT 
Bug321253 attachment contains the fix
Comment 2 Francois Chouinard CLA 2010-07-29 14:05:49 EDT 
The following are notes from Francois Godin who contributed the patch. They reflect the changes he made to the original Statistics View.

[1] Statistics structure are now stored in one central hash map. This reduces the number of 'get's necessary (per event) from 204 to 30. It also reduces the memory usage as no empty (or almost empty) maps are created.

[2] Correction of the event types values. This is a bug fix.

[3] Added some paths for feature parity with LTTv (Process->CPU->Function->Mode->*)

[4] Processes with the same name are no longer considered the same process: the pid is now taken into account when computing the stats.

[5] Functional CPU time. Some CPU time was not correctly accounted to e.g. processes. Now it is.
Comment 3 Francois Chouinard CLA 2010-07-29 14:07:00 EDT 
Fixed when patch in Bug321253 was committed.
Comment 4 Francois Chouinard CLA 2011-07-22 14:46:34 EDT 
Delivered with 0.7