summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2016-02-13 21:06:50 -0500
committerMike Marshall <hubcap@omnibond.com>2016-02-19 13:45:55 -0500
commit897c5df6cf8c10d2557c098641faa62f65ef8598 (patch)
tree6b081cc1edbf8cd5b108f868f2e944f01ee380e4
parent82d37f19ff885ece97b8a072182e39c9dc4ead7d (diff)
downloadlinux-rpi-897c5df6cf8c10d2557c098641faa62f65ef8598.tar.gz
linux-rpi-897c5df6cf8c10d2557c098641faa62f65ef8598.tar.bz2
linux-rpi-897c5df6cf8c10d2557c098641faa62f65ef8598.zip
orangefs: get rid of op->done
shouldn't be needed now Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: Mike Marshall <hubcap@omnibond.com>
-rw-r--r--fs/orangefs/devorangefs-req.c13
-rw-r--r--fs/orangefs/file.c6
-rw-r--r--fs/orangefs/orangefs-cache.c2
-rw-r--r--fs/orangefs/orangefs-kernel.h2
4 files changed, 2 insertions, 21 deletions
diff --git a/fs/orangefs/devorangefs-req.c b/fs/orangefs/devorangefs-req.c
index 790855a72e32..b27ed1cb9a36 100644
--- a/fs/orangefs/devorangefs-req.c
+++ b/fs/orangefs/devorangefs-req.c
@@ -424,19 +424,6 @@ wakeup:
* application reading/writing this device to return until
* the buffers are done being used.
*/
- if (op->downcall.type == ORANGEFS_VFS_OP_FILE_IO) {
- long n = wait_for_completion_interruptible_timeout(&op->done,
- op_timeout_secs * HZ);
- if (unlikely(n < 0)) {
- gossip_debug(GOSSIP_DEV_DEBUG,
- "%s: signal on I/O wait, aborting\n",
- __func__);
- } else if (unlikely(n == 0)) {
- gossip_debug(GOSSIP_DEV_DEBUG,
- "%s: timed out.\n",
- __func__);
- }
- }
out:
if (unlikely(op_is_cancel(op)))
put_cancel(op);
diff --git a/fs/orangefs/file.c b/fs/orangefs/file.c
index 3b1e9e83eb91..4eb009e8f19f 100644
--- a/fs/orangefs/file.c
+++ b/fs/orangefs/file.c
@@ -199,7 +199,7 @@ populate_shared_memory:
if (orangefs_cancel_op_in_progress(new_op))
return ret;
- goto done_copying;
+ goto out;
}
/*
@@ -212,7 +212,7 @@ populate_shared_memory:
iter,
new_op->downcall.resp.io.amt_complete);
if (ret < 0)
- goto done_copying;
+ goto out;
}
gossip_debug(GOSSIP_FILE_DEBUG,
"%s(%pU): Amount written as returned by the sys-io call:%d\n",
@@ -222,12 +222,10 @@ populate_shared_memory:
ret = new_op->downcall.resp.io.amt_complete;
-done_copying:
/*
* tell the device file owner waiting on I/O that this read has
* completed and it can return now.
*/
- complete(&new_op->done);
out:
if (buffer_index >= 0) {
diff --git a/fs/orangefs/orangefs-cache.c b/fs/orangefs/orangefs-cache.c
index 09194e69875f..817092a14429 100644
--- a/fs/orangefs/orangefs-cache.c
+++ b/fs/orangefs/orangefs-cache.c
@@ -122,8 +122,6 @@ struct orangefs_kernel_op_s *op_alloc(__s32 type)
atomic_set(&new_op->ref_count, 1);
- init_completion(&new_op->done);
-
new_op->upcall.type = ORANGEFS_VFS_OP_INVALID;
new_op->downcall.type = ORANGEFS_VFS_OP_INVALID;
new_op->downcall.status = -1;
diff --git a/fs/orangefs/orangefs-kernel.h b/fs/orangefs/orangefs-kernel.h
index 8613d4166d0f..1d20eadaefd8 100644
--- a/fs/orangefs/orangefs-kernel.h
+++ b/fs/orangefs/orangefs-kernel.h
@@ -205,8 +205,6 @@ struct orangefs_kernel_op_s {
struct completion waitq;
spinlock_t lock;
- struct completion done;
-
atomic_t ref_count;
/* VFS aio fields */