From owner-freebsd-net@FreeBSD.ORG  Tue Oct 23 07:17:20 2012
Return-Path: <owner-freebsd-net@FreeBSD.ORG>
Delivered-To: freebsd-net@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
 by hub.freebsd.org (Postfix) with ESMTP id 1EA18D4C;
 Tue, 23 Oct 2012 07:17:20 +0000 (UTC) (envelope-from zec@fer.hr)
Received: from mail.zvne.fer.hr (mail.zvne.fer.hr [161.53.66.5])
 by mx1.freebsd.org (Postfix) with ESMTP id 8ECCF8FC08;
 Tue, 23 Oct 2012 07:17:19 +0000 (UTC)
Received: from munja.zvne.fer.hr (161.53.66.248) by mail.zvne.fer.hr
 (161.53.66.5) with Microsoft SMTP Server id 14.2.298.4; Tue, 23 Oct 2012
 09:17:17 +0200
Received: from sluga.fer.hr ([161.53.66.244]) by munja.zvne.fer.hr with
 Microsoft SMTPSVC(6.0.3790.4675);	 Tue, 23 Oct 2012 09:17:16 +0200
Received: from localhost ([161.53.19.8]) by sluga.fer.hr over TLS secured
 channel with Microsoft SMTPSVC(6.0.3790.4675);	 Tue, 23 Oct 2012 09:17:16
 +0200
From: Marko Zec <zec@fer.hr>
To: Adrian Chadd <adrian@freebsd.org>
Subject: Re: VIMAGE crashes on 9.x with hotplug net80211 devices
Date: Tue, 23 Oct 2012 09:16:11 +0200
User-Agent: KMail/1.9.10
References: <CAJ-VmomchJZ7GUKrAjmmyBXDw-6H6O5fAxT_tfAFfhU=HknG1g@mail.gmail.com>
 <201210222317.00457.zec@fer.hr>
 <CAJ-Vmomg_pSZ=ZdGNf9FB7Tx_K4DHMsCQsBuFVyojEHr0r1_Ew@mail.gmail.com>
In-Reply-To: <CAJ-Vmomg_pSZ=ZdGNf9FB7Tx_K4DHMsCQsBuFVyojEHr0r1_Ew@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
Message-ID: <201210230916.11513.zec@fer.hr>
X-OriginalArrivalTime: 23 Oct 2012 07:17:16.0521 (UTC)
 FILETIME=[6E70E590:01CDB0EE]
Cc: freebsd-net@freebsd.org, freebsd-hackers@freebsd.org
X-BeenThere: freebsd-net@freebsd.org
X-Mailman-Version: 2.1.14
Precedence: list
List-Id: Networking and TCP/IP with FreeBSD <freebsd-net.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-net>,
 <mailto:freebsd-net-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-net>
List-Post: <mailto:freebsd-net@freebsd.org>
List-Help: <mailto:freebsd-net-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-net>,
 <mailto:freebsd-net-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 23 Oct 2012 07:17:20 -0000

On Monday 22 October 2012 23:43:11 Adrian Chadd wrote:
> Hi,
>
> I don't mind tackling the net80211 clone detach path.
>
> I do mind how the default for hotplug is "argh, it doesn't work." :-)
>
> So I'd like to come up with something to fix the basic device detach,
> rather than having to actually add CURVNET_*() calls around each
> if_free() in each device detach method.

As already mentioned earlier, I don't terribly object if you'd place 
CURVNET_SET(ifp->if_vnet) inside if_free() and a limited number of similar 
functions, but I don't quite believe this is will enough to solve the 
device_detach() issue without having to touch any of the drivers...

Marko