From 38bb7a001bdbf5f8c6256e75cbe7dd479a80d0a2 Mon Sep 17 00:00:00 2001 From: Benjamin Franzke Date: Mon, 9 Apr 2012 18:14:58 +0200 Subject: [PATCH] evdev: Disable the udev monitor when another vt becomes active We dont want to receive hotplug events while being inactive. When getting back active we enumerate all devices and would end up with two sources for one device that may be hotplugged in the mean time. --- src/evdev.c | 25 +++++++++++++++++++------ src/evdev.h | 6 ++++++ 2 files changed, 25 insertions(+), 6 deletions(-) diff --git a/src/evdev.c b/src/evdev.c index 9967e52..d328264 100644 --- a/src/evdev.c +++ b/src/evdev.c @@ -632,9 +632,10 @@ evdev_udev_handler(int fd, uint32_t mask, void *data) return 0; } -static int -evdev_config_udev_monitor(struct udev *udev, struct evdev_input *master) +int +evdev_enable_udev_monitor(struct udev *udev, struct weston_input_device *input_base) { + struct evdev_input *master = (struct evdev_input *) input_base; struct wl_event_loop *loop; struct weston_compositor *c = master->base.compositor; int fd; @@ -668,6 +669,20 @@ evdev_config_udev_monitor(struct udev *udev, struct evdev_input *master) } void +evdev_disable_udev_monitor(struct weston_input_device *input_base) +{ + struct evdev_input *input = (struct evdev_input *) input_base; + + if (!input->udev_monitor) + return; + + udev_monitor_unref(input->udev_monitor); + input->udev_monitor = NULL; + wl_event_source_remove(input->udev_monitor_source); + input->udev_monitor_source = NULL; +} + +void evdev_input_create(struct weston_compositor *c, struct udev *udev, const char *seat) { @@ -682,7 +697,7 @@ evdev_input_create(struct weston_compositor *c, struct udev *udev, wl_list_init(&input->devices_list); input->seat_id = strdup(seat); - if (!evdev_config_udev_monitor(udev, input)) { + if (!evdev_enable_udev_monitor(udev, &input->base)) { free(input->seat_id); free(input); return; @@ -709,9 +724,7 @@ evdev_input_destroy(struct weston_input_device *input_base) struct evdev_input *input = (struct evdev_input *) input_base; evdev_remove_devices(input_base); - - udev_monitor_unref(input->udev_monitor); - wl_event_source_remove(input->udev_monitor_source); + evdev_disable_udev_monitor(&input->base); wl_list_remove(&input->base.link); free(input->seat_id); diff --git a/src/evdev.h b/src/evdev.h index af5f6d9..4e29814 100644 --- a/src/evdev.h +++ b/src/evdev.h @@ -33,3 +33,9 @@ evdev_input_create(struct weston_compositor *c, struct udev *udev, void evdev_input_destroy(struct weston_input_device *input_base); + +int +evdev_enable_udev_monitor(struct udev *udev, struct weston_input_device *input_base); + +void +evdev_disable_udev_monitor(struct weston_input_device *input_base); -- 2.7.4