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 3/10] Add HVM support

To: "Keir Fraser" <keir@xxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH 3/10] Add HVM support
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Wed, 11 Jul 2007 09:14:15 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 10 Jul 2007 18:12:32 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C2B94DA7.1235A%keir@xxxxxxxxxxxxx>
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: Ace4v8AvEHvNElKqQi6Qqle8QqzaiAKGA7QQAAVN3AAAAXlToAABp7KOABeOrXA=
Thread-topic: [Xen-devel] [PATCH 3/10] Add HVM support
>From: Keir Fraser [mailto:keir@xxxxxxxxxxxxx]
>Sent: 2007年7月10日 21:52
>
>Okay.
>
>Anyway, back to your patch 3/10. With a view to cleanly adding VMXOFF
>on
>suspend, and to allow efficient VMCLEARing if we need it in future, e.g.,
>for deep-C states, I think you should change the suspend_domain() hook
>into
>suspend_cpu():
> 1. This is then symmetric with the resume_cpu() hook.
> 2. It's a natural place to put VMXOFF (unlike suspend_domain()).
>
>Of course, the question then is: how do you find the active VMCS's that
>need
>clearing? I suggest you add a list_head to arch_vmx_struct, have a
>per-cpu
>list of active VMCS's, enqueue on vmx_load_vmcs() and dequeue on
>__vmx_clear_vmcs().
>
>Could you revise patch 3/10 and resend, please?
>
> -- Keir

Sure, I'll do it.

Thanks,
Kevin

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

<Prev in Thread] Current Thread [Next in Thread>