From: Andi Kleen <andi@firstfloor.org>
To: Richard Holden <aciddeath@gmail.com>
Cc: linux-kernel@vger.kernel.org, mingo@elte.hu
Subject: Re: BKL still required for what functions?
Date: Tue, 07 Oct 2008 08:28:11 +0200 [thread overview]
Message-ID: <87ljx0nclg.fsf@basil.nowhere.org> (raw)
In-Reply-To: <F99FEC4F-DEDD-4E55-93B4-A0761F30E1A2@gmail.com> (Richard Holden's message of "Mon, 6 Oct 2008 22:24:12 -0600")
Richard Holden <aciddeath@gmail.com> writes:
> I'm doing some driver cleanup on the ixj telephony driver, mostly
> copy/ paste and common code refactoring. I'm also looking at trying to
> get rid of the BKL from the 2 places its used, so I was wondering if
> we have a list of kernel calls that still require the BKL to be held,
> and if we don't have a list would it be worthwhile to put one into
> Documentation, I'd be willing to maintain the list as we removed the
> BKL from the last core kernel places.
The standard answer is to check the callers.
But in general there are very little (in fact I cannot think of any)
general kernel utility functions left that need the BKL and if they
then usually take it on their own. So you should be ok
assuming that general kernel code (e.g. code living in kernel/*
or lib/*) doesn't need it.
If you rely on other driver frameworks that might be different,
but a good rule of thumb is to just check that directly if it
uses lock_kernel at all.
The reason that a lot of driver code still uses BKL implicitely
(e.g. in ioctl code) is more because it always needs someone
to audit.
-Andi
--
ak@linux.intel.com
next prev parent reply other threads:[~2008-10-07 6:28 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-07 4:24 Richard Holden
2008-10-07 6:28 ` Andi Kleen [this message]
2008-10-07 16:35 ` Alan Cox
2008-10-07 17:55 ` Richard Holden
2008-10-07 18:14 ` Alan Cox
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=87ljx0nclg.fsf@basil.nowhere.org \
--to=andi@firstfloor.org \
--cc=aciddeath@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
/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
Powered by JetHome