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