| Message ID | 20200120192947.31613-1-philmd@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | ui/console: Display the 'none' backend in '-display help' | expand |
On 1/20/20 1:29 PM, Philippe Mathieu-Daudé wrote: > Commit c388f408b5 added the possibility to list the display > backends using '-display help'. Since the 'none' backend is > is not implemented as a DisplayChangeListenerOps, it is not > registered to the dpys[] array with qemu_display_register(), > and is not listed in the help output. > > This might be confusing, as we list it in the man page: > > -display type > Select type of display to use. This option is a replacement for > the old style -sdl/-curses/... options. Valid values for type are > > none > Do not display video output. The guest will still see an > emulated graphics card, but its output will not be displayed > to the QEMU user. This option differs from the -nographic > option in that it only affects what is done with video > output; -nographic also changes the destination of the serial > and parallel port data. > > Fix by manually listing the special 'none' backend in the help. > > Suggested-by: Thomas Huth <thuth@redhat.com> > Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> > --- > ui/console.c | 1 + > 1 file changed, 1 insertion(+) Reviewed-by: Eric Blake <eblake@redhat.com> > > diff --git a/ui/console.c b/ui/console.c > index 69339b028b..179901c35e 100644 > --- a/ui/console.c > +++ b/ui/console.c > @@ -2338,6 +2338,7 @@ void qemu_display_help(void) > int idx; > > printf("Available display backend types:\n"); > + printf("none\n"); > for (idx = DISPLAY_TYPE_NONE; idx < DISPLAY_TYPE__MAX; idx++) { > if (!dpys[idx]) { > ui_module_load_one(DisplayType_str(idx)); >
On Mon, Jan 20, 2020 at 08:29:47PM +0100, Philippe Mathieu-Daudé wrote: > Commit c388f408b5 added the possibility to list the display > backends using '-display help'. Since the 'none' backend is > is not implemented as a DisplayChangeListenerOps, it is not > registered to the dpys[] array with qemu_display_register(), > and is not listed in the help output. > > This might be confusing, as we list it in the man page: > > -display type > Select type of display to use. This option is a replacement for > the old style -sdl/-curses/... options. Valid values for type are > > none > Do not display video output. The guest will still see an > emulated graphics card, but its output will not be displayed > to the QEMU user. This option differs from the -nographic > option in that it only affects what is done with video > output; -nographic also changes the destination of the serial > and parallel port data. > > Fix by manually listing the special 'none' backend in the help. > > Suggested-by: Thomas Huth <thuth@redhat.com> > Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> Added to ui queue. thanks, Gerd
diff --git a/ui/console.c b/ui/console.c index 69339b028b..179901c35e 100644 --- a/ui/console.c +++ b/ui/console.c @@ -2338,6 +2338,7 @@ void qemu_display_help(void) int idx; printf("Available display backend types:\n"); + printf("none\n"); for (idx = DISPLAY_TYPE_NONE; idx < DISPLAY_TYPE__MAX; idx++) { if (!dpys[idx]) { ui_module_load_one(DisplayType_str(idx));
Commit c388f408b5 added the possibility to list the display backends using '-display help'. Since the 'none' backend is is not implemented as a DisplayChangeListenerOps, it is not registered to the dpys[] array with qemu_display_register(), and is not listed in the help output. This might be confusing, as we list it in the man page: -display type Select type of display to use. This option is a replacement for the old style -sdl/-curses/... options. Valid values for type are none Do not display video output. The guest will still see an emulated graphics card, but its output will not be displayed to the QEMU user. This option differs from the -nographic option in that it only affects what is done with video output; -nographic also changes the destination of the serial and parallel port data. Fix by manually listing the special 'none' backend in the help. Suggested-by: Thomas Huth <thuth@redhat.com> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> --- ui/console.c | 1 + 1 file changed, 1 insertion(+)