Re: [Libguestfs] [PATCH v2 1/4] v2v: consolidate virtio-win file copying
On Tue, Oct 13, 2015 at 03:16:27PM +0100, Richard W.M. Jones wrote:
On Tue, Oct 13, 2015 at 03:50:51PM +0300, Roman Kagan wrote:
>
> - match drivers with
> - | [] ->
> - warning (f_"there are no virtio drivers available for this version of
Windows (%d.%d %s %s). virt-v2v looks for drivers in %s\n\nThe guest will be configured
to use slower emulated devices.")
> - inspect.i_major_version inspect.i_minor_version
> - inspect.i_arch inspect.i_product_variant
> - virtio_win;
> - ( IDE, RTL8139, Cirrus )
I believe this warning gets lost in this patch.
To get it back, let's make 'copy_virtio_drivers' returns a true/false
indication ('true' meaning that at least some drivers were copied),
and the caller code in 'install_virtio_drivers' should check this, and
either do the code above, or continue with the code below.
Is there any difference between not being able to locate some of the
virtio drivers or all of them?
With this patch, you'll get an individual warning for every one of the
missing drivers. I don't see why we need an extra case for "all drivers
missing" (for any particular value of "all").
Roman.