diff options
| author |   Mike Marciniszyn <mike.marciniszyn@intel.com> | 2016-09-06 04:37:07 -0700 |
|---|---|---|
| committer |   Doug Ledford <dledford@redhat.com> | 2016-10-02 08:42:09 -0400 |
| commit | 30a345cc01363032826d48406dff3a6cee1cdb16 (patch) | |
| tree | 2982bdb026797706dbe433f918d50232b803c329 /drivers/infiniband/sw/rdmavt | |
| parent | IB/hfi1: Move iowait_init() to priv allocate (diff) | |
| download | linux-dev-30a345cc01363032826d48406dff3a6cee1cdb16.tar.xz linux-dev-30a345cc01363032826d48406dff3a6cee1cdb16.zip | |
IB/rdmavt: Move reset calldown to reset path
The reset calldown is misplaced.
It should only be called in the code that actually
transitions the QP to reset.
Reviewed-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/infiniband/sw/rdmavt')
| -rw-r--r-- | drivers/infiniband/sw/rdmavt/qp.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/drivers/infiniband/sw/rdmavt/qp.c b/drivers/infiniband/sw/rdmavt/qp.c index 80d2c50f96a3..0a2981559c88 100644 --- a/drivers/infiniband/sw/rdmavt/qp.c +++ b/drivers/infiniband/sw/rdmavt/qp.c @@ -530,14 +530,13 @@ static void rvt_reset_qp(struct rvt_dev_info *rdi, struct rvt_qp *qp, spin_lock(&qp->s_lock); rvt_clear_mr_refs(qp, 1); + /* + * Let the driver do any tear down it needs to for a qp + * that has been reset + */ + rdi->driver_f.notify_qp_reset(qp); } - /* - * Let the driver do any tear down it needs to for a qp - * that has been reset - */ - rdi->driver_f.notify_qp_reset(qp); - qp->remote_qpn = 0; qp->qkey = 0; qp->qp_access_flags = 0; |