[prev in list] [next in list] [prev in thread] [next in thread] 

List:       oprofile-list
Subject:    Re: [PATCH 0/7] Update oprofile man page information
From:       Maynard Johnson <maynardj () us ! ibm ! com>
Date:       2013-07-15 22:48:46
Message-ID: 51E47C4E.6030503 () us ! ibm ! com
[Download RAW message or body]

On 07/15/2013 12:19 PM, William Cohen wrote:
> On 07/10/2013 11:44 AM, Maynard Johnson wrote:
> > On 07/05/2013 11:08 AM, William Cohen wrote:
> > > A recent review at Red Hat of the oprofile man pages showed a number
> > > of options that the man pages missed. Also a couple of executable
> > > (oprof_start and op-check-perfevents) that didn't have any man pages.
> > > This series of patches updates the man pages to better document what
> > > is availabe in oprofile.
> > Will,
> > Thanks for these patches.  Sorry for the delay in reviewing them -- I've been \
> > tied up on another project.  I'll try to get to them today. 
> > -Maynard
> 
> Hi Maynard,
> 
> Any comments on the patches?  If they look okay to you, I can go ahead and commit \
> them to the git repo.
Will,
Very sorry for the delay.  The patches look fine, except a couple of whitespace \
issues that 'git apply' complains about with patches 5 and 6.  Yes, please go ahead \
and commit them, but make sure to include a one-liner description for each in the \
commit message.

Thanks very much for these fixes.

-Maynard
> 
> -Will
> 
> > > 
> > > William Cohen (7):
> > > Add the "--no-header" short form option, "-n", to the opreport man
> > > page
> > > Document all the ophelp command options
> > > Add the --merge option description to opannotate man page.
> > > Document the opcontrol short form options on the man page
> > > Add a short man page for op-check-perfevents.
> > > Add man page for oprof_start.
> > > Remove obsolete --note-table-size option from opcontrol
> > > 
> > > configure.ac                 |  2 ++
> > > doc/Makefile.am              |  2 ++
> > > doc/op-check-perfevents.1.in | 36 ++++++++++++++++++++++++++++++++++++
> > > doc/opannotate.1.in          |  4 ++++
> > > doc/opcontrol.1.in           | 24 ++++++++++++------------
> > > doc/ophelp.1.in              | 22 ++++++++++++++++++++++
> > > doc/opreport.1.in            |  2 +-
> > > doc/oprof_start.1.in         | 43 +++++++++++++++++++++++++++++++++++++++++++
> > > utils/opcontrol              | 13 -------------
> > > 9 files changed, 122 insertions(+), 26 deletions(-)
> > > create mode 100644 doc/op-check-perfevents.1.in
> > > create mode 100644 doc/oprof_start.1.in
> > > 
> > 
> > 
> > ------------------------------------------------------------------------------
> > See everything from the browser to the database with AppDynamics
> > Get end-to-end visibility with application monitoring from AppDynamics
> > Isolate bottlenecks and diagnose root cause in seconds.
> > Start your free trial of AppDynamics Pro today!
> > http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
> > _______________________________________________
> > oprofile-list mailing list
> > oprofile-list@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/oprofile-list
> > 
> 


------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
oprofile-list mailing list
oprofile-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/oprofile-list


[prev in list] [next in list] [prev in thread] [next in thread]