[Libguestfs] [PATCH 2/2] v2v: fix priority of match in configure_kernel_modules
Richard W.M. Jones
rjones at redhat.com
Fri Jul 1 16:06:09 UTC 2016
On Fri, Jul 01, 2016 at 05:52:01PM +0200, Pino Toscano wrote:
> This makes sure that in the internal configure_kernel_modules function,
> for virtio or SCSI block types:
> a) the warnings about leftover Xen modules are printed
> b) the changes in Augeas are saved
Best to note in the commit message that this fixes
commit ee0219148352695ddf1102a426dcfe4e3be55fba.
ACK.
Rich.
> v2v/convert_linux.ml | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/v2v/convert_linux.ml b/v2v/convert_linux.ml
> index 0296844..a5ba8dd 100644
> --- a/v2v/convert_linux.ml
> +++ b/v2v/convert_linux.ml
> @@ -1147,7 +1147,7 @@ let rec convert ~keep_serial_console (g : G.guestfs) inspect source rcaps =
>
> (* Update 'alias scsi_hostadapter ...' *)
> let paths = augeas_modprobe ". =~ regexp('scsi_hostadapter.*')" in
> - match block_type with
> + (match block_type with
> | Virtio_blk | Virtio_SCSI ->
> let block_module =
> match block_type with
> @@ -1180,7 +1180,7 @@ let rec convert ~keep_serial_console (g : G.guestfs) inspect source rcaps =
> | IDE ->
> (* There is no scsi controller in an IDE guest. *)
> List.iter (fun path -> ignore (g#aug_rm path)) (List.rev paths)
> - ;
> + );
>
> (* Display a warning about any leftover Xen modules which we
> * haven't converted. These are likely to cause an error when
> --
> 2.5.5
>
> _______________________________________________
> Libguestfs mailing list
> Libguestfs at redhat.com
> https://www.redhat.com/mailman/listinfo/libguestfs
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-top is 'top' for virtual machines. Tiny program with many
powerful monitoring features, net stats, disk stats, logging, etc.
http://people.redhat.com/~rjones/virt-top
More information about the Libguestfs
mailing list