Justin, I've just test it with an unattended installation of Windows (virtio drivers injected) and the time was reduced from 106 minutes to 52 minutes, so, yes... roughly 50% faster.
I think it's worth to have that option to play with if you use the Virtio drivers for windows. Cheers, Luis. ________________________________________ From: Justin Clift [[email protected]] Sent: 07 November 2012 0:18 To: Luis Fernandez Alvarez Cc: aeolus-devel; Chris Lalancette Subject: Re: [PATCH] Added basic support for Virtio in Windows guests in Oz. On 05/11/2012, at 3:13 PM, Chris Lalancette wrote: > Thanks for the patch! I am going to be away for most of this week, > and probably won't have time to look at the code, but I will look at > it in the coming weekend. I just wanted to let you know I wasn't > ignoring you :). This definitely sounds very useful. :) Luis, did you capture timing figures for the performance improvement? (even very rough) Like 50% faster, etc? :) Regards and best wishes, Justin Clift > Thanks, > Chris > > On Mon, Nov 5, 2012 at 3:08 AM, Luis Fernandez Alvarez > <[email protected]> wrote: >> The Windows VirtIO Drivers allows Microsoft Windows guests to be >> installed using virtio devices. In order to make Oz support this >> case, it's proposed to modify the class Windows.py to get the >> diskbus and nicmodel from the configuration file. The performance >> boost installing Microsoft Windows guests is remarkable. >> --- >> oz/Windows.py | 12 ++++++++++-- >> 1 files changed, 10 insertions(+), 2 deletions(-) >> >> diff --git a/oz/Windows.py b/oz/Windows.py >> index ddc3e7f..ee4a266 100644 >> --- a/oz/Windows.py >> +++ b/oz/Windows.py >> @@ -33,8 +33,16 @@ class Windows(oz.Guest.CDGuest): >> Shared Windows base class. >> """ >> def __init__(self, tdl, config, output_disk): >> - oz.Guest.CDGuest.__init__(self, tdl, config, output_disk, "rtl8139", >> - "localtime", "usb", None, True, False) >> + diskbus = oz.ozutil.config_get_key(config, >> + 'libvirt', >> + 'diskbus', >> + 'ide') >> + nicmodel = oz.ozutil.config_get_key(config, >> + 'libvirt', >> + 'nicmodel', >> + 'rtl8139') >> + oz.Guest.CDGuest.__init__(self, tdl, config, output_disk, nicmodel, >> + "localtime", "usb", diskbus, True, False) >> >> if self.tdl.key is None: >> raise oz.OzException.OzException("A key is required when >> installing Windows") >> -- >> 1.7.1 >> -- Aeolus Community Manager http://www.aeolusproject.org
