Date: Sun, 17 Feb 2002 15:30:24 -0500 From: "Brian F. Feldman" <green@FreeBSD.org> To: Poul-Henning Kamp <phk@critter.freebsd.dk> Cc: Brian Feldman <green@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern kern_conf.c src/sys/sys conf.h Message-ID: <200202172030.g1HKUOs01319@green.bikeshed.org> In-Reply-To: Message from Poul-Henning Kamp <phk@critter.freebsd.dk> of "Sun, 17 Feb 2002 21:08:16 %2B0100." <5581.1013976496@critter.freebsd.dk>
next in thread | previous in thread | raw e-mail | index | archive | help
Poul-Henning Kamp <phk@critter.freebsd.dk> wrote: > In message <200202161735.g1GHZ5N03909@freefall.freebsd.org>, Brian Feldman writ > es: > >green 2002/02/16 09:35:05 PST > > > > Modified files: > > sys/kern kern_conf.c > > sys/sys conf.h > > Log: > > Add revoke_and_destroy_dev(), to be used by devices which decide when > > they choose to destroy themselves without regard to whether or not > > they are open. > > Hmm, I can see the point but I am not 100% convinced about the > solution, I may modify this API later. I had in my tree a dev_revoke() call which I used at times, previously, which IIRC was just the first part of that function, to be called before a destroy_dev() when a driver's decided a removeable device had gone away. I like the combination of the two functions because it serves to document what's going on pretty well, but have no strong attachment; I'll be happy to start sprinkling either around the src/sys tree where appropriate. I should compile a list of places in the kernel where it should be used other than the initial one I have it used for locally. -- Brian Fundakowski Feldman \'[ FreeBSD ]''''''''''\ <> green@FreeBSD.org <> bfeldman@tislabs.com \ The Power to Serve! \ Opinions expressed are my own. \,,,,,,,,,,,,,,,,,,,,,,\ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200202172030.g1HKUOs01319>