Re: [Xen-devel] [PATCH] libxl: Add none to vga parameter

[Fabio Fantoni <fabio.fantoni@xxxxxxx>]

Il 15/11/2013 12:49, Stefano Stabellini ha scritto:
> On Fri, 15 Nov 2013, Fabio Fantoni wrote:
> > Usage:
> >    vga="none"
> >
> > With upstream qemu it was impossible to disable emulated vga,
> > even with -nographic qemu parameter setted with nographic xl
> > paramter.
> >
> > Note:
> > Tested only with upstream qemu. With the qemu-trad. it probably
> > needs also the addition of -nographic qemu parameter.
> > Can someone test it with the traditional or should I do it?
> >
> > Signed-off-by: Fabio Fantoni <fabio.fantoni@xxxxxxx>
> > ---
> >   docs/man/xl.cfg.pod.5       |    2 +-
> >   tools/libxl/libxl_create.c  |    4 ++++
> >   tools/libxl/libxl_dm.c      |    3 +++
> >   tools/libxl/libxl_types.idl |    1 +
> >   tools/libxl/xl_cmdimpl.c    |    2 ++
> >   5 files changed, 11 insertions(+), 1 deletion(-)
> >
> > diff --git a/docs/man/xl.cfg.pod.5 b/docs/man/xl.cfg.pod.5
> > index 3dedd61..402a414 100644
> > --- a/docs/man/xl.cfg.pod.5
> > +++ b/docs/man/xl.cfg.pod.5
> > @@ -1050,7 +1050,7 @@ This option is deprecated, use vga="stdvga" instead.
> >   
> >   =item B<vga="STRING">
> >   
> > -Selects the emulated video card (stdvga|cirrus).
> > +Selects the emulated video card (none|stdvga|cirrus).
> >   The default is cirrus.
> >   
> >   =item B<vnc=BOOLEAN>
> > diff --git a/tools/libxl/libxl_create.c b/tools/libxl/libxl_create.c
> > index 2bb33e9..a9735d0 100644
> > --- a/tools/libxl/libxl_create.c
> > +++ b/tools/libxl/libxl_create.c
> > @@ -222,6 +222,8 @@ int libxl__domain_build_info_setdefault(libxl__gc *gc,
> >           switch (b_info->device_model_version) {
> >           case LIBXL_DEVICE_MODEL_VERSION_QEMU_XEN_TRADITIONAL:
> >               switch (b_info->u.hvm.vga.kind) {
> > +            case LIBXL_VGA_INTERFACE_TYPE_NONE:
> > +                break;
> >               case LIBXL_VGA_INTERFACE_TYPE_STD:
> >                   if (b_info->video_memkb == LIBXL_MEMKB_DEFAULT)
> >                       b_info->video_memkb = 8 * 1024;
> > @@ -242,6 +244,8 @@ int libxl__domain_build_info_setdefault(libxl__gc *gc,
> >           case LIBXL_DEVICE_MODEL_VERSION_QEMU_XEN:
> >           default:
> >               switch (b_info->u.hvm.vga.kind) {
> > +            case LIBXL_VGA_INTERFACE_TYPE_NONE:
> > +                break;
> >               case LIBXL_VGA_INTERFACE_TYPE_STD:
> >                   if (b_info->video_memkb == LIBXL_MEMKB_DEFAULT)
> >                       b_info->video_memkb = 16 * 1024;
> > diff --git a/tools/libxl/libxl_dm.c b/tools/libxl/libxl_dm.c
> > index 7be0a50..086e602 100644
> > --- a/tools/libxl/libxl_dm.c
> > +++ b/tools/libxl/libxl_dm.c
> > @@ -199,6 +199,7 @@ static char ** libxl__build_device_model_args_old(libxl__gc 
> > *gc,
> >               flexarray_append(dm_args, "-std-vga");
> >               break;
> >           case LIBXL_VGA_INTERFACE_TYPE_CIRRUS:
> > +        case LIBXL_VGA_INTERFACE_TYPE_NONE:
> >               break;
> >           }
> >   
> > @@ -498,6 +499,8 @@ static char ** libxl__build_device_model_args_new(libxl__gc *gc,
> >                   GCSPRINTF("vga.vram_size_mb=%d",
> >                   libxl__sizekb_to_mb(b_info->video_memkb)));
> >               break;
> > +        case LIBXL_VGA_INTERFACE_TYPE_NONE:
> > +            break;
> >           }
> Shouldn't we be passing -vga none to QEMU instead of nothing at all?
Is not needed, there is already -nodefaults and -vga is also deprecated 
and following the qemu docs:
http://xenbits.xen.org/gitweb/?p=qemu-upstream-unstable.git;a=blob;f=docs/qdev-device-use.txt
the new parameter for replace -vga=none is -nodefaults
the -nodefaults patch is already on git: 
http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=6ef823fdfa701b3659e4161520f43b5835338fb5
and also the new parameter for vgas: 
http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=74914a4b9daa7ce8bb65744f2715ed92b6f34e28
without this patch is impossible disabled emulated vga with hvm domUs 
(problem found time ago trying nographic xl paramter), the succesive 
patch fix also nographic parameter.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel