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/
Home Products Support Community News


RE: [Xen-ia64-devel][PATCH]fix vti inital broken after merge

To: "Alex Williamson" <alex.williamson@xxxxxx>, "Xu, Anthony" <anthony.xu@xxxxxxxxx>
Subject: RE: [Xen-ia64-devel][PATCH]fix vti inital broken after merge
From: "Zhang, Xing Z" <xing.z.zhang@xxxxxxxxx>
Date: Tue, 5 Dec 2006 09:07:31 +0800
Cc: xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Mon, 04 Dec 2006 17:19:09 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AccX6eUB1LHWu/YuTnadxEX/KpCtjQAHfBXg
Thread-topic: [Xen-ia64-devel][PATCH]fix vti inital broken after merge
Hi Alex:
        With this patch, SMP HVM only can boot as UP. Due to add_vcpus_hob() 
has been removed from build_hob(). Since parameter 'vcpus' not available in 
setup_guest(). We prefer to use index 'PAE' to save vcpus to xen in python code 
and get it back in setup_guest(). Codes probably like below:

In arch-ia64.h:
+/* HVM_PARAM_PAE_ENABLED not used in IA64, so we used this param to save vcpu
+  number */

In xc.c:
+#if defined(__ia64__)
+    xc_set_hvm_param(self->xc_handle, dom, HVM_PARAM_VCPUS, vcpus);

In xc_ia64_hvm_build.c
+       xc_get_hvm_param(xc_handle, dom, HVM_PARAM_VCPUS, &vcpus)

Good good study,day day up ! ^_^
-Wing(zhang xin)
OTC,Intel Corporation

>-----Original Message-----
>From: Alex Williamson [mailto:alex.williamson@xxxxxx]
>Sent: 2006年12月5日 5:19
>To: Xu, Anthony
>Cc: Zhang, Xing Z; xen-ia64-devel
>Subject: RE: [Xen-ia64-devel][PATCH]fix vti inital broken after merge
>On Mon, 2006-12-04 at 09:54 +0800, Xu, Anthony wrote:
>> Alex Williamson write on 2006年12月2日 1:44:
>> >
>> >    I think it would be cleaner just to make the ia64 specific
>> > xc_set_hvm_param() ignore HVM_PARAM_PAE_ENABLED (or have the xen side
>> > of the hypercall ignore it).
>> Alex,
>> For HVM_INFO and HVM_PARAM_PAE_ENABLED, I think they are IA32-spicific,
>> and should not be put in common path.
>Hi Anthony,
>   Does that mean SMP HVM guests work with this patch?  I was wondering
>if HVM_INFO is how x86 is setting up the number of vCPUs.  I removed the
>code in tools/libxc/ia64/xc_ia64_hvm_build.c that appeared to handle
>this, so thought we might need to migrate to something like HVM_INFO to
>do that now.  PAE is x86 specific of course, but it's part of the
>generic HVM params.  It won't hurt anything to set the parameter on
>ia64, it just won't do anything.  Maybe after 3.0.4 it would be
>worthwhile to create an interface for setting arch-specific parameters.
>       Alex
>Alex Williamson                             HP Open Source & Linux Org.

Xen-ia64-devel mailing list