Date: Thu, 25 Jul 2013 02:36:46 -0700 From: Craig Rodrigues <rodrigc@FreeBSD.org> To: Marius Strobl <marius@alchemy.franken.de> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r253346 - in head/sys: kern net netgraph netgraph/bluetooth/socket Message-ID: <CAG=rPVfiGGHyMY7DtC0WQCguvtvFrtLVQsGBLMh=SZ42YUwpkQ@mail.gmail.com> In-Reply-To: <20130725080758.GE948@alchemy.franken.de> References: <201307150132.r6F1WttU081255@svn.freebsd.org> <20130725080758.GE948@alchemy.franken.de>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Jul 25, 2013 at 1:07 AM, Marius Strobl <marius@alchemy.franken.de>wrote: > > Uhm - do we really need to have that layering violation in subr_bus.c? > Wouldn't it be sufficient to set curthread->td_vnet to vnet0 in if_alloc(9) > or if_attach(9) at least instead? > There was some discussion about this involving Marko Zec, Adrian Chadd, and myself starting in this thread: http://lists.freebsd.org/pipermail/svn-src-all/2013-July/071798.html Adrian and Marko converged on similar patches: http://lists.freebsd.org/pipermail/freebsd-hackers/2012-November/041120.html http://people.freebsd.org/~adrian/ath/20130712-vimage-default-attach-detach.diff As Marko mentioned in another e-mail on this thread, the patch as it is necessary in order to fix VIMAGE related kernel panics in many different scenarios, including ones involving Netgraph nodes. -- Craig
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAG=rPVfiGGHyMY7DtC0WQCguvtvFrtLVQsGBLMh=SZ42YUwpkQ>