From: Jurij Smakov <jurij@wooyd.org>
To: sparclinux@vger.kernel.org
Subject: Re: longjmp question
Date: Fri, 14 Oct 2011 23:06:53 +0000 [thread overview]
Message-ID: <20111014230653.GA14179@wooyd.org> (raw)
In-Reply-To: <20111007232209.GA11892@wooyd.org>
On Thu, Oct 13, 2011 at 06:35:18PM -0400, David Miller wrote:
> Something you might want to try, compile cont.c into an assembler
> file cont.s, then insert the following around the flushw
>
> mov %fp, %g1
> flushw
> mov %fp, %g2
>
> Then compile that into an object and link up ruby.
>
> In the debugger, breakpoint right after that "mov %fp, %g2" and
> print out from GDB the values of %g1 and %g2. This might give
> some hints as to what's going on exactly.
>
> Another test, go into Ruby's defines.h and get rid of the:
>
> # if defined(__sparc_v9__) || defined(__sparcv9) || defined(__arch64__)
> ("flushw")
> # else
>
> and make it always use "ta 0x03" instead of "flushw". This might
> explain why the Ruby developers can't reproduce this on Solaris. That
> could happen if for some reason their Solaris build isn't setting the
> defines that guard the flushw instruction usage.
>
> If using "ta 0x03" instead of "flushw" makes a difference that would
> be a huge clue.
Replacing "flushw" with "ta 0x03" makes the problem go away. What is
the difference between the two? I would naively think that the effect
of both should be saving register windows on the stack, allocating a
new stack frame for each of them, so fp would get adjusted in either
case. Then I would expect that the correct fix would be to indicate to
the compiler that flushw is clobbering fp/sp registers, so it cannot
rely on their contents afterwards. The fact that using "ta 0x03" fixes
it makes me feel lost again :-).
Unfortunately, I was unable to do a conclusive second test (copying fp
to some registers before and after flushw) so far, as inserting a
break even after flushw but before memcpy() call makes the problem go
away as well. I can break after memcpy, but it might clobber the
registers, so the saving part would need to be a bit more elaborate.
Best regards,
--
Jurij Smakov jurij@wooyd.org
Key: http://www.wooyd.org/pgpkey/ KeyID: C99E03CC
next prev parent reply other threads:[~2011-10-14 23:06 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-07 23:22 longjmp question Jurij Smakov
2011-10-07 23:42 ` David Miller
2011-10-08 6:43 ` David Miller
2011-10-08 12:55 ` Jurij Smakov
2011-10-08 19:22 ` David Miller
2011-10-12 21:22 ` Jurij Smakov
2011-10-12 22:09 ` David Miller
2011-10-12 22:17 ` David Miller
2011-10-12 23:06 ` David Miller
2011-10-12 23:21 ` Jurij Smakov
2011-10-12 23:42 ` David Miller
2011-10-13 22:06 ` Jurij Smakov
2011-10-13 22:35 ` David Miller
2011-10-14 23:06 ` Jurij Smakov [this message]
2011-10-14 23:26 ` David Miller
2011-10-16 17:07 ` Jurij Smakov
2011-10-17 0:56 ` David Miller
2011-10-18 20:46 ` Jurij Smakov
2011-10-18 20:53 ` David Miller
2011-10-22 9:00 ` Jurij Smakov
2011-10-22 9:05 ` David Miller
2011-10-22 9:43 ` Jurij Smakov
2011-10-22 22:54 ` David Miller
2011-10-23 13:47 ` Jurij Smakov
2011-10-23 20:35 ` David Miller
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=20111014230653.GA14179@wooyd.org \
--to=jurij@wooyd.org \
--cc=sparclinux@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.