This patch enables acpi and apic as they should normally be enabled. Fixes booting Windows. Signed-off-by: Ian Main <imain at redhat.com> --- src/task-omatic/task_vm.rb | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/task-omatic/task_vm.rb b/src/task-omatic/task_vm.rb index 507818f..a448d30 100644 --- a/src/task-omatic/task_vm.rb +++ b/src/task-omatic/task_vm.rb @@ -96,6 +96,10 @@ def create_vm_xml(name, uuid, memAllocated, memUsed, vcpus, bootDevice, doc.root.elements["devices"].add_element("input", {"type" => "mouse", "bus" => "ps2"}) doc.root.elements["devices"].add_element("graphics", {"type" => "vnc", "port" => "-1", "listen" => "0.0.0.0"}) + doc.root.add_element("features") + doc.root.elements["features"].add_element("acpi") + doc.root.elements["features"].add_element("apic") + serial = Element.new("serial") serial.add_attribute("type", "pty") serial.add_element("target", {"port" => "0"}) -- 1.6.0.6
Hugh O. Brock
2009-Mar-24 21:04 UTC
[Ovirt-devel] [PATCH server] Enable apic and acpi in VMs
On Tue, Mar 24, 2009 at 01:48:11PM -0700, Ian Main wrote:> This patch enables acpi and apic as they should normally be enabled. > Fixes booting Windows. > > Signed-off-by: Ian Main <imain at redhat.com> > --- > src/task-omatic/task_vm.rb | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/src/task-omatic/task_vm.rb b/src/task-omatic/task_vm.rb > index 507818f..a448d30 100644 > --- a/src/task-omatic/task_vm.rb > +++ b/src/task-omatic/task_vm.rb > @@ -96,6 +96,10 @@ def create_vm_xml(name, uuid, memAllocated, memUsed, vcpus, bootDevice, > doc.root.elements["devices"].add_element("input", {"type" => "mouse", "bus" => "ps2"}) > doc.root.elements["devices"].add_element("graphics", {"type" => "vnc", "port" => "-1", "listen" => "0.0.0.0"}) > > + doc.root.add_element("features") > + doc.root.elements["features"].add_element("acpi") > + doc.root.elements["features"].add_element("apic") > + > serial = Element.new("serial") > serial.add_attribute("type", "pty") > serial.add_element("target", {"port" => "0"})Tested on an installed server -- with the patch win2k3 VMs will install correctly. ACK. --Hugh