[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] msmouse: add MouseState, unregister handler
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] msmouse: add MouseState, unregister handler on close |
Date: |
Tue, 14 Jun 2016 15:38:02 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.0 |
On 14/06/2016 14:51, Gerd Hoffmann wrote:
> Add struct to track serial mouse state. Store mouse event handler
> there. Unregister properly on chardev close.
>
> Signed-off-by: Gerd Hoffmann <address@hidden>
> ---
> backends/msmouse.c | 22 +++++++++++++++++-----
> 1 file changed, 17 insertions(+), 5 deletions(-)
>
> diff --git a/backends/msmouse.c b/backends/msmouse.c
> index 8dea5a1..731784f 100644
> --- a/backends/msmouse.c
> +++ b/backends/msmouse.c
> @@ -29,6 +29,11 @@
> #define MSMOUSE_LO6(n) ((n) & 0x3f)
> #define MSMOUSE_HI2(n) (((n) & 0xc0) >> 6)
>
> +typedef struct {
> + CharDriverState *chr;
> + QEMUPutMouseEntry *entry;
> +} MouseState;
> +
> static void msmouse_event(void *opaque,
> int dx, int dy, int dz, int buttons_state)
> {
> @@ -60,7 +65,11 @@ static int msmouse_chr_write (struct CharDriverState *s,
> const uint8_t *buf, int
>
> static void msmouse_chr_close (struct CharDriverState *chr)
> {
> - g_free (chr);
> + MouseState *mouse = chr->opaque;
> +
> + qemu_remove_mouse_event_handler(mouse->entry);
> + g_free(mouse);
> + g_free(chr);
> }
>
> static CharDriverState *qemu_chr_open_msmouse(const char *id,
> @@ -69,17 +78,20 @@ static CharDriverState *qemu_chr_open_msmouse(const char
> *id,
> Error **errp)
> {
> ChardevCommon *common = backend->u.msmouse.data;
> + MouseState *mouse;
> CharDriverState *chr;
>
> chr = qemu_chr_alloc(common, errp);
> - if (!chr) {
> - return NULL;
> - }
> chr->chr_write = msmouse_chr_write;
> chr->chr_close = msmouse_chr_close;
> chr->explicit_be_open = true;
>
> - qemu_add_mouse_event_handler(msmouse_event, chr, 0, "QEMU Microsoft
> Mouse");
> + mouse = g_new0(MouseState, 1);
> + mouse->entry = qemu_add_mouse_event_handler(msmouse_event, chr, 0,
> + "QEMU Microsoft Mouse");
> +
> + mouse->chr = chr;
> + chr->opaque = mouse;
>
> return chr;
> }
>
Both patches
Acked-by: Paolo Bonzini <address@hidden>
Thanks!
Paolo