On Mon, 2015-01-26 at 12:27 +0200, Erez Shitrit wrote: > New (and full) dmesg attached, (after modprobe ib_ipoib, with all debug > flags set) it is all there. Thank you, I know what's going on here now. Will correct shortly. > >> The main cause is the concept that was broken for the send-only join, > >> when you treat the sendonly like a regular mcg and add it to the mc list > >> and to the mc_task etc. > > I'm looking at 3.18 right now, and 3.18 adds sendonly groups to the mcg > > just like my current code does. The only difference, and I do mean > > *only*, is that it calls sendonly_join directly instead of via the > > mcast_task. > > Yes, and i already wrote that it is more than just "only", it changed > the concept of the sendonly mc packet. Be more specific please. What do you mean by "concept"? And just so we are clear, this all started because the existing multicast code was super easy to break and was racy, so if the "concept" you are referring to is what made the original code easy to break and racy, I'm not going to care one whit that I changed that concept. -- Doug Ledford <dledford@xxxxxxxxxx> GPG KeyID: 0E572FDD
Attachment:
signature.asc
Description: This is a digitally signed message part