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(a)cern.ch>
To: "Justin Clift" <jclift(a)redhat.com>
Cc: "aeolus-devel" <aeolus-devel(a)lists.fedorahosted.org>, "Chris
Lalancette" <clalancette(a)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(a)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(a)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