mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2026-01-02 05:10:17 +01:00
gallium/util: whitespace, formatting fixes in u_debug_stack.c
This commit is contained in:
parent
5d2539cb49
commit
6691ba1fe8
1 changed files with 32 additions and 29 deletions
|
|
@ -2,7 +2,7 @@
|
|||
*
|
||||
* Copyright 2009 VMware, Inc.
|
||||
* All Rights Reserved.
|
||||
*
|
||||
*
|
||||
* Permission is hereby granted, free of charge, to any person obtaining a
|
||||
* copy of this software and associated documentation files (the
|
||||
* "Software"), to deal in the Software without restriction, including
|
||||
|
|
@ -10,11 +10,11 @@
|
|||
* distribute, sub license, and/or sell copies of the Software, and to
|
||||
* permit persons to whom the Software is furnished to do so, subject to
|
||||
* the following conditions:
|
||||
*
|
||||
*
|
||||
* The above copyright notice and this permission notice (including the
|
||||
* next paragraph) shall be included in all copies or substantial portions
|
||||
* of the Software.
|
||||
*
|
||||
*
|
||||
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
|
||||
* OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
|
||||
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT.
|
||||
|
|
@ -22,13 +22,13 @@
|
|||
* ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
|
||||
* TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
|
||||
* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
|
||||
*
|
||||
*
|
||||
**************************************************************************/
|
||||
|
||||
/**
|
||||
* @file
|
||||
* Stack backtracing.
|
||||
*
|
||||
*
|
||||
* @author Jose Fonseca <jfonseca@vmware.com>
|
||||
*/
|
||||
|
||||
|
|
@ -44,12 +44,13 @@
|
|||
/**
|
||||
* Capture stack backtrace.
|
||||
*
|
||||
* NOTE: The implementation of this function is quite big, but it is important not to
|
||||
* break it down in smaller functions to avoid adding new frames to the calling stack.
|
||||
* NOTE: The implementation of this function is quite big, but it is important
|
||||
* not to break it down in smaller functions to avoid adding new frames to the
|
||||
* calling stack.
|
||||
*/
|
||||
void
|
||||
debug_backtrace_capture(struct debug_stack_frame *backtrace,
|
||||
unsigned start_frame,
|
||||
unsigned start_frame,
|
||||
unsigned nr_frames)
|
||||
{
|
||||
const void **frame_pointer = NULL;
|
||||
|
|
@ -66,7 +67,8 @@ debug_backtrace_capture(struct debug_stack_frame *backtrace,
|
|||
*/
|
||||
#if defined(PIPE_OS_WINDOWS)
|
||||
{
|
||||
typedef USHORT (WINAPI *PFNCAPTURESTACKBACKTRACE)(ULONG, ULONG, PVOID *, PULONG);
|
||||
typedef USHORT (WINAPI *PFNCAPTURESTACKBACKTRACE)(ULONG, ULONG,
|
||||
PVOID *, PULONG);
|
||||
static PFNCAPTURESTACKBACKTRACE pfnCaptureStackBackTrace = NULL;
|
||||
|
||||
if (!pfnCaptureStackBackTrace) {
|
||||
|
|
@ -76,8 +78,9 @@ debug_backtrace_capture(struct debug_stack_frame *backtrace,
|
|||
assert(hModule);
|
||||
}
|
||||
if (hModule) {
|
||||
pfnCaptureStackBackTrace = (PFNCAPTURESTACKBACKTRACE)GetProcAddress(hModule,
|
||||
"RtlCaptureStackBackTrace");
|
||||
pfnCaptureStackBackTrace =
|
||||
(PFNCAPTURESTACKBACKTRACE)GetProcAddress(hModule,
|
||||
"RtlCaptureStackBackTrace");
|
||||
}
|
||||
}
|
||||
if (pfnCaptureStackBackTrace) {
|
||||
|
|
@ -88,7 +91,8 @@ debug_backtrace_capture(struct debug_stack_frame *backtrace,
|
|||
start_frame += 1;
|
||||
|
||||
assert(start_frame + nr_frames < 63);
|
||||
i = pfnCaptureStackBackTrace(start_frame, nr_frames, (PVOID *) &backtrace->function, NULL);
|
||||
i = pfnCaptureStackBackTrace(start_frame, nr_frames,
|
||||
(PVOID *) &backtrace->function, NULL);
|
||||
|
||||
/* Pad remaing requested frames with NULL */
|
||||
while (i < nr_frames) {
|
||||
|
|
@ -110,50 +114,49 @@ debug_backtrace_capture(struct debug_stack_frame *backtrace,
|
|||
#else
|
||||
frame_pointer = NULL;
|
||||
#endif
|
||||
|
||||
|
||||
|
||||
#ifdef PIPE_ARCH_X86
|
||||
while(nr_frames) {
|
||||
while (nr_frames) {
|
||||
const void **next_frame_pointer;
|
||||
|
||||
if(!frame_pointer)
|
||||
if (!frame_pointer)
|
||||
break;
|
||||
|
||||
if(start_frame)
|
||||
|
||||
if (start_frame)
|
||||
--start_frame;
|
||||
else {
|
||||
backtrace[i++].function = frame_pointer[1];
|
||||
--nr_frames;
|
||||
}
|
||||
|
||||
|
||||
next_frame_pointer = (const void **)frame_pointer[0];
|
||||
|
||||
|
||||
/* Limit the stack walk to avoid referencing undefined memory */
|
||||
if((uintptr_t)next_frame_pointer <= (uintptr_t)frame_pointer ||
|
||||
(uintptr_t)next_frame_pointer > (uintptr_t)frame_pointer + 64*1024)
|
||||
if ((uintptr_t)next_frame_pointer <= (uintptr_t)frame_pointer ||
|
||||
(uintptr_t)next_frame_pointer > (uintptr_t)frame_pointer + 64*1024)
|
||||
break;
|
||||
|
||||
|
||||
frame_pointer = next_frame_pointer;
|
||||
}
|
||||
#else
|
||||
(void) frame_pointer;
|
||||
#endif
|
||||
|
||||
while(nr_frames) {
|
||||
while (nr_frames) {
|
||||
backtrace[i++].function = NULL;
|
||||
--nr_frames;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
||||
void
|
||||
debug_backtrace_dump(const struct debug_stack_frame *backtrace,
|
||||
debug_backtrace_dump(const struct debug_stack_frame *backtrace,
|
||||
unsigned nr_frames)
|
||||
{
|
||||
unsigned i;
|
||||
|
||||
for(i = 0; i < nr_frames; ++i) {
|
||||
if(!backtrace[i].function)
|
||||
|
||||
for (i = 0; i < nr_frames; ++i) {
|
||||
if (!backtrace[i].function)
|
||||
break;
|
||||
debug_symbol_print(backtrace[i].function);
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue