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

RE: [Xen-ia64-devel][PATCH]PAL virtualization service related patch


  • To: "Alex Williamson" <alex.williamson@xxxxxx>
  • From: "Xu, Anthony" <anthony.xu@xxxxxxxxx>
  • Date: Mon, 7 Apr 2008 15:44:50 +0800
  • Cc: xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Mon, 07 Apr 2008 00:45:54 -0700
  • List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
  • Thread-index: AciUEuNtvPMJvA/QSqiJzfs++XKh1QEcDrvw
  • Thread-topic: [Xen-ia64-devel][PATCH]PAL virtualization service related patch

Thanks for your comments.
This is the revised one.

Thanks
- Anthony
 

-----Original Message-----
From: Alex Williamson [mailto:alex.williamson@xxxxxx] 
Sent: 2008年4月2日 0:00
To: Xu, Anthony
Cc: xen-ia64-devel
Subject: Re: [Xen-ia64-devel][PATCH]PAL virtualization service related patch

Hi Anthony,

   Could you add prototypes to the appropriate header files and include
them so we can avoid this extern kludgy-ness?  I think a descriptive
comment for this function would also be worthwhile.  Thanks,

        Alex

On Sat, 2008-03-29 at 16:17 +0800, Xu, Anthony wrote:
> --- a/xen/arch/ia64/vmx/vmx_init.c      Fri Mar 14 15:07:45 2008 -0600
> +++ b/xen/arch/ia64/vmx/vmx_init.c      Sat Mar 29 15:32:55 2008 +0800
> @@ -63,6 +63,33 @@ u64 __vsa_base = 0;  /* Run-time service 
>  u64 __vsa_base = 0;    /* Run-time service base of VMX */
>  
>  /* Check whether vt feature is enabled or not. */
> +
> +void vmx_vps_patch(void)
> +{
> +       extern void ia64_patch_imm64 (u64 , u64 );
> +       extern char vmx_vps_sync_read;
> +       extern char vmx_vps_sync_write;
> +       extern char vmx_vps_resume_normal;
> +       extern char vmx_vps_resume_handler;
> +       u64 addr;
-- 
Alex Williamson                             HP Open Source & Linux Org.

Attachment: vps_service2.patch
Description: vps_service2.patch

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