On Tue, Dec 06, 2016 at 12:45:11AM -0500, Oleg Drokin wrote: > Well, certainly if d_splice_alias was working like that so that even non-directory > dentry would find an alias (not necessarily unhashed even) for that same inode and use that instead, that would make ll_splice_alias/ll_find_alias unnecessary. > > We still retain the weird d_compare() that rejects otherwise perfectly valid aliases > if the lock guarding them is gone, triggering relookup (and necessiating the > above logic to pick up just rejected alias again now that we have the lock again). Why not have ->d_revalidate() kick them, instead? _IF_ we have a way to do unhash-and-trigger-lookup that way, do you really need those games with ->d_compare()? -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html