[prev in list] [next in list] [prev in thread] [next in thread] 

List:       v9fs-developer
Subject:    [V9fs-developer] [patch] fs/9p: check
From:       Jim Garlick <garlick () llnl ! gov>
Date:       2011-12-16 0:48:49
Message-ID: 20111216004849.GA31975 () llnl ! gov
[Download RAW message or body]

In v9fs_file_do_lock () we need to check return value of
schedule_timeout_interruptible () and exit the loop when it returns
nonzero, otherwise the loop is not really interruptible and
after the signal, the loop is no longer throttled by P9_LOCK_TIMEOUT.

Signed-off-by: Jim Garlick <garlick.jim@gmail.com>

9fccee084cdb5934 1f0f2fbdf1fcb5d441803e44e7f6a4fb5d800a92
diff --git a/v9fs/vfs_file.c b/v9fs/vfs_file.c
index 80bddcf..e4e3997 100644
--- a/v9fs/vfs_file.c
+++ b/v9fs/vfs_file.c
@@ -178,7 +178,8 @@ static int v9fs_file_do_lock(struct file *filp, int cmd, str
                        break;
                if (status == P9_LOCK_BLOCKED && !IS_SETLKW(cmd))
                        break;
-               schedule_timeout_interruptible(P9_LOCK_TIMEOUT);
+               if (schedule_timeout_interruptible(P9_LOCK_TIMEOUT) != 0)
+                       break;
        }
 
        /* map 9p status to VFS status */


------------------------------------------------------------------------------
Learn Windows Azure Live!  Tuesday, Dec 13, 2011
Microsoft is holding a special Learn Windows Azure training event for 
developers. It will provide a great way to learn Windows Azure and what it 
provides. You can attend the event by watching it streamed LIVE online.  
Learn more at http://p.sf.net/sfu/ms-windowsazure
_______________________________________________
V9fs-developer mailing list
V9fs-developer@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/v9fs-developer
[prev in list] [next in list] [prev in thread] [next in thread]