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

Francesco Vollero fvollero at redhat.com
Sat Nov 10 15:39:02 UTC 2012


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" <luis.fernandez.alvarez at cern.ch>
To: "Justin Clift" <jclift at redhat.com>
Cc: "aeolus-devel" <aeolus-devel at lists.fedorahosted.org>, "Chris Lalancette" <clalancette at gmail.com>
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 [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