WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] x86-64 tools fix question

To: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] x86-64 tools fix question
From: Anthony Liguori <aliguori@xxxxxxxxxx>
Date: Tue, 01 Mar 2005 15:23:32 -0600
Cc: Jerone Young <jyoung5@xxxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxxx>
Delivery-date: Wed, 02 Mar 2005 14:43:30 +0000
Envelope-to: xen+James.Bulpin@xxxxxxxxxxxx
In-reply-to: <1b8fe72b3f68af920c3ff9bba691daff@xxxxxxxxxxxx>
List-archive: <http://sourceforge.net/mailarchive/forum.php?forum=xen-devel>
List-help: <mailto:xen-devel-request@lists.sourceforge.net?subject=help>
List-id: List for Xen developers <xen-devel.lists.sourceforge.net>
List-post: <mailto:xen-devel@lists.sourceforge.net>
List-subscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>, <mailto:xen-devel-request@lists.sourceforge.net?subject=subscribe>
List-unsubscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>, <mailto:xen-devel-request@lists.sourceforge.net?subject=unsubscribe>
Organization: IBM
References: <1109707006.13010.15.camel@thinkpad> <1b8fe72b3f68af920c3ff9bba691daff@xxxxxxxxxxxx>
Sender: xen-devel-admin@xxxxxxxxxxxxxxxxxxxxx
User-agent: Mozilla Thunderbird 1.0 (X11/20041206)
Keir Fraser wrote:


On 1 Mar 2005, at 19:56, Jerone Young wrote:

I would like to discuss is this correct, dead wrong, or even needed at
all?


x86/64 has proper barrier instructions -- see include/asm-x86_64/system.h in Linux. It is from there that we should pull our definitions. Barrier macros are defined in a few places in the tools -- we ought to pull them all into one single header incorporated by all tools that need it.

I was just about to send this out myself :-) I think the consensus was to put them all in xc.h. I posted a patch recently that made everything include asm/system.h. All it should take is replacing asm/system.h with xc.h in the patch and then making the necessary changes to xc.h.

For reference, the proper x86-64 barriers are:

#define mb()     asm volatile("mfence":::"memory")
#define rmb()    asm volatile("lfence":::"memory")
#define wmb()    asm volatile("sfence":::"memory")

Regards,
Anthony Liguori

--
Regards,

Anthony Liguori



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/xen-devel