Help save net neutrality! Learn more.
Close

oprofile Log


Commit Date  
[c85362] by Maynard Johnson Maynard Johnson

Change "Nr. event lost ..." field in the oprofiled.log to "Nr. samples lost ..."

The "Nr. event lost due to buffer overflow" field in the oprofiled.log is
mis-named. It should be "Nr. samples lost due to buffer overflow" in spite
of the fact that the filename in oprofilefs is called 'event_lost_overflow'.

Signed-off-by: Maynard Johnson <maynardj@us.ibm.com>

2012-07-20 15:19:17 Tree
[91919f] by Maynard Johnson Maynard Johnson

[PATCH] Fix how "Nr. non-backtrace samples" from oprofiled.log is calculated

The "Nr. non-backtrace samples" is tracked in the daemon code with
opd_stats[OPD_SAMPLES]. This value is incremented every time sfile_find()
is called from opd_put_sample(). However, if we're processing samples for
which the sfile was previously found and the trans->current value is valid,
we will not call sfile_find() and, thus, the opd_stats[OPD_SAMPLES] stat
will not be incremented.

This problem is most easily seen by starting oprofile on a fairly quiesced
system, then run several instances of the same test program (ideally, at
least as many instances as you have processors). For example, on a laptop
with two processors, I ran 4 copies of a testcase and below is what the
oprofiled.log statistics looked like with this bug:

-----------------------------
-- OProfile Statistics --
Nr. sample dumps: 48
Nr. non-backtrace samples: 329952
Nr. kernel samples: 79505
Nr. lost samples (no kernel/user): 0
. . .
---- Statistics for cpu : 1
Nr. samples lost cpu buffer overflow: 0
Nr. samples received: 944623
Nr. backtrace aborted: 0
Nr. samples lost invalid pc: 0

---- Statistics for cpu : 0
Nr. samples lost cpu buffer overflow: 0
Nr. samples received: 925436
Nr. backtrace aborted: 0
Nr. samples lost invalid pc: 0
oprofiled stopped Thu Jul 19 08:51:47 2012
---------------------------------

As can be seen above, the individual CPU stats show 925436
samples collected, where as the Nr. non-backtrace samples
is 329952. With the fix in this path, the total of the CPU
stats and the Nr. non-backtrace samples are roughly equal.

Signed-off-by: Maynard Johnson <maynardj@us.ibm.com>

2012-07-20 15:17:06 Tree
[8f12d5] by Maynard Johnson Maynard Johnson

Fix configure to not alter user variables and remove non-working --with-gcc option

The configure script for oprofile had a few places
where user variables were being altered in contradiction
to the GNU Automake manual. See the following URL for details:

http://www.gnu.org/software/automake/manual/html_node/Flag-Variables-Ordering.html

In brief, user variables are those listed under "Some influential
environment variables" section of the output from 'configure --help'.
These are variables which the user may set prior to invoking configure,
and thus, the configure script should not alter them. This patch
makes use of new internal variables (e.g., OP_LDFLAGS and OP_CPPFLAGS),
which did unfortunately create quite a ripple effect with the Makefile.am
files. But this was unavoidable in order to correct this error.

Additionally, the '--with-gcc' option does not work anymore because
the generated configure script runs the AC_PROG_CC before processing the
"--with-gcc" option, so it fails to find a compiler. Unsure of when
this stopped working (assume it must have worked in 2005 when it was
initially added), but even fairly old distros (e.g., SLES 10) with
autoconf version 2.59 do not process this option as we would want.
If users wish to configure oprofile to use an alternate compiler,
they should pass the compiler pathnames on the configure command line, thusly:
./configure CC=/my-gcc-install/bin/gcc CXX=/my-gcc-instal/g++

Signed-off-by: Maynard Johnson <maynardj@us.ibm.com>

2012-07-13 17:44:49 Tree
[abdfd8] by Maynard Johnson Maynard Johnson

Add same fix to master branch as already made to perf-events to not discard samples from process 0

2012-05-15 19:56:40 Tree
[bea079] by Maynard Johnson Maynard Johnson

Fix patch committed on Feb 29, 2012 so that samples from PID 0 are not discarded.

2012-05-14 16:20:52 Tree
[a99538] by Maynard Johnson Maynard Johnson

Discard user context kernel samples for which we have no app_cookie

Symptom: Under certain conditions, samples are not attributed to the appropriate
app name. If profiling is done with --separate=lib,kernel,thread and then a
report is generated using a tgid or tid specification to see profile data for one
particular process or thread, you may see anomalies like that shown below. Note
the line with "ext3" under the 'app name' column, when all other lines (rightly)
have "spin".

samples % image name app name symbol name
1018088 99.8455 spin spin .spin
1566 0.1536 ext3 spin /ext3
4 3.9e-04 ext3 ext3 /ext3
1 9.8e-05 ld-2.12.1.so spin ._dl_map_object_from_fd
1 9.8e-05 ld-2.12.1.so spin ._dl_relocate_object
1 9.8e-05 ld-2.12.1.so spin .check_match.8837
1 9.8e-05 libc-2.12.1.so spin ._IO_cleanup

See the big comment in daemon/opd_sfile.c for details of the problem and how
this patch fixes it.

2012-02-29 18:13:20 Tree
[d29bec] by Andreas Krebbel Andreas Krebbel , pushed by Maynard Johnson Maynard Johnson

Fix up s390 implementation to match what was accepted upstream in the kernel

2011-12-13 22:06:16 Tree
[005f26] by Joakim Tjernlund Joakim Tjernlund , pushed by Maynard Johnson Maynard Johnson

Print correct clock speed when using timer mode

2011-11-28 23:55:14 Tree
[2a0cb3] by Maynard Johnson Maynard Johnson

Fix oprofile build warnings when using gcc 4.6.1 or newer

2011-10-24 14:36:57 Tree
[bb738f] by Paul Guo Paul Guo , pushed by Maynard Johnson Maynard Johnson

Add support for Tilera tile64/tilepro/tile-gx processor family

2011-10-19 16:02:42 Tree
[c673c7] by William Cohen William Cohen , pushed by Suravee Suthikulpanit Suravee Suthikulpanit

Fix possible buffer overruns in ibs_init()

2011-07-29 13:20:38 Tree
[5591c3] by William Cohen William Cohen , pushed by Suravee Suthikulpanit Suravee Suthikulpanit

Fix compilation error due to the assembly language instruction cpuid

2011-07-05 02:02:45 Tree
[9529ae] by William Cohen William Cohen

Convert the .cvsignore files into .gitignore files.

2011-02-15 19:50:17 Tree
[24b87e] by Maynard johnson Maynard johnson

Fix non-x86 build issue due to cpuid instruction

2010-12-14 17:40:20 Tree
[132581] by Suravee Suthikulpanit Suravee Suthikulpanit

Add additional IBS supports for AMD processors family12h/14h/15h

2010-11-29 15:05:44 Tree
[f24b68] by Daniel Hansel Daniel Hansel

Fix memory leak according to opening opd_pipe (added missing changes)

2010-04-12 10:01:33 Tree
[d9ff7e] by Daniel Hansel Daniel Hansel

Fix memory leak according to opening opd_pipe

2010-03-17 18:01:08 Tree
[1cb4f1] by Maynard johnson Maynard johnson

Fix start-daemon problem on ia64

2009-10-29 20:59:27 Tree
[4da265] by Maynard johnson Maynard johnson

Handle events that differ only by unit mask - patch 2/2

2009-06-05 15:26:40 Tree
[18fd2b] by Suravee Suthikulpanit Suravee Suthikulpanit

Add patch to fix ext initialization

2009-05-27 19:12:48 Tree
[70074d] by Maynard johnson Maynard johnson

Add IBS support, patch 1 of 4

2009-04-17 18:40:43 Tree
[c7d909] by Maynard johnson Maynard johnson

Add Extended Feature Interface

2009-04-01 20:57:37 Tree
[32a3a3] by Maynard johnson Maynard johnson

whitespace cleanup

2008-11-24 16:39:31 Tree
[688da6] by Daniel Hansel Daniel Hansel

Fixed call of execvp() to execute opjitconv if it is installed in a custom directory (option "--prefix")

2008-08-08 15:08:20 Tree
[9f740c] by Maynard johnson Maynard johnson

Do not force printing of sample_invalid_eip

2008-06-23 19:52:29 Tree
Older >