On Fri, 07 Jan 2011 00:12:09 +1100, Nick Piggin said: > > Thanks. It actually passed in a dentry that appears to have already been > > used for something. This is not exactly a nice thing for a filesystem to do > > and probably indicates an underlying bug anyway (or at least something > > the vfs doesn't guarantee the safety of). > > > > Taking a look now. > > This patch fixes it here > config fs: avoid switching ->d_op on live dentry > Switching d_op on a live dentry is racy in general, so avoid it. In this case > it is a negative dentry, which is safer, but there are still concurrent ops > which may be called on d_op in that case (eg. d_revalidate). So in general > a filesystem may not do this. Fix configfs so as not to do this. Confirming this patch fixes the crash I was seeing, so now I'm off to try to finish bisecting my way through the other issue I was seeing...
Attachment:
pgpDPqBCPQgKJ.pgp
Description: PGP signature
