public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "Guillaume Boissiere" <boissiere@attbi.com>
To: Dave Hansen <haveblue@us.ibm.com>
Cc: linux-kernel@vger.kernel.org,
	kernel-janitor-discuss@lists.sourceforge.net
Subject: Re: [STATUS 2.5]  May 1, 2002 (BKL status)
Date: Thu, 2 May 2002 07:10:14 -0400	[thread overview]
Message-ID: <3CD0E656.5451.7D1A7885@localhost> (raw)
In-Reply-To: <3CD0047B.4060605@us.ibm.com>

> I may not be the leading BKL expert, but I play one on TV :)
> 
> Perhaps one of the kernel-janitor people would like to assist me with 
> this (cc'ing that list).  I'd be willing to keep a web page to list all 
> current BKL uses and keep track of them as they are removed/added 
> Perhaps a set of web pages which resemble the directory structure of the 
> kernel tree would be helpful??
> 
> Here's a good question for kernel-janitor, and anyone else who's 
> interested, what format describing BKL use would most encourage you to 
> go and remove it?  We already have Rick Lindsley's Global spinlock list: 
>   http://prdownloads.sourceforge.net/lse/locking_doc-2.4.16 .  The BKL 
> use in there is somewhat dated, but might be a good start.
> 
> I have some awk scripts that I use on each new kernel release to check 
> for new and removed uses of the BKL.  I can adapt these to start 
> checking new BK changesets for BKL changes.

I'm by no means a BKL expert, not even on TV  :-) , but here is a 
suggestion.
What about a listing sorted by how the BKL should be replaced, 
something like this:

BKL --> dma_spin_lock
- alpha/kernel/ptrace.c  Held while doing a ptrace      (John Doe)
- drivers/block/rd.c  Held while doing whatever         (?)
- ...

BKL --> xprt_lock
- ...

BKL --> no lock required
- ...

BKL instances that will NOT be removed
- ...

The advantage is that it should be easier to group together instances
that require the same type of work.
Comments?

-- Guillaume





  reply	other threads:[~2002-05-02 11:10 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-05-01 13:53 [STATUS 2.5] May 1, 2002 Guillaume Boissiere
2002-05-01 15:06 ` [STATUS 2.5] May 1, 2002 (BKL status) Dave Hansen
2002-05-02 11:10   ` Guillaume Boissiere [this message]
2002-05-01 20:19 ` [STATUS 2.5] May 1, 2002 Mike Fedyk
2002-05-01 21:38   ` Andrew Morton
2002-05-02  1:11     ` Stephen Lord
2002-05-02  3:14       ` Eric W. Biederman
2002-05-02  3:29         ` Stephen Lord

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=3CD0E656.5451.7D1A7885@localhost \
    --to=boissiere@attbi.com \
    --cc=haveblue@us.ibm.com \
    --cc=kernel-janitor-discuss@lists.sourceforge.net \
    --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