On Tue, 2007-09-25 at 17:36 +0900, Tejun Heo wrote: > Hmmm.... There doesn't seem to any reason why the blocking should be > after calling ->exit(). And, yeah, it would be more useful and > intuitive if blocking happens before ->exit(). What do you think? *That* I have no problem with. I was going to say "just grab a reference to every module" except if a new module is loaded you don't know about it. If you move your blocking, it seems fine. Thanks! Rusty. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
- Follow-Ups:
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: Tejun Heo <[email protected]>
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- References:
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: [email protected] (Jonathan Corbet)
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: Tejun Heo <[email protected]>
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: Rusty Russell <[email protected]>
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: Tejun Heo <[email protected]>
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: Rusty Russell <[email protected]>
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: Tejun Heo <[email protected]>
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: Rusty Russell <[email protected]>
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: Tejun Heo <[email protected]>
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: Rusty Russell <[email protected]>
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: Tejun Heo <[email protected]>
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- From: Tejun Heo <[email protected]>
- Re: [PATCH 1/4] module: implement module_inhibit_unload()
- Prev by Date: Re: [PATCH 5/4] [-mm patch] Rename macros returning the size.
- Next by Date: Re: [PATCH 6/4] [-mm patch] use the existing offsetof().
- Previous by thread: Re: [PATCH 1/4] module: implement module_inhibit_unload()
- Next by thread: Re: [PATCH 1/4] module: implement module_inhibit_unload()
- Index(es):
 |