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][RESEND] Control panel changes for IA64/VTI

To: "Tian, Kevin" <kevin.tian@xxxxxxxxx>, "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH][RESEND] Control panel changes for IA64/VTI
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Sun, 23 Oct 2005 11:53:30 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, Xen-IA64-Devel <Xen-IA64-Devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Sun, 23 Oct 2005 03:50:54 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
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: AcXW0pxOKwFXs6btQrCm49nNJpMLkQAri+wwAAD5KkA=
Thread-topic: [Xen-devel] [PATCH][RESEND] Control panel changes for IA64/VTI
BTW, the reason for an ia64 specific get_pfn_list (by Matt) is that machine 
pages for domain on ia64 are allocated on demand. So there's requirement for 
parameters to contain start_pfn and nr_pages so that Xen can allocate necessary 
pages for that specific range. On x86 side, since all the pages are allocated 
when domain creation, get_pfn_list is aimed to query once for all pages and so 
the parameter can be only nr_pages. Just the background here. ;-)

Thanks,
Kevin

>-----Original Message-----
>From: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
>[mailto:xen-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Tian, Kevin
>Sent: 2005年10月23日 11:47
>To: Keir Fraser
>Cc: xen-devel@xxxxxxxxxxxxxxxxxxx; Xen-IA64-Devel
>Subject: [Xen-devel] [PATCH][RESEND] Control panel changes for IA64/VTI
>
>>From: Keir Fraser [mailto:Keir.Fraser@xxxxxxxxxxxx]
>>Sent: 2005年10月22日 14:26
>>On 22 Oct 2005, at 04:20, Tian, Kevin wrote:
>>
>>>     Attached are the necessary changes in control panel to support
>>> ia64/vti domain builder. One is in common side, and the other is in vti
>>> specific image builder. Please take a look, and if no objection, I'd
>>> like to ask Dan's help to check into xen-ia64-unstable.hg first.
>>
>>The common changes to libxc look ia64 specific. Why not stick all of
>>them in ia64_stubs?
>
>OK, see updated version and I moved all interfaces within "ifdef __ia64__" to
>ia64_stubs.
>
>>
>>Adding code to memset to force into the TLB won't be accepted. Either
>>those whole functions are ia64 specific or (better) replace use of
>>mlock/munlock with map_to_hypervisor/unmap_from_hypervisor which do
>>mlock/memset/munlock on ia64 and skip the memset on x86.
>>
>>  -- Keir
>
>Actually those changes are all ia64 specific. With normal diff command, I can 
>use "-p"
>option to add one line like "@@ -322,46 +322,6 @@ int xc_ia64_get_pfn_list(int
>xc_handle,". However I don't know how to produce same format with "hg diff". 
>That
>brings some trouble to people when similar code pieces exist in same file. ;-( 
>I also
>removed memset in xc_ia64_get_pfn_list since mlock seems to have the effect.
>
>Thanks,
>Kevin


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