On Mon, Nov 15, 2010 at 10:09:06PM +0100, Christoph Hellwig wrote: > On Mon, Nov 15, 2010 at 04:11:20PM +1100, Nick Piggin wrote: > > This patch is totally broken. You can't just dget() a dentry with > > nothing but RCU critical section open. > > The plain dget is indeed wrong as we should at least take d_lock > and check d_count for zero before incrementing it to protect > against shrink_dentry_list. > > I'm not quite sure it really matters as d_validate already has Explain why it doesn't matter. It's an oopsable bug introduced. > and always ad much worse bugs, such as the complete lack of > protection against renames. What are the much worse bugs? What do you mean by rename protection? > Anyway, I'll send a patch to Linus to fix this issue for now. A revert is appropriate. Like I said, there is no need for this patch at all and no justification provided. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html