[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


  • To: "Alex Williamson" <alex.williamson@xxxxxx>
  • From: "Xu, Anthony" <anthony.xu@xxxxxxxxx>
  • Date: Tue, 5 Dec 2006 09:22:19 +0800
  • Cc: xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Mon, 04 Dec 2006 17:23:11 -0800
  • List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
  • Thread-index: AccX6eWGJ5hVVeAMR9Gr3mK98/iebAAIR7Pg
  • Thread-topic: [Xen-ia64-devel][PATCH]fix vti inital broken after merge

Alex Williamson write on 2006年12月5日 5:19:
> On Mon, 2006-12-04 at 09:54 +0800, Xu, Anthony wrote:
>> Alex Williamson write on 2006年12月2日 1:44:
>    Does that mean SMP HVM guests work with this patch?  
Yes, it works,
HVM_INFO is only removed for IPF side.

> 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.

In IA32 side, it uses HVM_INFO to pass parameter to hvmloader.

But in IPF side, it uses HOB interface to pass parameter to guest Firmware
in a predifined address, such as vcpu number.
SMP vti has nothing to do with HVM_INFO.
We are working at SMP VTI, the root cause is that vcpu parameter is removed for
Merge, we'll add it at somewhere, we'll send out patch ASAP.



> 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. Thanks,
Agree,


--Anthony

> 
>       Alex

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.