luffywu / gperftools

Automatically exported from code.google.com/p/gperftools
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

pprof misses pcs with --base (patch included) #71

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Generate two different pprofs of the same program.
   artificial expample: 
   ls -l /usr/bin | CPUPROFILE=/tmp/b LD_PRELOAD=libprofiler.so.0.0.0 sort 
   ls -l /usr/bin | CPUPROFILE=/tmp/a LD_PRELOAD=libprofiler.so.0.0.0 sort 
2. Use pprof with --base and --lines
     pprof --text --lines --base /tmp/b  /bin/sort /tmp/a

What is the expected output? What do you see instead?

I did not expect a lot of "Use of uninitialized value in concatenation"
messages and addresses instead of lines in the output

What version of the product are you using? On what operating system?

0.98

Please provide any additional information below.

The pcs were not updated with --base. In particular for heapprofiling, one
cannot expect that the pcs in one snapshot are all available in a previous
snapshot.

The attached patch solves the problem.

Tobias

Original issue reported on code.google.com by weidenri...@gmx.de on 1 Aug 2008 at 3:04

Attachments:

GoogleCodeExporter commented 9 years ago
Thank you for the nice bug report and the patch to fix!  I'll have the pprof 
experts
here look this over to make sure there aren't any other, similar issues 
elsewhere,
and get this into the next release.

Original comment by csilv...@gmail.com on 1 Aug 2008 at 11:59

GoogleCodeExporter commented 9 years ago
This was fixed in perftools 0.99, just released.

Original comment by csilv...@gmail.com on 19 Sep 2008 at 8:10