On Mon, Oct 19, 2015 at 02:33:56PM -0500, Jeremy White wrote: > Otherwise, you constantly spin loop, getting a return code of 0, > and pegging CPU usage at 100%. > > Signed-off-by: Jeremy White <[email protected]> > --- > src/spiceqxl_uinput.c | 13 ++++++++++++- > src/spiceqxl_uinput.h | 1 + > src/spiceqxl_vdagent.c | 2 ++ > 3 files changed, 15 insertions(+), 1 deletion(-) > > diff --git a/src/spiceqxl_uinput.c b/src/spiceqxl_uinput.c > index 1e61907..38ed2ed 100644 > --- a/src/spiceqxl_uinput.c > +++ b/src/spiceqxl_uinput.c > @@ -121,6 +121,17 @@ void spiceqxl_uinput_init(qxl_screen_t *qxl) > uinput_filename, strerror(errno)); > return; > } > - qxl->core->watch_add(uinput_fd, SPICE_WATCH_EVENT_READ, > spiceqxl_uinput_read_cb, qxl); > spice_server_set_agent_mouse(qxl->spice_server, 1); > } > + > +void spiceqxl_uinput_watch(qxl_screen_t *qxl, Bool on) > +{ > + static SpiceWatch *handle = NULL;
I would store this field in qxl_screen_t.
Looks good otherwise, ACK
Christophe
> +
> + if (on)
> + handle = qxl->core->watch_add(uinput_fd, SPICE_WATCH_EVENT_READ,
> spiceqxl_uinput_read_cb, qxl);
> + else if (handle) {
> + qxl->core->watch_remove(handle);
> + handle = NULL;
> + }
> +}
> diff --git a/src/spiceqxl_uinput.h b/src/spiceqxl_uinput.h
> index 59c704c..33715a0 100644
> --- a/src/spiceqxl_uinput.h
> +++ b/src/spiceqxl_uinput.h
> @@ -4,5 +4,6 @@
> #include "qxl.h"
>
> void spiceqxl_uinput_init(qxl_screen_t *qxl);
> +void spiceqxl_uinput_watch(qxl_screen_t *qxl, Bool on);
>
> #endif
> diff --git a/src/spiceqxl_vdagent.c b/src/spiceqxl_vdagent.c
> index 315ae53..a62c1e6 100644
> --- a/src/spiceqxl_vdagent.c
> +++ b/src/spiceqxl_vdagent.c
> @@ -62,6 +62,7 @@ static int vmc_read(SpiceCharDeviceInstance *sin, uint8_t
> *buf, int len)
> vdagent_sin.qxl->core->watch_remove(virtio_client_watch);
> virtio_client_watch = NULL;
> spice_server_remove_interface(&vdagent_sin.base.base);
> + spiceqxl_uinput_watch(vdagent_sin.qxl, FALSE);
> }
> return nbytes;
> }
> @@ -123,6 +124,7 @@ static void on_accept(int fd, int event, void *opaque)
> /* TODO - SPICE_WATCH_EVENT_WRITE */, on_read_available, qxl);
>
> spice_server_add_interface(qxl->spice_server, &vdagent_sin.base.base);
> + spiceqxl_uinput_watch(qxl, TRUE);
>
> return;
>
> --
> 2.1.4
>
> _______________________________________________
> Spice-devel mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/spice-devel
