Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/roland/infiniband

* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/roland/infiniband: (46 commits)
  IB/uverbs: Don't serialize with ib_uverbs_idr_mutex
  IB/mthca: Make all device methods truly reentrant
  IB/mthca: Fix memory leak on modify_qp error paths
  IB/uverbs: Factor out common idr code
  IB/uverbs: Don't decrement usecnt on error paths
  IB/uverbs: Release lock on error path
  IB/cm: Use address handle helpers
  IB/sa: Add ib_init_ah_from_path()
  IB: Add ib_init_ah_from_wc()
  IB/ucm: Get rid of duplicate P_Key parameter
  IB/srp: Factor out common request reset code
  IB/srp: Support SRP rev. 10 targets
  [SCSI] srp.h: Add I/O Class values
  IB/fmr: Use device's max_map_map_per_fmr attribute in FMR pool.
  IB/mthca: Fill in max_map_per_fmr device attribute
  IB/ipath: Add client reregister event generation
  IB/mthca: Add client reregister event generation
  IB: Move struct port_info from ipath to <rdma/ib_smi.h>
  IPoIB: Handle client reregister events
  IB: Add client reregister event type
  ...

1 files changed, 4 insertions, 19 deletions

diff --git a/drivers/infiniband/hw/mthca/mthca_cmd.c b/drivers/infiniband/hw/mthca/mthca_cmd.c
index 798e13e14faf..d0f7731802c9 100644
--- a/drivers/infiniband/hw/mthca/mthca_cmd.c
+++ b/drivers/infiniband/hw/mthca/mthca_cmd.c
@@ -174,7 +174,6 @@ enum {
 
 struct mthca_cmd_context {
         struct completion done;
-        struct timer_list timer;
         int               result;
         int               next;
         u64               out_param;
@@ -362,15 +361,6 @@ void mthca_cmd_event(struct mthca_dev *dev,
         complete(&context->done);
 }
 
-static void event_timeout(unsigned long context_ptr)
-{
-        struct mthca_cmd_context *context =
-                (struct mthca_cmd_context *) context_ptr;
-
-        context->result = -EBUSY;
-        complete(&context->done);
-}
-
 static int mthca_cmd_wait(struct mthca_dev *dev,
                           u64 in_param,
                           u64 *out_param,
@@ -401,11 +391,10 @@ static int mthca_cmd_wait(struct mthca_dev *dev,
         if (err)
                 goto out;
 
-        context->timer.expires  = jiffies + timeout;
-        add_timer(&context->timer);
-
-        wait_for_completion(&context->done);
-        del_timer_sync(&context->timer);
+        if (!wait_for_completion_timeout(&context->done, timeout)) {
+                err = -EBUSY;
+                goto out;
+        }
 
         err = context->result;
         if (err)
@@ -535,10 +524,6 @@ int mthca_cmd_use_events(struct mthca_dev *dev)
         for (i = 0; i < dev->cmd.max_cmds; ++i) {
                 dev->cmd.context[i].token = i;
                 dev->cmd.context[i].next = i + 1;
-                init_timer(&dev->cmd.context[i].timer);
-                dev->cmd.context[i].timer.data     =
-                        (unsigned long) &dev->cmd.context[i];
-                dev->cmd.context[i].timer.function = event_timeout;
         }
 
         dev->cmd.context[dev->cmd.max_cmds - 1].next = -1;