Hi, > Attached below is my take on how to address this problem. > This addresses any concerns you may have had about checking > po->pppoe_dev==NULL, > because accesses to this field are now synchronized with pppoe_hash_lock.
That indeed looks like a much cleaner solution, so I'd certainly prefer it over mine. Testing so far didn't turn up any problems, either. > Once we can settle on a fix for this, I'll deal with the SID==0 issue > (trying to do that now would just cause patch conflicts). Well, yeah, this stuff probably is more important than support for SID 0 :-) Florian - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html