Re: [PATCH] call drv->shutdown at rmmod
From: Patrick Mochel (mochel_at_osdl.org)
Date: 08/14/03
- Previous message: Randy.Dunlap: "Re: 2.6.0-test3-mm1 /proc/ikconfig/config adds space after numbers"
- In reply to: Eric W. Biederman: "[PATCH] call drv->shutdown at rmmod"
- Next in thread: Eric W. Biederman: "Re: [PATCH] call drv->shutdown at rmmod"
- Reply: Eric W. Biederman: "Re: [PATCH] call drv->shutdown at rmmod"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Date: Thu, 14 Aug 2003 09:02:20 -0700 (PDT) To: "Eric W. Biederman" <ebiederm@xmission.com>
[ I'm curious as to why you didn't cc me in the first place.. ]
> At the kexec BOF at OSDL there was some discussion on calling the
> device shutdown method at module remove time, in addition to calling
> it during reboot. The driver was the observation that the primary
> source of problems in booting linux from linux are drivers with bad
> or missing drv->shutdown() routines. The hope is this will increase
> the testing so people can get it right and kexec can become more
> useful. In addition to making normal reboots more reliable.
>
> The following patch is an implementation of that idea it calls drv->shutdown()
> before calling drv->remove(). If drv->shutdown() is implemented.
I like the idea behind the patch, but we shouldn't be calling it
unconditionally. We're bound to run into some suprises that could really
kill us this late in the 2.6 game.
I do think it should go in, as long as there is a flag telling the core
whether or not to call shutdown for that particular device. I think it
could also be extended to include a power state, so the core could suspend
the device before removing the module.
The default would always be 'Do Nothing', but with a per-device sysfs
file, a developer/user/gui app could be used to set the policy from user
space.
-pat
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
- Previous message: Randy.Dunlap: "Re: 2.6.0-test3-mm1 /proc/ikconfig/config adds space after numbers"
- In reply to: Eric W. Biederman: "[PATCH] call drv->shutdown at rmmod"
- Next in thread: Eric W. Biederman: "Re: [PATCH] call drv->shutdown at rmmod"
- Reply: Eric W. Biederman: "Re: [PATCH] call drv->shutdown at rmmod"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
|
