ceph: update ceph_dentry_info::lease_session when necessary
authorYan, Zheng <zyan@redhat.com>
Mon, 3 Jul 2017 01:09:10 +0000 (09:09 +0800)
committerIlya Dryomov <idryomov@gmail.com>
Fri, 7 Jul 2017 15:25:14 +0000 (17:25 +0200)
Current code does not update ceph_dentry_info::lease_session once
it is set. If auth mds of corresponding dentry changes, dentry lease
keeps in an invalid state.

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
Reviewed-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
fs/ceph/inode.c

index 53f23c920266d1683292f4337d616a2390aa2752..220dfd87cbfaa139d2d8f959d353bd70f032a82d 100644 (file)
@@ -1016,6 +1016,7 @@ static void update_dentry_lease(struct dentry *dentry,
        long unsigned ttl = from_time + (duration * HZ) / 1000;
        long unsigned half_ttl = from_time + (duration * HZ / 2) / 1000;
        struct inode *dir;
+       struct ceph_mds_session *old_lease_session = NULL;
 
        /*
         * Make sure dentry's inode matches tgt_vino. NULL tgt_vino means that
@@ -1051,8 +1052,10 @@ static void update_dentry_lease(struct dentry *dentry,
            time_before(ttl, di->time))
                goto out_unlock;  /* we already have a newer lease. */
 
-       if (di->lease_session && di->lease_session != session)
-               goto out_unlock;
+       if (di->lease_session && di->lease_session != session) {
+               old_lease_session = di->lease_session;
+               di->lease_session = NULL;
+       }
 
        ceph_dentry_lru_touch(dentry);
 
@@ -1065,6 +1068,8 @@ static void update_dentry_lease(struct dentry *dentry,
        di->time = ttl;
 out_unlock:
        spin_unlock(&dentry->d_lock);
+       if (old_lease_session)
+               ceph_put_mds_session(old_lease_session);
        return;
 }