xen-devel
Re: [Xen-devel] [PATCH] rename VMX option lapic to apic, and pass it to
On 2 Jan 2006, at 10:16, Keir Fraser wrote:
I like the name 'lapic' rather than 'apic' as it explicitly
disambiguates from 'ioapic'. And it's only one more character.
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?
Also, does anything in Xen actually use the nr_vcpus variable? There's
an accessor function for obtaining its value, but noone calls it, nor
can I see any reason someone would want to in the future. There's
already a setmaxvcpus dom0_op. Seems the Xen code that reads that part
of the hvm_info struct could be removed.
-- Keir
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
| <Prev in Thread] |
Current Thread |
[Next in Thread>
|
- [Xen-devel] [PATCH] rename VMX option lapic to apic, and pass it to xen by hvm_info_table , Li, Xin B
- RE: [Xen-devel] [PATCH] rename VMX option lapic to apic, and pass it to xen by hvm_info_table , Li, Xin B
- RE: [Xen-devel] [PATCH] rename VMX option lapic to apic, and pass it to xen by hvm_info_table , Li, Xin B
- RE: [Xen-devel] [PATCH] rename VMX option lapic to apic, and pass it to xen by hvm_info_table , Li, Xin B
- RE: [Xen-devel] [PATCH] rename VMX option lapic to apic, and pass it to xen by hvm_info_table , Li, Xin B
- RE: [Xen-devel] [PATCH] rename VMX option lapic to apic, and pass it to xen by hvm_info_table , Li, Xin B
|
|