On 03/25/2016 11:10 PM, Andrew Collins wrote: > On 03/25/2016 02:43 PM, Matthias Schiffer wrote: >> We've tried your patch, and it changes the symptoms a bit, but doesn't fix >> the panic. I've attached kernel logs of the crash both before and after >> applying the patch. >> >> One note: I did not reproduce this issue myself, it was first reported in >> [1], and then forwarded to the batman-adv issue tracker [2] by me. >> >> Regards, >> Matthias >> >> >> [1] https://github.com/freifunk-gluon/gluon/issues/680 >> [2] https://www.open-mesh.org/issues/247 > > On the off chance it helps, the version of the patch I integrated locally > takes a somewhat different approach > than the one I sent to the mailing list (propagates adj_list refcnts). > I've attached it in case it's useful. > > I haven't submitted this upstream yet as it's still rather ugly. I'm of > the opinion that the whole "every device > knows every upperdev and lowerdev in its tree" model is rather broken, and > the patch is just working around > a design that needs some rework. > > Thanks, > Andrew Collins
It's ugly, but it seems to help. No crashes so far with the new version of your patch. Thanks, Matthias
signature.asc
Description: OpenPGP digital signature
