From: Greg KH <greg@kroah.com>
To: Zwane Mwaikambo <zwane@linuxpower.ca>
Cc: Linux Kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH][2.6][CFT] rmmod floppy kills box fixes + default_device_remove
Date: Mon, 8 Sep 2003 08:50:48 -0700 [thread overview]
Message-ID: <20030908155048.GA10879@kroah.com> (raw)
In-Reply-To: <Pine.LNX.4.53.0309072228470.14426@montezuma.fsmlabs.com>
On Sun, Sep 07, 2003 at 10:53:08PM -0400, Zwane Mwaikambo wrote:
> Randy gave me the courage to delve in there... now that i'm lost to the
> world here is one picked up from bugzilla;
>
> The crux of it is that the floppy driver isn't deleting timers before
> unloading itself. I've tested it locally somewhat, but the ioport
> busy looks very strange (although things do function). There is one part
> of this patch that i'd like Greg to look at, it's the
> default_device_release addition...
Ick, no, I do not want to see this function get added, sorry.
What happens if someone grabs the struct device reference by opening a
sysfs file and then you unload the module? Yeah, not nice. Please do
_not_ create "empty" release() functions, unless you _really_ know what
you are doing (and providing a "default" one like this is just ripe for
abuse, that warning message in the kernel is there for a reason.)
thanks,
greg k-h
next prev parent reply other threads:[~2003-09-08 15:50 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-09-08 2:53 [PATCH][2.6][CFT] rmmod floppy kills box fixes + default_device_remove Zwane Mwaikambo
2003-09-08 15:50 ` Greg KH [this message]
2003-09-08 21:27 ` Zwane Mwaikambo
2003-09-08 23:08 ` Greg KH
2003-09-09 11:50 ` Zwane Mwaikambo
2003-09-09 16:38 ` Zwane Mwaikambo
2003-09-09 17:13 ` Greg KH
2003-09-09 19:18 ` Zwane Mwaikambo
2003-09-09 19:45 ` John Levon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20030908155048.GA10879@kroah.com \
--to=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=zwane@linuxpower.ca \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.