diff options
| author | Roland Dreier <roland@purestorage.com> | 2015-07-28 00:43:23 +0300 |
|---|---|---|
| committer | Doug Ledford <dledford@redhat.com> | 2015-08-29 05:54:47 +0300 |
| commit | d6c7276be180fbd0a30cda8fac5c82d483be47b6 (patch) | |
| tree | 7eebccae134bf571ce53f8b9911e6d3fbc01cf15 /drivers/infiniband/hw/mlx5 | |
| parent | d6f1c17e162b2a11e708f28fa93f2f79c164b442 (diff) | |
| download | linux-d6c7276be180fbd0a30cda8fac5c82d483be47b6.tar.xz | |
IB/mlx5: Remove dead code from alloc_cached_mr()
The only place that assigns mr inside the loop already does a break.
So "if (mr)" will never be true here since the function initializes mr
to NULL at the top. We can just drop the extra if and break here.
Signed-off-by: Roland Dreier <roland@purestorage.com>
Acked-by: Eli Cohen <eli@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/infiniband/hw/mlx5')
| -rw-r--r-- | drivers/infiniband/hw/mlx5/mr.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/infiniband/hw/mlx5/mr.c b/drivers/infiniband/hw/mlx5/mr.c index bc9a0de897cb..10d2b213a4d8 100644 --- a/drivers/infiniband/hw/mlx5/mr.c +++ b/drivers/infiniband/hw/mlx5/mr.c @@ -441,9 +441,6 @@ static struct mlx5_ib_mr *alloc_cached_mr(struct mlx5_ib_dev *dev, int order) spin_unlock_irq(&ent->lock); queue_work(cache->wq, &ent->work); - - if (mr) - break; } if (!mr) |