public inbox for linux-arch@vger.kernel.org
 help / color / mirror / Atom feed
From: Steven Rostedt <rostedt@goodmis.org>
To: David Howells <dhowells@redhat.com>
Cc: linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org,
	mingo@redhat.com, akpm@osdl.org, torvalds@osdl.org
Subject: Re: [PATCH 1/12]: MUTEX: Implement mutexes
Date: Fri, 16 Dec 2005 22:58:34 -0500	[thread overview]
Message-ID: <1134791914.13138.167.camel@localhost.localdomain> (raw)
In-Reply-To: <200512162313.jBGND7g4019623@warthog.cambridge.redhat.com>

On Fri, 2005-12-16 at 23:13 +0000, David Howells wrote:
> The attached patch introduces a simple mutex implementation as an alternative
> to the usual semaphore implementation where simple mutex functionality is all
> that is required.
> 
> This is useful in two ways:
> 
>  (1) A number of archs only provide very simple atomic instructions (such as
>      XCHG on i386, TAS on M68K, SWAP on FRV) which aren't sufficient to
>      implement full semaphore support directly. Instead spinlocks must be
>      employed to implement fuller functionality.
> 
>  (2) This makes it more obvious that a mutex is a mutex and restricts the
>      capabilites to make it more easier to debug.
> 
> This patch set does the following:
> 
>  (1) Renames DECLARE_MUTEX and DECLARE_MUTEX_LOCKED to be DECLARE_SEM_MUTEX and
>      DECLARE_SEM_MUTEX_LOCKED for counting semaphores.
> 

Could we really get rid of that "MUTEX" part.  A counting semaphore is
_not_ a mutex, although a mutex _is_ a counting semaphore.  As is a Jack
Russell is a dog, but a dog is not a Jack Russell.

What's the reason not to just use DECLARE_SEM and DECLARE_SEM_LOCKED?

-- Steve

  reply	other threads:[~2005-12-17  3:58 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-12-16 23:13 [PATCH 0/12]: MUTEX: Introduce mutex implementation David Howells
2005-12-16 23:13 ` [PATCH 3/12]: MUTEX: Rename DECLARE_MUTEX for arch/ dir David Howells
2005-12-16 23:13 ` [PATCH 2/12]: MUTEX: Provide SWAP-based mutex for FRV David Howells
2005-12-16 23:13 ` [PATCH 1/12]: MUTEX: Implement mutexes David Howells
2005-12-17  3:58   ` Steven Rostedt [this message]
2005-12-17  7:35     ` Linus Torvalds
2005-12-17 19:21       ` David Howells
2005-12-17 20:11         ` Linus Torvalds
2005-12-17 21:44           ` Russell King
2005-12-18  1:29           ` Nicolas Pitre
2005-12-18  2:34             ` Linus Torvalds
2005-12-18  4:07               ` Nicolas Pitre
2005-12-18  4:18                 ` Steven Rostedt
2005-12-18  6:30                 ` Linus Torvalds
2005-12-18  9:26                   ` Russell King
2005-12-18 18:42                     ` Linus Torvalds
2005-12-18 19:41                       ` James Bottomley
2005-12-18 19:54                         ` Linus Torvalds
2005-12-19  1:48                       ` Nicolas Pitre
2005-12-19  9:27                         ` Russell King
2005-12-19 13:54                           ` Matthew Wilcox
2005-12-19 15:49                             ` Nicolas Pitre
2005-12-19 15:45                           ` Nicolas Pitre
2005-12-18 17:29                   ` Nicolas Pitre
2005-12-18 13:38             ` Alan Cox
2005-12-18 17:21               ` Nicolas Pitre
2005-12-17  7:55     ` Nick Piggin
2005-12-17 12:36       ` Steven Rostedt
2005-12-16 23:13 ` [PATCH 9/12]: MUTEX: Rename DECLARE_MUTEX for net/ dir David Howells
2005-12-16 23:13 ` [PATCH 11/12]: MUTEX: Rename DECLARE_MUTEX for miscellaneous directories David Howells
2005-12-16 23:13 ` [PATCH 5/12]: MUTEX: Rename DECLARE_MUTEX for drivers/ dir, N thru Z David Howells
2005-12-16 23:13 ` [PATCH 4/12]: MUTEX: Rename DECLARE_MUTEX for drivers/ dir, A thru M David Howells
2005-12-16 23:13 ` [PATCH 6/12]: MUTEX: Rename DECLARE_MUTEX for fs/ dir David Howells
2005-12-16 23:13 ` [PATCH 10/12]: MUTEX: Rename DECLARE_MUTEX for sound/ dir David Howells
2005-12-16 23:13 ` [PATCH 7/12]: MUTEX: Rename DECLARE_MUTEX for include/asm-*/ dirs David Howells
2005-12-16 23:13 ` [PATCH 8/12]: MUTEX: Rename DECLARE_MUTEX for kernel/ dir David Howells
2005-12-16 23:13 ` [PATCH 12/12]: MUTEX: Provide synchronisation primitive testing module David Howells

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=1134791914.13138.167.camel@localhost.localdomain \
    --to=rostedt@goodmis.org \
    --cc=akpm@osdl.org \
    --cc=dhowells@redhat.com \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=torvalds@osdl.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