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] rename VMX option lapic to apic, and pass it to

To: "Li, Xin B" <xin.b.li@xxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] rename VMX option lapic to apic, and pass it to xen by hvm_info_table
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Tue, 3 Jan 2006 13:25:25 +0000
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 03 Jan 2006 13:24:36 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <59D45D057E9702469E5775CBB56411F10147FF5E@pdsmsx406>
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>
References: <59D45D057E9702469E5775CBB56411F10147FF5E@pdsmsx406>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx

On 3 Jan 2006, at 12:47, Li, Xin B wrote:

In current implementation, if 'lapic=1', both lapic and ioapic are
enabled for VMX guest, and this is what we prefer.
VMX guest has 3 PIC/APIC modes:
1. no lapic and ioapic, only PIC, and this is the default option.
2. both lapic and ioapic, enabled by 'lapic=1' in VMX config file.
3. only lapic and PIC, no ioapic. No such option, but VMX guest can only
use lapic by not using its ioapic.
This is the reason why I made this change.

Okay, that makes sense.

Placing the hvm info definitions in public/io/ioreq.h is not right. It
probably needs its own header file. How about a new subdirectory
include/public/hvm, containing hvm_info.h, ioreq.h and vmx_assist.h?


Yes, I like this new directory for VMX guest using, we do have some
structures defined twice.

You'll send another patch?

 -- Keir


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