From: Mikael Pettersson <mikpe@it.uu.se>
To: Mikael Pettersson <mikpe@it.uu.se>
Cc: Al Viro <viro@ZenIV.linux.org.uk>,
Geert Uytterhoeven <geert@linux-m68k.org>,
linux-m68k@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: aranym bug, manifests as "ida_remove called for id=13" on recent kernels
Date: Mon, 11 Oct 2010 13:50:42 +0200 [thread overview]
Message-ID: <19634.64018.979987.971553@pilspetsen.it.uu.se> (raw)
In-Reply-To: <19634.55413.877788.492484@pilspetsen.it.uu.se>
Mikael Pettersson writes:
> Al Viro writes:
> > On Mon, Oct 11, 2010 at 12:52:56AM +0100, Al Viro wrote:
> > > On Sun, Oct 10, 2010 at 10:18:03PM +0200, Geert Uytterhoeven wrote:
> > > > >> This is on aranym-0.9.9-1 from Ubuntu/amd64.
> > > > >
> > > > > stat -L /proc/2/exec, otherwise you'll hit lstat() instead of stat().
> > > > > And FWIW 0.9.10-1 squeeze/amd64 also triggers here...
> > > >
> > > > Still, just "stat: cannot stat `proc/2/exe': No such file or directory" here...
> > >
> > > Interesting... Which gcc version is used?
> >
> > Argh... OK, going through aranym with debugger has exhonorated it. My
> > apologies ;-/ It *is* gcc in sid. Testcase:
> >
> > extern int foo(int);
> > void *bar(int n)
> > {
> > return (void *)foo(n);
> > }
> >
> > and gcc -S -O2 turns that into
> > bar:
> > link.w %fp,#0
> > unlk %fp
> > jra foo
> >
> > Spot the obvious bug... BTW, why on the Earth does debian-ports m68k tree
> > use gcc-4.3 with Cthulhu-scaring 700Kb gzipped patch and does *not* have
> > gcc-4.4?
>
> I can confirm that the bug exists in gcc-4.3.4 and gcc-4.4.5,
> but it has been fixed in gcc-4.5.1 which generates:
>
> bar:
> link.w %fp,#0
> move.l 8(%fp),-(%sp)
> jsr foo
> move.l %d0,%a0
> unlk %fp
> rts
>
> I don't yet know the gcc PR number or svn commit # for the fix
> (in case people want a backport).
It's gcc PR41302 which was fixed for gcc trunk on November 4 2009
in r153890. The patch backports easily to gcc-4.4 and solves the
test case there (manual inspection using a cross). It also backports
easily to gcc-4.3 but I haven't tested it there.
next prev parent reply other threads:[~2010-10-11 11:50 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-07 17:49 aranym bug, manifests as "ida_remove called for id=13" on recent kernels Al Viro
2010-10-10 9:47 ` Geert Uytterhoeven
2010-10-10 14:49 ` Al Viro
2010-10-10 20:18 ` Geert Uytterhoeven
2010-10-10 23:52 ` Al Viro
2010-10-11 2:41 ` Al Viro
2010-10-11 4:36 ` Brad Boyer
2010-10-11 4:48 ` Al Viro
2010-10-11 12:21 ` Thorsten Glaser
2010-10-11 13:10 ` Andreas Schwab
2010-10-11 13:35 ` Al Viro
2010-10-11 13:45 ` Thorsten Glaser
2010-10-11 14:15 ` Andreas Schwab
2010-10-11 14:24 ` Al Viro
2010-10-11 22:02 ` Al Viro
2010-11-02 15:30 ` Geert Uytterhoeven
2010-10-11 19:05 ` Mikael Pettersson
2010-10-11 9:27 ` Mikael Pettersson
2010-10-11 11:50 ` Mikael Pettersson [this message]
2010-10-11 12:29 ` Andreas Schwab
2010-10-11 8:39 ` Geert Uytterhoeven
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=19634.64018.979987.971553@pilspetsen.it.uu.se \
--to=mikpe@it.uu.se \
--cc=geert@linux-m68k.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-m68k@vger.kernel.org \
--cc=viro@ZenIV.linux.org.uk \
/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