aboutsummaryrefslogtreecommitdiff
path: root/ui
diff options
context:
space:
mode:
authorPeter Maydell <peter.maydell@linaro.org>2016-03-24 16:00:14 +0000
committerPeter Maydell <peter.maydell@linaro.org>2016-03-24 16:00:14 +0000
commitf18f2e7cfcdc31d7f82af15947aae60c4b16e131 (patch)
treeda9999caa961abf68c83a264cae13f71cee6fa2a /ui
parent490dda053e3324c97b28439813fab4c2f362183e (diff)
parent569a93cbbe428bb5c583ae4bf31447eb3acc30fe (diff)
Merge remote-tracking branch 'remotes/kraxel/tags/pull-ui-20160324-1' into staging
input-linux + spice fixes # gpg: Signature made Thu 24 Mar 2016 07:54:45 GMT using RSA key ID D3E87138 # gpg: Good signature from "Gerd Hoffmann (work) <kraxel@redhat.com>" # gpg: aka "Gerd Hoffmann <gerd@kraxel.org>" # gpg: aka "Gerd Hoffmann (private) <kraxel@gmail.com>" * remotes/kraxel/tags/pull-ui-20160324-1: spice: Disallow use of gl + TCP port input-linux: fix Coverity warning input-linux: switch over to -object Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'ui')
-rw-r--r--ui/input-linux.c161
-rw-r--r--ui/spice-core.c5
2 files changed, 133 insertions, 33 deletions
diff --git a/ui/input-linux.c b/ui/input-linux.c
index 2cf5d4990c..6ddaa6740f 100644
--- a/ui/input-linux.c
+++ b/ui/input-linux.c
@@ -10,6 +10,7 @@
#include "qemu/sockets.h"
#include "sysemu/sysemu.h"
#include "ui/input.h"
+#include "qom/object_interfaces.h"
#include <sys/ioctl.h>
#include "standard-headers/linux/input.h"
@@ -127,10 +128,21 @@ static int qemu_input_linux_to_qcode(unsigned int lnx)
return linux_to_qcode[lnx];
}
+#define TYPE_INPUT_LINUX "input-linux"
+#define INPUT_LINUX(obj) \
+ OBJECT_CHECK(InputLinux, (obj), TYPE_INPUT_LINUX)
+#define INPUT_LINUX_GET_CLASS(obj) \
+ OBJECT_GET_CLASS(InputLinuxClass, (obj), TYPE_INPUT_LINUX)
+#define INPUT_LINUX_CLASS(klass) \
+ OBJECT_CLASS_CHECK(InputLinuxClass, (klass), TYPE_INPUT_LINUX)
+
typedef struct InputLinux InputLinux;
+typedef struct InputLinuxClass InputLinuxClass;
struct InputLinux {
- const char *evdev;
+ Object parent;
+
+ char *evdev;
int fd;
bool repeat;
bool grab_request;
@@ -139,9 +151,14 @@ struct InputLinux {
bool keydown[KEY_CNT];
int keycount;
int wheel;
+ bool initialized;
QTAILQ_ENTRY(InputLinux) next;
};
+struct InputLinuxClass {
+ ObjectClass parent_class;
+};
+
static QTAILQ_HEAD(, InputLinux) inputs = QTAILQ_HEAD_INITIALIZER(inputs);
static void input_linux_toggle_grab(InputLinux *il)
@@ -196,6 +213,13 @@ static void input_linux_event_keyboard(void *opaque)
*/
continue;
}
+ if (event.code >= KEY_CNT) {
+ /*
+ * Should not happen. But better safe than sorry,
+ * and we make Coverity happy too.
+ */
+ continue;
+ }
/* keep track of key state */
if (!il->keydown[event.code] && event.value) {
il->keydown[event.code] = true;
@@ -309,25 +333,21 @@ static void input_linux_event_mouse(void *opaque)
}
}
-int input_linux_init(void *opaque, QemuOpts *opts, Error **errp)
+static void input_linux_complete(UserCreatable *uc, Error **errp)
{
- InputLinux *il = g_new0(InputLinux, 1);
+ InputLinux *il = INPUT_LINUX(uc);
uint32_t evtmap;
int rc, ver;
- il->evdev = qemu_opt_get(opts, "evdev");
- il->grab_all = qemu_opt_get_bool(opts, "grab-all", false);
- il->repeat = qemu_opt_get_bool(opts, "repeat", false);
-
if (!il->evdev) {
error_setg(errp, "no input device specified");
- goto err_free;
+ return;
}
il->fd = open(il->evdev, O_RDWR);
if (il->fd < 0) {
error_setg_file_open(errp, errno, il->evdev);
- goto err_free;
+ return;
}
qemu_set_nonblock(il->fd);
@@ -356,36 +376,111 @@ int input_linux_init(void *opaque, QemuOpts *opts, Error **errp)
}
input_linux_toggle_grab(il);
QTAILQ_INSERT_TAIL(&inputs, il, next);
- return 0;
+ il->initialized = true;
+ return;
err_close:
close(il->fd);
-err_free:
- g_free(il);
- return -1;
+ return;
+}
+
+static void input_linux_instance_finalize(Object *obj)
+{
+ InputLinux *il = INPUT_LINUX(obj);
+
+ if (il->initialized) {
+ QTAILQ_REMOVE(&inputs, il, next);
+ close(il->fd);
+ }
+ g_free(il->evdev);
+}
+
+static char *input_linux_get_evdev(Object *obj, Error **errp)
+{
+ InputLinux *il = INPUT_LINUX(obj);
+
+ return g_strdup(il->evdev);
+}
+
+static void input_linux_set_evdev(Object *obj, const char *value,
+ Error **errp)
+{
+ InputLinux *il = INPUT_LINUX(obj);
+
+ if (il->evdev) {
+ error_setg(errp, "evdev property already set");
+ return;
+ }
+ il->evdev = g_strdup(value);
}
-static QemuOptsList qemu_input_linux_opts = {
- .name = "input-linux",
- .head = QTAILQ_HEAD_INITIALIZER(qemu_input_linux_opts.head),
- .implied_opt_name = "evdev",
- .desc = {
- {
- .name = "evdev",
- .type = QEMU_OPT_STRING,
- },{
- .name = "grab-all",
- .type = QEMU_OPT_BOOL,
- },{
- .name = "repeat",
- .type = QEMU_OPT_BOOL,
- },
- { /* end of list */ }
- },
+static bool input_linux_get_grab_all(Object *obj, Error **errp)
+{
+ InputLinux *il = INPUT_LINUX(obj);
+
+ return il->grab_all;
+}
+
+static void input_linux_set_grab_all(Object *obj, bool value,
+ Error **errp)
+{
+ InputLinux *il = INPUT_LINUX(obj);
+
+ il->grab_all = value;
+}
+
+static bool input_linux_get_repeat(Object *obj, Error **errp)
+{
+ InputLinux *il = INPUT_LINUX(obj);
+
+ return il->repeat;
+}
+
+static void input_linux_set_repeat(Object *obj, bool value,
+ Error **errp)
+{
+ InputLinux *il = INPUT_LINUX(obj);
+
+ il->repeat = value;
+}
+
+static void input_linux_instance_init(Object *obj)
+{
+ object_property_add_str(obj, "evdev",
+ input_linux_get_evdev,
+ input_linux_set_evdev, NULL);
+ object_property_add_bool(obj, "grab_all",
+ input_linux_get_grab_all,
+ input_linux_set_grab_all, NULL);
+ object_property_add_bool(obj, "repeat",
+ input_linux_get_repeat,
+ input_linux_set_repeat, NULL);
+}
+
+static void input_linux_class_init(ObjectClass *oc, void *data)
+{
+ UserCreatableClass *ucc = USER_CREATABLE_CLASS(oc);
+
+ ucc->complete = input_linux_complete;
+}
+
+static const TypeInfo input_linux_info = {
+ .name = TYPE_INPUT_LINUX,
+ .parent = TYPE_OBJECT,
+ .class_size = sizeof(InputLinuxClass),
+ .class_init = input_linux_class_init,
+ .instance_size = sizeof(InputLinux),
+ .instance_init = input_linux_instance_init,
+ .instance_finalize = input_linux_instance_finalize,
+ .interfaces = (InterfaceInfo[]) {
+ { TYPE_USER_CREATABLE },
+ { }
+ }
};
-static void input_linux_register_config(void)
+static void register_types(void)
{
- qemu_add_opts(&qemu_input_linux_opts);
+ type_register_static(&input_linux_info);
}
-opts_init(input_linux_register_config);
+
+type_init(register_types);
diff --git a/ui/spice-core.c b/ui/spice-core.c
index e1179258d0..61db3c18b3 100644
--- a/ui/spice-core.c
+++ b/ui/spice-core.c
@@ -828,6 +828,11 @@ void qemu_spice_init(void)
#ifdef HAVE_SPICE_GL
if (qemu_opt_get_bool(opts, "gl", 0)) {
+ if ((port != 0) || (tls_port != 0)) {
+ error_report("SPICE GL support is local-only for now and "
+ "incompatible with -spice port/tls-port");
+ exit(1);
+ }
if (egl_rendernode_init() == 0) {
display_opengl = 1;
}