Date: Mon, 27 Aug 2001 17:06:50 +0300 (EEST) From: Maxim Sobolev <sobomax@FreeBSD.ORG> To: dima@unixfreak.org (Dima Dorfman) Cc: sobomax@FreeBSD.ORG (Maxim Sobolev), cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/dev/md md.c Message-ID: <200108271407.f7RE7qM49216@vega.vega.com> In-Reply-To: <20010827133317.4CFF83E31@bazooka.unixfreak.org> from "Dima Dorfman" at Aug 27, 2001 06:33:12 AM
next in thread | previous in thread | raw e-mail | index | archive | help
> > Maxim Sobolev <sobomax@FreeBSD.org> wrote: > > sobomax 2001/08/27 06:25:48 PDT > > > > Modified files: > > sys/dev/md md.c > > Log: > > - On module unload try to detach all configured disks and let unload proceed > > if all disks were detached sucessfully; > > I'm not sure this is wise. More likely than not, trying to unload md > while there are still disks attached is a mistake on the user's part, > especially considering that most people aren't going to be loading and > unloading it on a regular basis. I understand why this may be useful > for a developer working on md, but it seems counter-intuitive from a > regular user's perspective. I'm not sure that I understood your concern. If the user wants to unload md driver it means that he/she no longer needs disks that are currently configured. After all, with this patch applied the driver will autodetach only idle disks, not touching those which are currently in use (i.e. still open()'ed), so no harm will be done anyway. I really don't see why the driver shouldn't help user in this case. -Maxim 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?200108271407.f7RE7qM49216>