Update of /cvsroot/oprofile/oprofile
In directory sc8-pr-cvs1:/tmp/cvs-serv11724
Modified Files:
Tag: pp-interface-branch
ChangeLog TODO
Log Message:
basic --accumulated. Seems to sort of work for now at least.
Index: ChangeLog
===================================================================
RCS file: /cvsroot/oprofile/oprofile/ChangeLog,v
retrieving revision 1.964.2.69
retrieving revision 1.964.2.70
diff -u -d -r1.964.2.69 -r1.964.2.70
--- ChangeLog 8 May 2003 00:30:41 -0000 1.964.2.69
+++ ChangeLog 8 May 2003 07:30:19 -0000 1.964.2.70
@@ -1,3 +1,9 @@
+2003-05-08 John Levon <levon@...>
+
+ * opreport_options.h: add accumulated
+
+ * opreport.cpp: handle --accumulated
+
2003-05-08 Philippe Elie <phil.el@...>
* libpp/symbol_container.cpp: minor tidy
Index: TODO
===================================================================
RCS file: /cvsroot/oprofile/oprofile/TODO,v
retrieving revision 1.203.2.25
retrieving revision 1.203.2.26
diff -u -d -r1.203.2.25 -r1.203.2.26
--- TODO 8 May 2003 00:39:59 -0000 1.203.2.25
+++ TODO 8 May 2003 07:30:19 -0000 1.203.2.26
@@ -12,6 +12,8 @@
- race with opcontrol --shutdown on odb_open() ?
+ o default to opreport -n as discussed
+ o we show a header but not column headers for image report
o write the man page and xml doc
o test opannotate -s -a on an application containing relative path debug
info and profiled in another directory than the application build once.
@@ -32,17 +34,28 @@
o inhibit build of opdiff in pp/Makefile.am
o consider to do the symbol (details too ?) sort at output, format_output receive
a vector of symbol to consider, we can perhaps sort in format_output
+ o cumulative headers do not line up properly
+ o disallow --accumulated without -l
o opreport unhandled option (probably not exhaustive)
--global-percent is not handled according to pp_interface
- --accumulated
--sort= (?)
--output-file filename
o pp/opreport --merge lib /lib/i686/libc-2.3.2.so doesn't do anything when using
separate
+ o pp/opreport --merge=lib. This should give a list of all the binaries shown separately,
+ including the libs, e.g.
+
+ 44 45% /bin/bash
+ 54 45% /lib/libc.so
+
+ it doesn't work ...
+
o opreport /usr/bin/whatever should imply -l (like oprofpp back-compat) but
NOT if opreport <some spec> and it just *happens* to match one image. The
same idea can be applied to remove the app_name field in output since it's
- annoying to get a report with the same application showed on each line
+ annoying to get a report with the same application showed on each line.
+ - difficult one: with -n (to be the default), it's actually useful information.
+ what to do ?
o merging vmlinux as dependent library and as primary image
$ pp/opreport --merge=lib -n | more
@@ -57,6 +70,10 @@
encoding primary image as {root}/usr/bin/make/{dep}/{root}/usr/bin/make. If
we fix it in this way I defer the fix after daemon rewrite
+ - this fix is fine for me (probably better for call stacks anyway) but I'd
+ prefer it to be done *before* any long-drawn daemon rewrite. A daemon rewrite
+ should be an equivalent transformation IMHO otherwise we will have problems...
+
o in some case (--separate=library and not using --merge=lib) we get multiple
time error message "last modified time ... does not match ..."
@@ -71,6 +88,7 @@
It would be nice to allow at least the first alternate form.
o fix John movement on 2.5: odb_insert failure (and message error corrupted ?)
+ - haven't been able to reproduce yet, I just get too many files open instead ;)
After merging with trunk
------------------------
|