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

Justin Clift jclift at redhat.com
Tue Nov 6 23:18:40 UTC 2012


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