Re: [libunwind] Unwinding using the context passed to a signal handler (doesn't work)

[Matthieu Delahaye]

On Thu, Mar 25, 2004 at 11:04:34AM +0100, Johan Walles wrote:
> David Mosberger wrote:
> >>>>>>On Wed, 24 Mar 2004 10:53:12 +0100, Johan Walles 
> >>>>>><address@hidden> said:
> >
> >  Johan> 1 foo() {
> >  Johan> 2   getcontext();
> >  Johan> 3 }
> >  Johan> 4
> >  Johan> 5 main() {
> >  Johan> 6   foo();
> >  Johan> 7 }
> >
> >  Johan> Inside of foo(), b0 will point to line 7.
> >
> >Wrong.  On _entry_ to foo() b0 will point to line 7.  After the call
> >to getcontext(), b0 will point to line 3.
> 
> My point is that "getcontext(&ctxt); ctxt.uc_mcontext.sc_br[0]" gives me 
> the value 3 that b0 has inside of getcontext(), not the value 7 that b0 has 
> inside of foo() (aka "the call site").

This is false. At the return of getcontext() b0 is still to 3. AFAIK, b0 should
be restored by the callee [It is a scratch register, not a preserved].
The only garanty that b0 is to 7 is at the _entry_ of the function, and
at the return. 



> 
> >  Johan> case UNW_IA64_IP: addr = &uc->uc_mcontext.sc_br[0]; break;
> >
> >This says is that the IP of the caller of
> >getcontext() is stored in sc_br[0].  Could it be stored in sc_ip
> >instead?  Sure.  What would it change?  Absolutely nothing.
> 
> What it would change is that I could then take a signal context that I get 
> from the Linux kernel, fill in the missing registers, pass the result to 
> unw_init_local() and get a complete stack trace, including the first stack 
> frame.  If this doesn't matter to you, could you please change it for my 
> sake?

There is something I don't understand. First you are complaining that the
context you get is a machine-state you say is corresponding to a state inside 
of getcontext() [And thus after a step you get the one from foo()], and now you 
want to use the context sended by the kernel, and add yourself the missing 
information so that by unwinding, you get foo() inside?

> 
> _______________________________________________
> libunwind mailing list
> address@hidden
> http://www.hpl.hp.com/hosted/linux/mail-archives/libunwind/

-- 
It were not best that we should all think alike; it is difference of opinion
that makes horse-races.
                -- Mark Twain, "Pudd'nhead Wilson's Calendar"