diff options
| author |   Mike Marciniszyn <mike.marciniszyn@intel.com> | 2016-05-12 10:22:57 -0700 |
|---|---|---|
| committer |   Doug Ledford <dledford@redhat.com> | 2016-05-13 19:39:15 -0400 |
| commit | 17f15bf66884b9e33c5005ee0149b039af8f7af2 (patch) | |
| tree | 77308e38ae45f9b5313a4404e132cd36bbdab19f /drivers/staging/rdma/hfi1/verbs.c | |
| parent | IB/hfi1: Remove no-op QSFP reset code (diff) | |
| download | linux-dev-17f15bf66884b9e33c5005ee0149b039af8f7af2.tar.xz linux-dev-17f15bf66884b9e33c5005ee0149b039af8f7af2.zip | |
IB/hfi1: Fix pio wait counter double increment
The code unconditionlly increments the pio wait counter
making the counter inacurate and unusable.
Fixes: 14553ca11039 ("staging/rdma/hfi1: Adaptive PIO for short messages")
Reviewed-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/staging/rdma/hfi1/verbs.c')
| -rw-r--r-- | drivers/staging/rdma/hfi1/verbs.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/staging/rdma/hfi1/verbs.c b/drivers/staging/rdma/hfi1/verbs.c index 9cdc85fa366f..f3531fdfef01 100644 --- a/drivers/staging/rdma/hfi1/verbs.c +++ b/drivers/staging/rdma/hfi1/verbs.c @@ -946,7 +946,6 @@ static int pio_wait(struct rvt_qp *qp, dev->n_piowait += !!(flag & RVT_S_WAIT_PIO); dev->n_piodrain += !!(flag & RVT_S_WAIT_PIO_DRAIN); - dev->n_piowait++; qp->s_flags |= flag; was_empty = list_empty(&sc->piowait); list_add_tail(&priv->s_iowait.list, &sc->piowait); |