public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
From: Ray Bryant <raybry@engr.sgi.com>
To: linux-ia64@vger.kernel.org
Subject: Re: [Linux-ia64] cmpxchg patch
Date: Tue, 05 Feb 2002 07:13:47 +0000	[thread overview]
Message-ID: <marc-linux-ia64-105590698805989@msgid-missing> (raw)
In-Reply-To: <marc-linux-ia64-105590698805982@msgid-missing>

Of course.  I'll give it a try and see what happens.  I hope that
building for a dig machine with 2.4.17 would be a good enough test.  The
easiest machine for me to boot it on would be a SNIA machine though.

Also, I see that previous versions of the in-line asm spinlock code in
IA64 used a proceudure to handle the complicated cases.  However, that
procedure is not part of the source tree at the present time.  I'm
trying to do something similar to create NUMA aware locks that are as
fast (or very nearly so) as the current spin locks, but have the good
NUMA properties of mcs locks.  Any experience you have with creating
spin lock code that calls out to a C procedure to handle the non-common
cases would be appreciated.

David Mosberger wrote:
> 
> >>>>> On Sat, 02 Feb 2002 10:26:42 -0600, Ray Bryant <raybry@engr.sgi.com> said:
> 
>   Ray> It seems to me that the size of the result of
>   Ray> cmpxchg(ptr,old,new) should depend on the size of *ptr and not
>   Ray> the size of old.
> 
>   Ray> Attached is a patch to fix.
> 
> Have you actually tried building a kernel with the patch applied?  I
> think you'll get errors in atomic.h.  I'm willing to entertain a
> better solution, but it needs to be a complete solution.
> 
> Thanks,
> 
>         --david

-- 
Best Regards,
Ray
-----------------------------------------------
Ray Bryant               SGI
512-453-9679 (work)      512-507-7807 (cell)
raybry@sgi.com           raybry@austin.rr.com
     Computers are like air conditioners.
   They stop working when you open Windows.
-----------------------------------------------


  parent reply	other threads:[~2002-02-05  7:13 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-02-02 16:26 [Linux-ia64] cmpxchg patch Ray Bryant
2002-02-05  1:35 ` David Mosberger
2002-02-05  7:13 ` Ray Bryant [this message]
2002-02-05 19:22 ` David Mosberger

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=marc-linux-ia64-105590698805989@msgid-missing \
    --to=raybry@engr.sgi.com \
    --cc=linux-ia64@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