Skip to content

Commit 9eae554

Browse files
author
Roland Dreier
committed
IPoIB: Get rid of ipoib_mcast_detach() wrapper
ipoib_mcast_detach() does nothing except call ib_detach_mcast(), so just use the core API in the one place that does a multicast group detach. add/remove: 0/1 grow/shrink: 0/1 up/down: 0/-105 (-105) function old new delta ipoib_mcast_leave 357 319 -38 ipoib_mcast_detach 67 - -67 Signed-off-by: Roland Dreier <[email protected]>
1 parent d0de136 commit 9eae554

File tree

3 files changed

+3
-17
lines changed

3 files changed

+3
-17
lines changed

drivers/infiniband/ulp/ipoib/ipoib.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -486,8 +486,6 @@ void ipoib_path_iter_read(struct ipoib_path_iter *iter,
486486

487487
int ipoib_mcast_attach(struct net_device *dev, u16 mlid,
488488
union ib_gid *mgid, int set_qkey);
489-
int ipoib_mcast_detach(struct net_device *dev, u16 mlid,
490-
union ib_gid *mgid);
491489

492490
int ipoib_init_qp(struct net_device *dev);
493491
int ipoib_transport_dev_init(struct net_device *dev, struct ib_device *ca);

drivers/infiniband/ulp/ipoib/ipoib_multicast.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -627,10 +627,10 @@ static int ipoib_mcast_leave(struct net_device *dev, struct ipoib_mcast *mcast)
627627
IPOIB_GID_ARG(mcast->mcmember.mgid));
628628

629629
/* Remove ourselves from the multicast group */
630-
ret = ipoib_mcast_detach(dev, be16_to_cpu(mcast->mcmember.mlid),
631-
&mcast->mcmember.mgid);
630+
ret = ib_detach_mcast(priv->qp, &mcast->mcmember.mgid,
631+
be16_to_cpu(mcast->mcmember.mlid));
632632
if (ret)
633-
ipoib_warn(priv, "ipoib_mcast_detach failed (result = %d)\n", ret);
633+
ipoib_warn(priv, "ib_detach_mcast failed (result = %d)\n", ret);
634634
}
635635

636636
return 0;

drivers/infiniband/ulp/ipoib/ipoib_verbs.c

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -72,18 +72,6 @@ int ipoib_mcast_attach(struct net_device *dev, u16 mlid, union ib_gid *mgid, int
7272
return ret;
7373
}
7474

75-
int ipoib_mcast_detach(struct net_device *dev, u16 mlid, union ib_gid *mgid)
76-
{
77-
struct ipoib_dev_priv *priv = netdev_priv(dev);
78-
int ret;
79-
80-
ret = ib_detach_mcast(priv->qp, mgid, mlid);
81-
if (ret)
82-
ipoib_warn(priv, "ib_detach_mcast failed (result = %d)\n", ret);
83-
84-
return ret;
85-
}
86-
8775
int ipoib_init_qp(struct net_device *dev)
8876
{
8977
struct ipoib_dev_priv *priv = netdev_priv(dev);

0 commit comments

Comments
 (0)