From: Carlos O'Donell <carlos@baldric.uwo.ca>
To: John David Anglin <dave@hiauly3.hia.nrc.ca>
Cc: randolph@tausq.org, dave.anglin@nrc-cnrc.gc.ca,
parisc-linux@lists.parisc-linux.org
Subject: [parisc-linux] Re: [glibc] tststatic failues, reduced to simple testcase.
Date: Sat, 30 Aug 2003 12:15:04 -0400 [thread overview]
Message-ID: <20030830161504.GE5194@systemhalted> (raw)
In-Reply-To: <200308292244.SAA29829@hiauly3.hia.nrc.ca>
On Fri, Aug 29, 2003 at 06:44:03PM -0400, John David Anglin wrote:
> > > __asm(
> > > 0x4000883c: be,l 100(sr2,r0),%sr0,%r31
> > > 0x40008840: ldi 2,r20 !! FORK !!
> > > );
> > > 0x40008844: ldi -1000,r20 <--Corrupted--- r19 = 0x10106368
>
> Looking at the kernel syscall code, it seems at first glance that
> r19 is saved and restored. Thus, the problem may be specific to fork.
The first place I went to was syscall.S and entry.S to see if r19 was
saved and restored. It is infact saved and restored, _but_ there seems
to be a case in the sys_fork_wrapper where r19 is written back as a temp
slot (PT_XX struct).
linux-2.4/arch/parisc/kernel/entry.S
2004 /* These are call-clobbered registers and therefore
2005 also syscall-clobbered (we hope). */
2006 STREG %r2,PT_GR19(%r1) /* save for child */
2007 STREG %r30,PT_GR21(%r1)
This is done just before the call to 'sys_clone', but it's never used
anywhere. The comment indicates that the author believed he had all
right to use caller saves registers, and they should.
Aflicted: sys_fork_wrapper, sys_clone_wrapper, sys_vfork_wrapper
I'm tempted to remove the store and load of call-clobbered registers
from our syscall path, push them into the glibc wrappers, and see what
happens :)
c.
next prev parent reply other threads:[~2003-08-30 16:16 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20030829084816.GD19341@systemhalted>
[not found] ` <200308291507.LAA13539@hiauly3.hia.nrc.ca>
2003-08-29 20:04 ` [parisc-linux] Re: [glibc] tststatic failues, reduced to simple testcase Carlos O'Donell
2003-08-29 22:03 ` John David Anglin
2003-08-29 22:44 ` John David Anglin
2003-08-30 16:15 ` Carlos O'Donell [this message]
2003-08-31 0:00 ` John David Anglin
2003-08-31 15:38 ` Carlos O'Donell
2003-08-31 18:21 ` [parisc-linux] Re: [glibc] tststatic failues, reduced to simp le testcase John David Anglin
2003-08-31 15:29 ` [parisc-linux] Re: [glibc] tststatic failues, reduced to simple testcase Carlos O'Donell
2003-08-30 15:35 ` Carlos O'Donell
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=20030830161504.GE5194@systemhalted \
--to=carlos@baldric.uwo.ca \
--cc=dave.anglin@nrc-cnrc.gc.ca \
--cc=dave@hiauly3.hia.nrc.ca \
--cc=parisc-linux@lists.parisc-linux.org \
--cc=randolph@tausq.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