Date: Wed, 21 Jun 2006 10:16:30 +0000 (UTC) From: "Bjoern A. Zeeb" <bzeeb-lists@lists.zabbadoz.net> To: Max Laier <max@love2party.net> Cc: cvs-src@freebsd.org, Maxim Konovalov <maxim@macomnet.ru>, Gleb Smirnoff <glebius@freebsd.org>, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/net if.c Message-ID: <20060621100956.I60374@maildrop.int.zabbadoz.net> In-Reply-To: <200606211113.31621.max@love2party.net> References: <200606210602.k5L62Zor029243@repoman.freebsd.org> <20060621120212.D93005@mp2.macomnet.net> <200606211113.31621.max@love2party.net>
next in thread | previous in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060621100956.I60374>