summaryrefslogtreecommitdiffstats
path: root/fs/gfs2
diff options
context:
space:
mode:
authorDenis Cheng <crquan@gmail.com>2007-07-25 17:53:58 +0800
committerSteven Whitehouse <swhiteho@redhat.com>2007-10-10 08:54:58 +0100
commitcee23c79d08c57bbbb9923703409af3b17518c58 (patch)
tree57a09aa5c4d846a1fc65e8c576b5fd9bfbe8bc4f /fs/gfs2
parent6760bdcd03a12d7d082794311ccbaf44bfc23b06 (diff)
downloadop-kernel-dev-cee23c79d08c57bbbb9923703409af3b17518c58.zip
op-kernel-dev-cee23c79d08c57bbbb9923703409af3b17518c58.tar.gz
[GFS2] use an temp variable to reduce a spin_unlock
this is more clear. Signed-off-by: Denis Cheng <crquan@gmail.com> Signed-off-by: David Teigland <teigland@redhat.com> Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2')
-rw-r--r--fs/gfs2/locking/dlm/plock.c11
1 files changed, 6 insertions, 5 deletions
diff --git a/fs/gfs2/locking/dlm/plock.c b/fs/gfs2/locking/dlm/plock.c
index fba1f1d..1f7b038 100644
--- a/fs/gfs2/locking/dlm/plock.c
+++ b/fs/gfs2/locking/dlm/plock.c
@@ -346,15 +346,16 @@ static ssize_t dev_write(struct file *file, const char __user *u, size_t count,
static unsigned int dev_poll(struct file *file, poll_table *wait)
{
+ unsigned int mask = 0;
+
poll_wait(file, &send_wq, wait);
spin_lock(&ops_lock);
- if (!list_empty(&send_list)) {
- spin_unlock(&ops_lock);
- return POLLIN | POLLRDNORM;
- }
+ if (!list_empty(&send_list))
+ mask = POLLIN | POLLRDNORM;
spin_unlock(&ops_lock);
- return 0;
+
+ return mask;
}
static const struct file_operations dev_fops = {
OpenPOWER on IntegriCloud