Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 272039

Summary: Write OProfile plugin user documentation
Product: z_Archived Reporter: Andrew Overholt <overholt>
Component: LinuxToolsAssignee: Kent Sebastian <ksebasti>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: jjohnstn, linux.oprofile-inbox
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on: 272037    
Bug Blocks:    

Description Andrew Overholt CLA 2009-04-13 12:02:45 EDT
Let's use WikiText to write documentation.  I've made this bug depend upon it.  Related:  bug #272038
Comment 1 Jeff Johnston CLA 2009-04-29 15:24:18 EDT
Comments:

1. Add a brief introduction section on what the OProfile plugin allows one to do (i.e. give brief intro on OProfile, mention profiling running C or C++ applications..whatever).

2. Add File -> Import -> Existing Project -> Archived File to insns on importing demo project.

3. Why is Refresh View needed in OProfile view?  Does it not open all current runs when first opened and refresh as they are modified?

4. In options settings, move up the Timer Interrupt Mode section first which occurs if your platform doesn't support the other events, then the full events page.

5. Mention in configurations that the other tabs match those of a run configuration dialog for options to run the program.

6. Need earlier info on no OProfile output.  I believe that there should always be some output, but that should be a separate bug.
Comment 2 Kent Sebastian CLA 2009-04-29 17:30:23 EDT
(In reply to comment #1)
> Comments:
> 
> 1. Add a brief introduction section on what the OProfile plugin allows one to
> do (i.e. give brief intro on OProfile, mention profiling running C or C++
> applications..whatever).
Done.

> 2. Add File -> Import -> Existing Project -> Archived File to insns on
> importing demo project.
Done.

> 3. Why is Refresh View needed in OProfile view?  Does it not open all current
> runs when first opened and refresh as they are modified?
It doesn't open the current runs when the view first pops up because the oprofile kernel module might not be loaded, so this may trigger a root authentication at a random time, perhaps before the workbench is even shown. I added a line to explain this.

> 4. In options settings, move up the Timer Interrupt Mode section first which
> occurs if your platform doesn't support the other events, then the full events
> page.
Also done.

> 6. Need earlier info on no OProfile output.  I believe that there should always
> be some output, but that should be a separate bug.
Indeed, it has been a thought in the back of my mind, but I let it slide. I opened bz274361 for it.

Anything else? Otherwise I'll close the bug.
Comment 3 Jeff Johnston CLA 2009-04-30 12:09:30 EDT
> 
> Anything else? Otherwise I'll close the bug.
> 

Nope, go ahead and close.
Comment 4 Kent Sebastian CLA 2009-04-30 12:11:41 EDT
Awesome. Thanks for reviewing :)