Hi Pino,
On 2/26/19 5:52 PM, Mike Latimer wrote:
 On 2/21/19 3:07 AM, Pino Toscano wrote:
> My question is: is using cirrus still the best choice for SUSE guests?
> If not, what about using qxl as well, as done for any non-SUSE guest?
> (We can also do that depending on the version of the guest, in case
> only newer SUSE versions work fine with qxl).
 At the time of my commit, we had to use cirrus. However, I believe all
 the currently supported SUSE versions should now work with qxl. Let me
 verify that, and hopefully the exception can be removed. 
After a bit of checking, defaulting to qxl for SUSE guests is a better
choice, and should work fine for all versions we expect to encounter in
the wild. Do you mind pulling this out yourself, or would you prefer I
submit a patch containing this change? I've moved on to other things,
and really only have enough bandwidth for a minimal change like:
diff -Nurp a/v2v/convert_linux.ml b/v2v/convert_linux.ml
--- a/v2v/convert_linux.ml    2019-02-28 16:30:58.668800431 -0700
+++ b/v2v/convert_linux.ml    2019-02-28 16:33:14.729907825 -0700
@@ -104,7 +104,7 @@ let convert (g : G.guestfs) inspect sour
 
     let video =
       match rcaps.rcaps_video with
-      | None -> get_display_driver ()
+      | None -> QLX
       | Some video -> video in
 
     let block_type =
@@ -771,9 +771,6 @@ let convert (g : G.guestfs) inspect sour
     else
       true
 
-  and get_display_driver () =
-    if family = `SUSE_family then Cirrus else QXL
-
   and configure_display_driver video =
     let video_driver = match video with QXL -> "qxl" | Cirrus ->
"cirrus" in
I'd be happy to submit this as an official patch, or just let you do it
with some of your planned changes.
Thanks!
Mike