[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Mon, 1 Jan 2018 08:38:33 +0200
From: Leon Romanovsky <leonro@...lanox.com>
To: Jason Gunthorpe <jgg@...pe.ca>
Cc: Denis Drozdov <denisd@...lanox.com>, davem@...emloft.net,
dledford@...hat.com, linux-rdma@...r.kernel.org,
netdev@...r.kernel.org
Subject: Re: [PATCH net 2/2] IB/ipoib: Fix netlink support in IPoIB
On Sun, Dec 31, 2017 at 01:23:08PM -0700, Jason Gunthorpe wrote:
> On Sun, Dec 31, 2017 at 01:16:23PM +0200, Denis Drozdov wrote:
> > IPoIB netlink support was broken by commit cd565b4b51e5
> > ("IB/IPoIB: Support acceleration options callbacks"),
> > that added flow which allocates netdev rdma structures
> > after netlink object is already created. Such situation leads
> > to crash in __ipoib_device_add, once trying to reuse netlink
> > device.
> > This commit restores the netlink support.
> >
> > Fixes: cd565b4b51e5 ("IB/IPoIB: Support acceleration options callbacks")
> > Reviewed-by: Erez Shitrit <erezsh@...lanox.com>
> > Reviewed-by: Leon Romanovsky <leonro@...lanox.com>
> > Reviewed-by: Saeed Mahameed <saeedm@...lanox.com>
> > Signed-off-by: Denis Drozdov <denisd@...lanox.com>
> > drivers/infiniband/ulp/ipoib/ipoib.h | 2 ++
> > drivers/infiniband/ulp/ipoib/ipoib_main.c | 23 +++++++++--------
> > drivers/infiniband/ulp/ipoib/ipoib_netlink.c | 38 +++++++++++++++++++++++++---
> > drivers/infiniband/ulp/ipoib/ipoib_vlan.c | 20 ++++-----------
> > 4 files changed, 54 insertions(+), 29 deletions(-)
>
> I assume this is going through DaveM's tree?
Yes, it was our plan because of the first patch in this series.
> In which case for the drivers/infiniband part:
>
> Acked-by: Jason Gunthorpe <jgg@...lanox.com>
>
> > @@ -131,11 +157,15 @@ static int ipoib_new_child_link(struct net *src_net, struct net_device *dev,
> > */
> > child_pkey |= 0x8000;
> >
> > - err = __ipoib_vlan_add(ppriv, ipoib_priv(dev),
> > - child_pkey, IPOIB_RTNL_CHILD);
> > + down_write(&ppriv->vlan_rwsem);
> > +
> > + priv = ipoib_priv(dev);
> > + err = __ipoib_vlan_add(ppriv, priv, child_pkey, IPOIB_RTNL_CHILD);
> > + up_write(&ppriv->vlan_rwsem);
>
> This looks like a necessary and correct change, but does it need to be
> a seperate bug fix commit?
It can.
>
> Jason
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)
Powered by blists - more mailing lists