On 12/09/2010 01:19 AM, Chuck Lever wrote: > Hi Stefan- > > On Dec 8, 2010, at 3:35 PM, Chuck Lever wrote: > >> >> On Dec 8, 2010, at 2:03 PM, Trond Myklebust wrote: >> >>> On Wed, 2010-12-08 at 18:30 +0000, bugzilla-daemon@bugzilla.kernel.org >>> wrote: >>>> https://bugzilla.kernel.org/show_bug.cgi?id=24302 >>>> >>>> >>>> Andrew Morton changed: >>>> >>>> What |Removed |Added >>>> ---------------------------------------------------------------------------- >>>> >>>> Component|Other |NFS >>>> AssignedTo|fs_other@kernel-bugs.osdl.o |trond.myklebust@fys.uio.no |rg >>>> | >>>> >>>> >>>> >>>> >>>> --- Comment #8 from Andrew Morton >>>> 2010-12-08 18:30:32 --- OK, I reassigned it to NFS. If that was wrong >>>> then at least the NFS guys should be able to help point things in the >>>> right direction. >>> >>> >> directly, since that will lose the above Cc information> Missed to read the above, so I have to duplicate things a bit, sorry. Removed the bugzilla cc, so the report does not get spammed. >>> >>> >>> Chuck, >>> >>> Stefan appears to be hitting a panic in the nfs_umount() call from >>> nfs_walk_authlist(). Can you take a look, please? >> >> Recv'd. I'll have a look. > > Apologies in advance for the attachment. There are a few other clean ups > that can be done, but this seems to be the minimal fix. Please try this and > let us know if it addresses your panic. > > > > > > > The faulty commit went in a while back, so this patch is probably appropriate > for stable kernels (back to 2.6.31, where this was likely introduced). > I can confirm that this was the root cause of the crash. The sha referenced as the causing commit, got in with a 2.6.32-rc so I added the stable info accordingly. I also added my tested-by and slightly modified the bug reference format (Chuck, I hope this is ok). -Stefan