On Thu, 16 Jan 2014, J. Bruce Fields wrote:
> From: "J. Bruce Fields" <bfields@xxxxxxxxxx>
>
> "disconnected" is too easily confused with "DCACHE_DISCONNECTED". I
> think "unhashed" is the more precise term here.
Good point. Applied, thanks!
sage
>
> Signed-off-by: J. Bruce Fields <bfields@xxxxxxxxxx>
> ---
> fs/ceph/caps.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> Just ran across this while wondering what d_find_alias callers do....
>
> diff --git a/fs/ceph/caps.c b/fs/ceph/caps.c
> index 3c0a4bd..697f9d7 100644
> --- a/fs/ceph/caps.c
> +++ b/fs/ceph/caps.c
> @@ -2350,11 +2350,11 @@ static void invalidate_aliases(struct inode *inode)
> d_prune_aliases(inode);
> /*
> * For non-directory inode, d_find_alias() only returns
> - * connected dentry. After calling d_invalidate(), the
> - * dentry become disconnected.
> + * hashed dentry. After calling d_invalidate(), the
> + * dentry becomes unhashed.
> *
> * For directory inode, d_find_alias() can return
> - * disconnected dentry. But directory inode should have
> + * unhashed dentry. But directory inode should have
> * one alias at most.
> */
> while ((dn = d_find_alias(inode))) {
> --
> 1.7.9.5
>
>
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
