This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH]: Add proper backtrace functions for sparc.


On Fri, May 09, 2008 at 05:23:45AM -0700, David Miller wrote:
> 2008-05-09  David S. Miller  <davem@davemloft.net>
> 
> 	* sysdeps/sparc/sparc32/backtrace.c: New file.
> 	* sysdeps/sparc/sparc64/backtrace.c: New file.

Any reason why don't you use just the generic debug/backtrace.c and
instead add sparc specific frame.h?  This header would define
struct layout, FIRST_FRAME_POINTER, CURRENT_STACK_FRAME and on sparc64
also ADVANCE_STACK_FRAME (to add STACK_BIAS), FIRST_FRAME_POINTER would
flush register windows in addition to returning fp.
The reason why several other arches have their own backtrace.c is mainly
because they use unwind info.

	Jakub


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]