diff options
| author | Ivan Penkov <ivanpe@chromium.org> | 2015-10-05 11:35:09 -0700 |
|---|---|---|
| committer | Ivan Penkov <ivanpe@chromium.org> | 2015-10-05 11:35:09 -0700 |
| commit | f948d8d623ea140ceadca140e273baf13ca16b3f (patch) | |
| tree | 7de76d836e656f87268c0d93607a970aecac18c4 /src/processor/stackwalker_arm_unittest.cc | |
| parent | [mac] Teach dump_syms to handle additional zerofill sections (diff) | |
| download | breakpad-f948d8d623ea140ceadca140e273baf13ca16b3f.tar.xz | |
Increasing the Breakpad stack walker max scan limit from 30 to 40.
Chrome started hitting some crashes in v8 jitted code which happens to be
non ABI compliant and debuggers (including WinDBG) are unable to produce
meaningful stack traces.
The Breakpad stack walker has some builtin heuristics to deal with such cases.
More specifically, when unable to find a good parent frame, it scans the raw
stack to find a suitable parent frame. The max scan size was set at 30
pointers which was (apparently) not enough to recover in this case.
I'm increasing it to 40 pointers. I confirmed that at 34 pointers it was able
to recover however I'm setting it to 40 in order to it some slack.
I needed to update two unittests which were expecting the previous scan limit.
BUG=
R=mark@chromium.org
Review URL: https://codereview.chromium.org/1379433005 .
Diffstat (limited to 'src/processor/stackwalker_arm_unittest.cc')
| -rw-r--r-- | src/processor/stackwalker_arm_unittest.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/processor/stackwalker_arm_unittest.cc b/src/processor/stackwalker_arm_unittest.cc index c73322e6..8a0fd5e9 100644 --- a/src/processor/stackwalker_arm_unittest.cc +++ b/src/processor/stackwalker_arm_unittest.cc @@ -370,7 +370,7 @@ TEST_F(GetCallerFrame, ScanFirstFrame) { .D32(0xF0000000) // more junk .D32(0x0000000D) - .Append(96, 0) // more space + .Append(136, 0) // more space .D32(return_address2) // actual return address // (won't be found) |