public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
From: David Mosberger <davidm@hpl.hp.com>
To: linux-ia64@vger.kernel.org
Subject: Re: [Linux-ia64] Incorrect unwind data in entry.S
Date: Wed, 17 Jan 2001 20:04:48 +0000	[thread overview]
Message-ID: <marc-linux-ia64-105590693005068@msgid-missing> (raw)
In-Reply-To: <marc-linux-ia64-105590678205870@msgid-missing>

>>>>> On Wed, 17 Jan 2001 10:54:26 -0800, Cary Coutant <cary@cup.hp.com> said:

  Cary> Perhaps there's something in the first set of "assorted
  Cary> .spillsp directives," but I don't see where the outer 416-byte
  Cary> stack frame is being allocated or deallocated.

That happens in a very different place of the kernel (the IVT).
Basically, what's happening is that a code fragment A sets up a stack
frame and then calls a handler B, while arranging that B will return
to a code fragment C.  Logically, A and C form a single procedure, but
there really are several different As (and only one C).

  Cary> The code then allocates an additional 576 bytes, but prologue
  Cary> 2 describes the *total* frame size as 576.  It then pops
  Cary> prologue 2, but doesn't deallocate the 576 bytes until the
  Cary> beginning of body region 3, leaving a narrow window where the
  Cary> unwinder will think the frame size is 416 bytes.

Oops, you are right: the size/offsets should have been adjusted by the
size of the first frame.  However, I think we can just drop prologue 1
and 3.  I believe the reason we used to need those was because the
return pointer didn't point to the right place (because the handler B
knew that it's going to "return" to C and simply hardcoded a branch to
C), but with the current setup, "rp" (or the location at which it was
saved) will point to the right place so there is no need to manually
craft unwind info for the frame set up by A.

	--david


      parent reply	other threads:[~2001-01-17 20:04 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-12-21  3:42 [Linux-ia64] Incorrect unwind data in entry.S Keith Owens
2001-01-05 18:48 ` David Mosberger
2001-01-06  0:51 ` Keith Owens
2001-01-06  1:58 ` David Mosberger
2001-01-16 23:41 ` Cary Coutant
2001-01-17  1:34 ` David Mosberger
2001-01-17 18:54 ` Cary Coutant
2001-01-17 20:04 ` David Mosberger [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=marc-linux-ia64-105590693005068@msgid-missing \
    --to=davidm@hpl.hp.com \
    --cc=linux-ia64@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox