cvs commit: src/sys/net if.c

[Bjoern A. Zeeb]

On Wed, 21 Jun 2006, Max Laier wrote:

> On Wednesday 21 June 2006 10:06, Maxim Konovalov wrote:
>> On Wed, 21 Jun 2006, 06:02-0000, Gleb Smirnoff wrote:
>>> glebius     2006-06-21 06:02:35 UTC
>>>
>>>   FreeBSD src repository
>>>
>>>   Modified files:
>>>     sys/net              if.c
>>>   Log:
>>>   - First initialize ifnet, and then insert it into global
>>>     list.
>>>   - First remove from global list, then start destroying.
>>>
>>>   PR:             kern/97679
>>>   Submitted by:   Alex Lyashkov <shadow itt.net.ru>
>>>   Reviewed by:    rwatson, brooks
>>
>> FWIW, even with Yar's recent commit to if_vlan.c it still panics.  I
>> told Alex about that.  A testcase below.  Run it for several minutes.
>
> Yes, we simply keep too many implicit references to ifnets.  The real sequence
> for interface destruction should be:

I had been looking at that code somewhen last year I think
trying to clean up some ordering along the af/domain logic and
my conclusion was that in netinet6/* some bad things are done
that by that time prevented me from further working on this
to get it anywhere w/o a panic. Welcome to the club ;)

-- 
Bjoern A. Zeeb				bzeeb at Zabbadoz dot NeT