All of lore.kernel.org
 help / color / mirror / Atom feed
From: "David C. Hansen" <dave@sr71.net>
To: Ryan Cumming <bodnar42@phalynx.dhs.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [RFC] Change locking in block_dev.c:do_open()
Date: Wed, 12 Dec 2001 18:33:34 -0800	[thread overview]
Message-ID: <3C18137E.78A6FF0A@sr71.net> (raw)
In-Reply-To: <3C17F8B2.6080700@us.ibm.com> <E16EKFr-000305-00@phalynx>

Ryan Cumming wrote:
> On December 12, 2001 16:39, David C. Hansen wrote:
> > We can add a semaphore which must be acquired before a module can be
> > unloaded, and hold it over the area where the module must not be
> > unloaded.  We could replace the unload_lock spinlock with a semaphore,
> > which I'll call it unload_sem here.  It would look something like this:
> Why not use a read-write semaphore? The sections that require the module to
> stay resident use a read lock, and module unloading aquires a write lock. In
> addition to containing the evil, evil BKL, you might actually get a tangiable
> scalability gain out of it.
Cool idea.  I'll do that.  Now that we have those locking primitives in
the kernel I wish that we used them more often. 
-- 
David C. Hansen
dave@sr71.net

  reply	other threads:[~2001-12-13  2:34 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-12-13  0:39 [RFC] Change locking in block_dev.c:do_open() David C. Hansen
2001-12-13  1:01 ` Ryan Cumming
2001-12-13  2:33   ` David C. Hansen [this message]
2001-12-13  2:16 ` Alexander Viro
2001-12-13 15:42   ` Roman Zippel
2001-12-13 17:24   ` Richard Gooch
2001-12-13 17:47     ` Alexander Viro
2001-12-13 18:26       ` Richard Gooch
     [not found]         ` <Pine.GSO.4.21.0112140318290.22724-100000@weyl.math.psu.edu>
2001-12-14 17:07           ` Richard Gooch
  -- strict thread matches above, loose matches on Subject: below --
2001-12-13  2:40 Torrey Hoffman
2001-12-13  2:49 ` Ryan Cumming

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=3C18137E.78A6FF0A@sr71.net \
    --to=dave@sr71.net \
    --cc=bodnar42@phalynx.dhs.org \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.