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

[Alex Williamson <alex.williamson@xxxxxx>]

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

        Alex

-- 
Alex Williamson                             HP Open Source & Linux Org.


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