Date: Wed, 19 May 2004 18:19:57 +0200 From: Pawel Jakub Dawidek <pjd@FreeBSD.org> To: Poul-Henning Kamp <phk@phk.freebsd.dk> Cc: Lukas Ertl <le@freebsd.org> Subject: Re: Unloading GEOM classes Message-ID: <20040519161957.GA845@darkness.comp.waw.pl> In-Reply-To: <20077.1084981408@critter.freebsd.dk> References: <20040519165626.W4275@pcle2.cc.univie.ac.at> <20077.1084981408@critter.freebsd.dk>
next in thread | previous in thread | raw e-mail | index | archive | help
--p+u7T7tYDgrfw07S Content-Type: text/plain; charset=iso-8859-2 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, May 19, 2004 at 05:43:28PM +0200, Poul-Henning Kamp wrote: +> In message <20040519165626.W4275@pcle2.cc.univie.ac.at>, Lukas Ertl writ= es: +> >Hello fellow GEOM hackers, +> > +> >I'd like to get the discussion about unloading GEOM classes rolling aga= in, +> >and I'm really pleading for a change of the current code in geom_subr.c. +>=20 +> OK, I just checked with peter@ and he belives that we can simply do +> a DROP_GIANT() ... PICKUP_GIANT() in the g_modevent() and avoid the +> need for the eventqueue there. +>=20 +> Can somebody try that ? Here is the patch: http://people.freebsd.org/~pjd/patches/g_unload_class.patch UNTESTED. Lukas, could you try it and see if it fix your problem. The whole trick could a g_waitidle() after calling destroy_geom method. BTW. We should really commit g_waitidle_lock() version, it will be useful here too: http://people.freebsd.org/~pjd/patches/g_waitidle_lock.patch --=20 Pawel Jakub Dawidek http://www.FreeBSD.org pjd@FreeBSD.org http://garage.freebsd.pl FreeBSD committer Am I Evil? Yes, I Am! --p+u7T7tYDgrfw07S Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.4 (FreeBSD) iD8DBQFAq4ktForvXbEpPzQRAtOEAJ9p2b/BhzLSkVVCfuHhB/u//t7R3gCePsna kANDWhstw1OduLZ5LOhTmfI= =R2rX -----END PGP SIGNATURE----- --p+u7T7tYDgrfw07S--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040519161957.GA845>