ceph: wake up writer if vmtruncate work get blocked
authorYan, Zheng <zheng.z.yan@intel.com>
Sun, 21 Jul 2013 12:25:26 +0000 (20:25 +0800)
committerSage Weil <sage@inktank.com>
Sat, 10 Aug 2013 00:54:33 +0000 (17:54 -0700)
To write data, the writer first acquires the i_mutex, then try getting
caps. The writer may sleep while holding the i_mutex. If the MDS revokes
Fb cap in this case, vmtruncate work can't do its job because i_mutex
is locked. We should wake up the writer and let it truncate the pages.

Signed-off-by: Yan, Zheng <zheng.z.yan@intel.com>
Reviewed-by: Sage Weil <sage@inktank.com>
fs/ceph/inode.c

index 4906ada4a97c6be9f00ce45245911dbb6efa98c2..55aaddb4047ea9ca01dc9d0ea203e562e58dccd8 100644 (file)
@@ -1465,7 +1465,14 @@ static void ceph_vmtruncate_work(struct work_struct *work)
        struct inode *inode = &ci->vfs_inode;
 
        dout("vmtruncate_work %p\n", inode);
-       mutex_lock(&inode->i_mutex);
+       if (!mutex_trylock(&inode->i_mutex)) {
+               /*
+                * the i_mutex can be hold by a writer who is waiting for
+                * caps. wake up waiters, they will do pending vmtruncate.
+                */
+               wake_up_all(&ci->i_cap_wq);
+               mutex_lock(&inode->i_mutex);
+       }
        __ceph_do_pending_vmtruncate(inode);
        mutex_unlock(&inode->i_mutex);
        iput(inode);