On Thu, 03 Dec 2009, Eric W. Biederman wrote:
> Ah. I thought you were thinking about the mandatory have_submounts()
> check in dentry->d_op->d_revalidate().
>
> I expect the generic d_invalidate will simply hit the:
> spin_lock(&dcache_lock);
> if (d_unhashed(dentry)) {
> spin_unlock(&dcache_lock);
> return 0;
> }
>
> After the distributed filesystem has called d_drop in
> dentry->d_op->d_revalidate (when appropriate.
Ah, right, NFS's d_revalidate does do d_drop(). Okay, I have no
problem with the patch, as long as path lookup performance doesn't
show a regression.
Thanks,
Miklos
--
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
