[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/4] fbdev: add monitor command to enable/disabl
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 4/4] fbdev: add monitor command to enable/disable |
Date: |
Tue, 11 Sep 2012 11:30:56 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) |
Gerd Hoffmann <address@hidden> writes:
> This patch adds a fbdev monitor command to enable/disable
> the fbdev display at runtime to both qmp and hmp.
>
> qmp: fbdev enable=on|off
> hmp: fbdev on|off
> ---
> hmp-commands.hx | 15 +++++++++++++++
> hmp.c | 9 +++++++++
> hmp.h | 1 +
> qapi-schema.json | 15 +++++++++++++++
> qmp-commands.hx | 6 ++++++
> qmp.c | 17 +++++++++++++++++
> 6 files changed, 63 insertions(+), 0 deletions(-)
>
> diff --git a/hmp-commands.hx b/hmp-commands.hx
> index f6104b0..1ef372b 100644
> --- a/hmp-commands.hx
> +++ b/hmp-commands.hx
> @@ -1378,6 +1378,21 @@ passed since 1970, i.e. unix epoch.
> ETEXI
>
> {
> + .name = "fbdev",
> + .args_type = "enable:b",
> + .params = "on|off",
> + .help = "enable/disable fbdev",
> + .mhandler.cmd = hmp_fbdev,
> + },
> +
> +STEXI
> address@hidden fbdev on | off
> address@hidden fbdev
> +
> +enable/disable fbdev
> +ETEXI
> +
> + {
> .name = "info",
> .args_type = "item:s?",
> .params = "[subcommand]",
> diff --git a/hmp.c b/hmp.c
> index 81c8acb..a7f9ecb 100644
> --- a/hmp.c
> +++ b/hmp.c
> @@ -1102,3 +1102,12 @@ void hmp_closefd(Monitor *mon, const QDict *qdict)
> qmp_closefd(fdname, &errp);
> hmp_handle_error(mon, &errp);
> }
> +
> +void hmp_fbdev(Monitor *mon, const QDict *qdict)
> +{
> + int enable = qdict_get_bool(qdict, "enable");
> + Error *errp = NULL;
> +
> + qmp_fbdev(enable, &errp);
> + hmp_handle_error(mon, &errp);
> +}
> diff --git a/hmp.h b/hmp.h
> index 7dd93bf..1eb02aa 100644
> --- a/hmp.h
> +++ b/hmp.h
> @@ -71,5 +71,6 @@ void hmp_netdev_add(Monitor *mon, const QDict *qdict);
> void hmp_netdev_del(Monitor *mon, const QDict *qdict);
> void hmp_getfd(Monitor *mon, const QDict *qdict);
> void hmp_closefd(Monitor *mon, const QDict *qdict);
> +void hmp_fbdev(Monitor *mon, const QDict *qdict);
>
> #endif
> diff --git a/qapi-schema.json b/qapi-schema.json
> index bd8ad74..d83ee2b 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -2493,3 +2493,18 @@
> # Since: 1.2.0
> ##
> { 'command': 'query-target', 'returns': 'TargetInfo' }
> +
> +##
> +# @fbdev:
> +#
> +# Enable/disable fbdev.
> +#
> +# @enable: true to set the link status to be up
> +#
> +# Returns: Nothing on success
> +# If @name is not a valid network device, DeviceNotFound
Doc pasto, please fix.
> +#
> +# Since: 1.3
> +#
> +##
> +{ 'command': 'fbdev', 'data': {'enable': 'bool'} }
> diff --git a/qmp-commands.hx b/qmp-commands.hx
> index 3745a21..8382020 100644
> --- a/qmp-commands.hx
> +++ b/qmp-commands.hx
> @@ -2514,3 +2514,9 @@ EQMP
> .args_type = "",
> .mhandler.cmd_new = qmp_marshal_input_query_target,
> },
> +
> + {
> + .name = "fbdev",
> + .args_type = "enable:b",
> + .mhandler.cmd_new = qmp_marshal_input_fbdev,
> + },
> diff --git a/qmp.c b/qmp.c
> index 8463922..d7306ab 100644
> --- a/qmp.c
> +++ b/qmp.c
> @@ -391,6 +391,23 @@ void qmp_change(const char *device, const char *target,
> }
> }
>
> +void qmp_fbdev(bool enable, Error **errp)
> +{
> +#if defined(CONFIG_LINUX)
> + DisplayState *ds = get_displaystate();
> +
> + if (enable) {
> + if (fbdev_display_init(ds, NULL) != 0) {
> + error_set(errp, QERR_UNDEFINED_ERROR);
Please upgrade to error_setg(), and make the message fit for human
consumption.
> + }
> + } else {
> + fbdev_display_uninit(ds);
> + }
> +#else
> + error_set(errp, QERR_FEATURE_DISABLED, "fbdev");
> +#endif
> +}
> +
> static void qom_list_types_tramp(ObjectClass *klass, void *data)
> {
> ObjectTypeInfoList *e, **pret = data;
- [Qemu-devel] [PATCH 0/4] linux framebuffer display driver, Gerd Hoffmann, 2012/09/10
- [Qemu-devel] [PATCH 4/4] fbdev: add monitor command to enable/disable, Gerd Hoffmann, 2012/09/10
- Re: [Qemu-devel] [PATCH 4/4] fbdev: add monitor command to enable/disable,
Markus Armbruster <=
- [Qemu-devel] [PATCH 1/4] QLIST-ify display change listeners., Gerd Hoffmann, 2012/09/10
- [Qemu-devel] [PATCH 2/4] add unregister_displaychangelistener, Gerd Hoffmann, 2012/09/10
- [Qemu-devel] [PATCH 3/4] fbdev: add linux framebuffer display driver., Gerd Hoffmann, 2012/09/10
- Re: [Qemu-devel] [PATCH 0/4] linux framebuffer display driver, Alon Levy, 2012/09/11