| commit | 89dd2a87e58c75b0b119de03a5fa258a96b0e8fe | [log] [tgz] |
|---|---|---|
| author | Mao Huang <littlecvr@chromium.org> | Tue Jul 31 21:44:26 2018 |
| committer | Commit Bot <commit-bot@chromium.org> | Tue Jul 31 21:44:26 2018 |
| tree | 2e6fbe85f38b41b8f5bda3e1428181982145dc1d | |
| parent | 10906a154860653a7a6ae7c94b30cd01af59c6c5 [diff] |
tcmalloc: fix heap profiler to always append the process id to the heap dump. Original CLs: - http://codereview.chromium.org/6532051 Fix heap profiler to always append the process id to the heap dump. To run Chrome: HEAP_PROFILE_ALLOCATION_INTERVAL=1000000 HEAPPROFILE=heapprof out/Debug/chrome To analyze: pprof --text out/Debug/chrome heapprof.21026.0039.heap pprof --gv out/Debug/chrome heapprof.21026.0039.heap (pprof is slooow) Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=86273 - https://codereview.chromium.org/8957007 Change the heap file names to be Cleanup'ed in third_party/tcmalloc/chromium. In spite of Chromium's changing heap file names dumped in heap-profiler.cc:DumpProfileLocked(), the file names cleaned-up in heap-profile-table.cc:HeapProfileTable::CleanupOldProfiles are not modified. (See the difference between third_party/tcmalloc/chromium/src/heap-profiler.cc and .../vendor/src/heap-profiler.cc) This patch is originally from http://codereview.chromium.org/7865021/. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114777 BUG=724399,b:70905156 Change-Id: Iab590a6a34f3e3922b0958460169e517accef7df Reviewed-on: https://chromium-review.googlesource.com/1130782 Commit-Queue: Gabriel Marin <gmx@chromium.org> Reviewed-by: Will Harris <wfh@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#579581} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: bf3721aec76107e6b45c766a025fbd0da99946dd