From: Keith Owens <kaos@ocs.com.au>
To: Dawson Engler <engler@csl.Stanford.EDU>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [QUESTION] MOD_INC/MOD_DEC: useful to check for correct usage?
Date: Tue, 24 Apr 2001 08:57:14 +1000 [thread overview]
Message-ID: <12667.988066634@ocs3.ocs-net> (raw)
In-Reply-To: Your message of "Wed, 04 Apr 2001 18:25:08 PDT." <200104050125.SAA21252@csl.Stanford.EDU>
On Wed, 4 Apr 2001 18:25:08 -0700 (PDT),
Dawson Engler <engler@csl.Stanford.EDU> wrote:
>in the old days you couldn't call a sleeping function in a module
>before doing a MOD_INC or after doing a MOD_DEC. Then some safety nets
>were added that made these obsolete (in some number of places). I was
>told that people had decided to potentially get rid of all safety
>nets. Is this true? Is it worthwhile to have a checker for these two
>rules?
I expect to reintroduce the MOD_{INC,DEC} rules in 2.5. Al Viro's
patches to bump the module use count in the caller work up to a point
but they are not a complete fix. You cannot bump the module use count
from an interrupt, which causes problem for netfilter (ask Rusty for
details).
A module can have multiple associated areas including the code segment,
the exception table and arch specific data like the IA64 unwind lists.
Al Viro's fix does not cover all of these areas, Alan Cox added a
spinlock for the exception table but nothing covers the arch specific
data. Adding more spinlocks is not the answer, it penalizes the
mainline code to guard against an unusual event (module unload).
prev parent reply other threads:[~2001-04-23 22:59 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-04-05 1:25 [QUESTION] MOD_INC/MOD_DEC: useful to check for correct usage? Dawson Engler
2001-04-05 1:34 ` Alexander Viro
2001-04-23 22:57 ` Keith Owens [this message]
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=12667.988066634@ocs3.ocs-net \
--to=kaos@ocs.com.au \
--cc=engler@csl.Stanford.EDU \
--cc=linux-kernel@vger.kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox