On Sat, Jul 30, 2011 at 12:32:04PM -0400, Ted Ts'o wrote: > spin_lock(&inode->i_lock); > if (!list_empty(&inode->i_dentry)) { > dentry = list_first_entry(&inode->i_dentry, > struct dentry, d_alias); > dget(dentry); > } > spin_unlock(&inode->i_lock); dentry = d_find_alias(inode); actually... > if (!dentry) > break; > next = igrab(dentry->d_parent->d_inode); and that one needs dentry->d_lock around it, to stabilize ->d_parent. -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html