diff options
author | Roland Dreier <roland@purestorage.com> | 2015-07-27 21:43:23 (GMT) |
---|---|---|
committer | Doug Ledford <dledford@redhat.com> | 2015-08-29 02:54:47 (GMT) |
commit | d6c7276be180fbd0a30cda8fac5c82d483be47b6 (patch) | |
tree | 7eebccae134bf571ce53f8b9911e6d3fbc01cf15 | |
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>
-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 bc9a0de..10d2b21 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) |