diff options
| author | Julia Lawall <Julia.Lawall@lip6.fr> | 2018-12-23 03:57:01 -0500 |
|---|---|---|
| committer | Jason Gunthorpe <jgg@mellanox.com> | 2019-01-02 17:42:43 -0500 |
| commit | 2fb458953a892020f36a323f386d8c7009f10777 (patch) | |
| tree | dd2e4b3a22ebb542355a988adca3521b7e4568d6 | |
| parent | 2f1927b090345d7e65e6e873baba1599b18a0e26 (diff) | |
IB/ipoib: drop useless LIST_HEAD
Drop LIST_HEAD where the variable it declares is never used.
Commit 31c02e215700 ("IPoIB: Avoid using stale last_send counter
when reaping AHs") removed the uses, but not the declaration.
The semantic patch that fixes this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@@
identifier x;
@@
- LIST_HEAD(x);
... when != x
// </smpl>
Fixes: 31c02e215700 ("IPoIB: Avoid using stale last_send counter when reaping AHs")
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
| -rw-r--r-- | drivers/infiniband/ulp/ipoib/ipoib_ib.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_ib.c b/drivers/infiniband/ulp/ipoib/ipoib_ib.c index 6d35570092d6..78fa777c87b1 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_ib.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_ib.c | |||
| @@ -669,7 +669,6 @@ static void __ipoib_reap_ah(struct net_device *dev) | |||
| 669 | { | 669 | { |
| 670 | struct ipoib_dev_priv *priv = ipoib_priv(dev); | 670 | struct ipoib_dev_priv *priv = ipoib_priv(dev); |
| 671 | struct ipoib_ah *ah, *tah; | 671 | struct ipoib_ah *ah, *tah; |
| 672 | LIST_HEAD(remove_list); | ||
| 673 | unsigned long flags; | 672 | unsigned long flags; |
| 674 | 673 | ||
| 675 | netif_tx_lock_bh(dev); | 674 | netif_tx_lock_bh(dev); |
