Hi
On Thu, Dec 14, 2023 at 1:29 PM Philippe Mathieu-Daudé
<phi...@linaro.org> wrote:
>
> Hi Akihiko,
>
> On 14/12/23 07:47, Akihiko Odaki wrote:
> > -display lists display backends, but does not tell their options.
> > Use the help messages from qemu-options.def, which include the list of
> > options.
> >
> > Signed-off-by: Akihiko Odaki <akihiko.od...@daynix.com>
> > ---
> > include/ui/console.h | 1 -
> > system/vl.c | 11 ++++++-----
> > ui/console.c | 20 --------------------
> > 3 files changed, 6 insertions(+), 26 deletions(-)
>
>
> > diff --git a/ui/console.c b/ui/console.c
> > index 7db921e3b7d6..6aee5e9a7ffb 100644
> > --- a/ui/console.c
> > +++ b/ui/console.c
> > @@ -1691,23 +1691,3 @@ const char *qemu_display_get_vc(DisplayOptions *opts)
> > }
> > return vc;
> > }
> > -
> > -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]) {
> > - Error *local_err = NULL;
> > - int rv = ui_module_load(DisplayType_str(idx), &local_err);
> > - if (rv < 0) {
> > - error_report_err(local_err);
> > - }
> > - }
> > - if (dpys[idx]) {
> > - printf("%s\n", DisplayType_str(dpys[idx]->type));
>
> Is the "qapi/qapi-commands-ui.h" header still necessary?
>
> > - }
> > - }
> > -}
>
> So we go from:
>
> $ ./qemu-system-aarch64 -display help
> Available display backend types:
> none
> gtk
> sdl
> curses
> cocoa
> dbus
I think this used to show only the available display/ui modules
>
> to:
>
> $ ./qemu-system-aarch64 -display help
> -display sdl[,gl=on|core|es|off][,grab-mod=<mod>][,show-cursor=on|off]
> [,window-close=on|off]
> -display gtk[,full-screen=on|off][,gl=on|off][,grab-on-hover=on|off]
> [,show-tabs=on|off][,show-cursor=on|off][,window-close=on|off]
> [,show-menubar=on|off]
> -display vnc=<display>[,<optargs>]
> -display curses[,charset=<encoding>]
> -display cocoa[,full-grab=on|off][,swap-opt-cmd=on|off]
> -display dbus[,addr=<dbusaddr>]
> [,gl=on|core|es|off][,rendernode=<file>]
> -display cocoa[,show-cursor=on|off][,left-command-key=on|off]
> -display none
> select display backend type
> The default display is equivalent to
> "-display gtk"
>
> The latter is indeed more helpful.
While this will provide help for all the modules built.
Not a big deal to me, but probably worth mentioning in the commit message.
