[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20100713182527.GG15122@anguilla.noreply.org>
Date: Tue, 13 Jul 2010 20:25:27 +0200
From: Peter Palfrader <peter@...frader.org>
To: Avi Kivity <avi@...hat.com>
Cc: Linus Torvalds <torvalds@...ux-foundation.org>,
Ingo Molnar <mingo@...e.hu>, "H. Peter Anvin" <hpa@...or.com>,
Greg KH <gregkh@...e.de>, linux-kernel@...r.kernel.org,
stable@...nel.org, stable-review@...nel.org,
akpm@...ux-foundation.org, alan@...rguk.ukuu.org.uk,
Glauber Costa <glommer@...hat.com>,
Zachary Amsden <zamsden@...hat.com>,
Jeremy Fitzhardinge <jeremy@...p.org>,
Marcelo Tosatti <mtosatti@...hat.com>
Subject: Re: [patch 134/149] x86, paravirt: Add a global synchronization
point for pvclock
On Tue, 13 Jul 2010, Avi Kivity wrote:
> Well, current upstream uses "m":
>
>> case 8: \
>> asm volatile(lock "cmpxchgq %1,%2" \
>> : "=a"(__ret) \
>> : "r"(__new), "m"(*__xg(ptr)), "0"(__old) \
>> : "memory"); \
>> break; \
>
> and works; I also failed to reproduce with 2.6.32.16. So I expect some
> toolchain involvement.
>
> Peter, what gcc are you using?
gcc (Debian 4.3.2-1.1) 4.3.2
--
| .''`. ** Debian GNU/Linux **
Peter Palfrader | : :' : The universal
http://www.palfrader.org/ | `. `' Operating System
| `- http://www.debian.org/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists