[PATCH]: Add proper backtrace functions for sparc.

David Miller davem@davemloft.net
Tue May 20 12:35:00 GMT 2008


From: Jakub Jelinek <jakub@redhat.com>
Date: Tue, 20 May 2008 04:05:11 -0400

> 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.

It was simply easier to do it this way, nothing more.

If you want to do it the debug/backtrace.c way, feel free.



More information about the Libc-alpha mailing list