diff options
| author | Christoph Lameter <cl@linux.com> | 2015-10-11 18:49:42 -0500 |
|---|---|---|
| committer | Doug Ledford <dledford@redhat.com> | 2015-10-13 16:43:59 -0400 |
| commit | 0b5c9279e568d90903acedc2b9b832d8d78e8288 (patch) | |
| tree | 6dca079bc1801a09f1ac2d5246a62051ad77228a /drivers/infiniband/ulp/ipoib/ipoib_main.c | |
| parent | b8cab5dab15ff5c2acc3faefdde28919b0341c11 (diff) | |
| download | op-kernel-dev-0b5c9279e568d90903acedc2b9b832d8d78e8288.zip op-kernel-dev-0b5c9279e568d90903acedc2b9b832d8d78e8288.tar.gz | |
IB/ipoib: For sendonly join free the multicast group on leave
When we leave the multicast group on expiration of a neighbor we
do not free the mcast structure. This results in a memory leak
that causes ib_dealloc_pd to fail and print a WARN_ON message
and backtrace.
Fixes: bd99b2e05c4d (IB/ipoib: Expire sendonly multicast joins)
Signed-off-by: Christoph Lameter <cl@linux.com>
Tested-by: Sagi Grimberg <sagig@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/infiniband/ulp/ipoib/ipoib_main.c')
| -rw-r--r-- | drivers/infiniband/ulp/ipoib/ipoib_main.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c index f74316e..babba05 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c @@ -1207,8 +1207,10 @@ static void __ipoib_reap_neigh(struct ipoib_dev_priv *priv) out_unlock: spin_unlock_irqrestore(&priv->lock, flags); - list_for_each_entry_safe(mcast, tmcast, &remove_list, list) + list_for_each_entry_safe(mcast, tmcast, &remove_list, list) { ipoib_mcast_leave(dev, mcast); + ipoib_mcast_free(mcast); + } } static void ipoib_reap_neigh(struct work_struct *work) |
