New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 694801 link

Starred by 1 user

Issue metadata

Status: Archived
Owner: ----
Closed: Oct 31
Cc:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug

Blocking:
issue 686208



Sign in to add a comment

Increase backtrace size on windows for use in Native Heap Profiling

Project Member Reported by ajwong@chromium.org, Feb 21 2017

Issue description

This is the windows equivalent of https://bugs.chromium.org/p/chromium/issues/detail?id=617772.

Needed because the native heap profiling code expects a certain number of frames and windows is lagging other platforms.

Note that the existing stack walk code (Based on RtlCaptureStackBackTrace) apparently has an internal limit that we will hit so it may not be as simple as changing a constant.
 

Comment 1 by ajwong@chromium.org, Feb 21 2017

Description: Show this description

Comment 2 by ajwong@chromium.org, Feb 21 2017

Blocking: 686208

Comment 3 by ssid@chromium.org, Feb 23 2017

Cc: ssid@chromium.org dskiba@chromium.org

Comment 4 by dskiba@chromium.org, Feb 23 2017

Hmm, so heap profiler on Windows is using RtlCaptureStackBackTrace and not base::debug::TraceStackFramePointers?

How does RtlCaptureStackBackTrace work - does it rely on frame pointers?

Comment 5 by w...@chromium.org, Mar 12 2017

Re #4: TraceStackFramePointers() isn't currently implemented for Windows, and Windows only supports frame pointers in x86, not x64 or ARM.

base::debug::StackTrace has secondary implementations for Windows which use StackWalk64, which should work for arbitrary stack depths, but requires dbghelp.dll, so we probably need an alternative impl based on RtlVirtualUnwind() or similar.
Labels: Needs-Milestone TE-NeedsTriageHelp
Unable to triage this issue from TE-End, hence adding TE-NeedsTriageHelp label for further triage
Project Member

Comment 7 by sheriffbot@chromium.org, Oct 31

Status: Archived (was: Unconfirmed)
Issue has not been modified or commented on in the last 365 days, please re-open or file a new bug if this is still an issue.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment