Hi Luis, great catch! Thanks for the patch, be aware that such virtio drivers can make so much difference it's really important for us to know.
Cheers, -FV ----- Original Message ----- From: "Luis Fernandez Alvarez" <[email protected]> To: "Justin Clift" <[email protected]> Cc: "aeolus-devel" <[email protected]>, "Chris Lalancette" <[email protected]> Sent: Wednesday, November 7, 2012 2:53:21 PM Subject: RE: [PATCH] Added basic support for Virtio in Windows guests in Oz. 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
