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] [PATCH] Cleanup/fix virt_to_maddr

To: Christoph Egger <Christoph.Egger@xxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Cleanup/fix virt_to_maddr
From: Keir Fraser <keir@xxxxxxxxxxxxx>
Date: Wed, 24 Jan 2007 11:06:00 +0000
Delivery-date: Wed, 24 Jan 2007 03:05:46 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <200701241201.32215.Christoph.Egger@xxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acc/p6E0343fWKuaEduptAAX8io7RQ==
Thread-topic: [Xen-devel] [PATCH] Cleanup/fix virt_to_maddr
User-agent: Microsoft-Entourage/11.2.5.060620


On 24/1/07 11:01, "Christoph Egger" <Christoph.Egger@xxxxxxx> wrote:

>> They're only used on Xen heap virtual addresses whose physical addresses
>> are always below 64MB. So there should be no issues in returning a ulong.
> 
> Oh, I see. So it's absolutely not used for domains (which I assumed)?

No, all it does is a simple arithmetic transformation (subtracting
PAGE_OFFSET) which is only valid for non-guest pages on x86_32. It can be
used for any page on x86_64, but there we have paddr_t==ulong.

 -- Keir


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>