From: Neil Brown <neilb@suse.de>
To: Paul Collins <paul@burly.ondioline.org>
Cc: "J. Bruce Fields" <bfields@fieldses.org>,
linuxppc-dev@ozlabs.org, nfsv4@linux-nfs.org,
linux-kernel@vger.kernel.org
Subject: Re: nfsd, v4: oops in find_acceptable_alias, ppc32 Linux, post-2.6.27-rc1
Date: Mon, 4 Aug 2008 14:08:07 +1000 [thread overview]
Message-ID: <18582.32935.501672.689845@notabene.brown> (raw)
In-Reply-To: message from Paul Collins on Monday August 4
On Monday August 4, paul@burly.ondioline.org wrote:
> Neil Brown <neilb@suse.de> writes:
> >
> > What filesystem is being exported here?
>
> Boring old ext3 (on LVM, on dm-crypt).
Good. That makes it easier.
>
> > Can you get an assembly version of exportfs_decode_fh, so we can check
> > what is happening at 0xa8 (and 0x4c).
>
Thanks.
bctrl appears to be the indirect-function-call opcode. There are
three of them one each for
->fh_to_dentry
acceptable
->fh_to_parent
0xa8 is 'acceptable'.
In the first traceback, the crash was a call from very early in
find_acceptable_alias, The first significant thing it does is call
the 'acceptable' function.
So it seems clear that 'acceptable' is NULL.
It is equally clear that we never ever set it to NULL in the code.
The logical conclusion is "compiler error".
We can confirm (hopefully) by looking at a disassembly of fh_verify.
Maybe because nfsd_acceptable is 'static' and never explicitly called,
gcc gets confused and optimises it away. Maybe a disassembly of
nfsd_acceptable would be informative ... particularly if it turns out
to be empty.
Could you try removing the 'static' declaration for nfsd_acceptable
and recompile?
Or maybe try a different compiler?
Thanks,
NeilBrown
next prev parent reply other threads:[~2008-08-04 4:08 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-02 12:03 nfsd, v4: oops in find_acceptable_alias, ppc32 Linux, post-2.6.27-rc1 Paul Collins
2008-08-02 18:45 ` J. Bruce Fields
2008-08-02 22:36 ` Paul Collins
2008-08-03 6:47 ` Paul Collins
2008-08-03 12:09 ` Neil Brown
2008-08-03 12:25 ` Paul Collins
2008-08-04 4:08 ` Neil Brown [this message]
2008-08-04 5:11 ` Paul Collins
2008-08-04 10:00 ` Paul Collins
2008-08-04 14:36 ` Michael Ellerman
2008-08-04 20:51 ` Paul Collins
2008-08-04 20:59 ` J. Bruce Fields
2008-08-05 0:16 ` Michael Ellerman
2008-08-05 3:43 ` Paul Collins
2008-08-05 4:34 ` Michael Ellerman
2008-08-05 4:47 ` Paul Collins
2008-08-05 7:16 ` Benjamin Herrenschmidt
2008-08-06 6:29 ` Benjamin Herrenschmidt
2008-08-25 20:12 ` Bill Davidsen
2008-08-05 9:43 ` Paul Collins
2008-08-05 11:53 ` Michael Ellerman
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=18582.32935.501672.689845@notabene.brown \
--to=neilb@suse.de \
--cc=bfields@fieldses.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=nfsv4@linux-nfs.org \
--cc=paul@burly.ondioline.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;
as well as URLs for NNTP newsgroup(s).