[PATCH] Added basic support for Virtio in Windows guests in Oz.

Luis Fernandez Alvarez luis.fernandez.alvarez at cern.ch
Wed Nov 7 13:53:21 UTC 2012


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 [jclift at redhat.com]
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
> <luis.fernandez.alvarez at cern.ch> 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




More information about the aeolus-devel mailing list