[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <46AF3CA8.6050201@gmail.com>
Date: Tue, 31 Jul 2007 16:44:08 +0300
From: Moni Shoua <monisonlists@...il.com>
To: Roland Dreier <rdreier@...co.com>
CC: Moni Shoua <monis@...taire.com>, netdev@...r.kernel.org,
fubar@...ibm.com, davem@...emloft.net,
general@...ts.openfabrics.org
Subject: Re: [ofa-general] Re: [PATCH V3 0/7] net/bonding: ADD IPoIB support
for the bonding driver
Roland Dreier wrote:
> > 1. When bonding enslaves an IPoIB device the bonding neighbor holds a
> > reference to a cleanup function in the IPoIB drives. This makes it unsafe to
> > unload the IPoIB module if there are bonding neighbors in the air. So, to
> > avoid this race one must unload bonding before unloading IPoIB.
>
> I think we really want to resolve this somehow. Getting an oops by
> doing "modprobe -r ipoib" isn't that friendly.
>
You are right and we want to resolve that.
One way is to clean the neigh destructor function from all IPoIB neighs.
The other way is to prevent ipoib unload if device is a slave or is referenced from
somewhere else.
I guess I would like an advice here.
> Also, what happened to the problem of having an address handle
> belonging to the wrong device on bond failover? Did you figure out a
> way to fix that one?
This is what patch 2 handles.
>
> - R.
> _______________________________________________
> general mailing list
> general@...ts.openfabrics.org
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general
>
> To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
>
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists