[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 4/8] msmouse: send short messages if possible.
From: |
Gerd Hoffmann |
Subject: |
[Qemu-devel] [PULL 4/8] msmouse: send short messages if possible. |
Date: |
Tue, 12 Jul 2016 10:21:39 +0200 |
Keep track of button changes. Send the extended 4-byte messages for
three button mice only in case we have something to report for the
middle button. Use the short 3-byte messages (original protocol for
two-button microsoft mouse) otherwise.
Signed-off-by: Gerd Hoffmann <address@hidden>
Message-id: address@hidden
---
backends/msmouse.c | 20 ++++++++++++--------
1 file changed, 12 insertions(+), 8 deletions(-)
diff --git a/backends/msmouse.c b/backends/msmouse.c
index b1e1bea..aeb9055 100644
--- a/backends/msmouse.c
+++ b/backends/msmouse.c
@@ -35,6 +35,7 @@ typedef struct {
QemuInputHandlerState *hs;
int axis[INPUT_AXIS__MAX];
bool btns[INPUT_BUTTON__MAX];
+ bool btnc[INPUT_BUTTON__MAX];
uint8_t outbuf[32];
int outlen;
} MouseState;
@@ -62,7 +63,7 @@ static void msmouse_chr_accept_input(CharDriverState *chr)
static void msmouse_queue_event(MouseState *mouse)
{
unsigned char bytes[4] = { 0x40, 0x00, 0x00, 0x00 };
- int dx, dy;
+ int dx, dy, count = 3;
dx = mouse->axis[INPUT_AXIS_X];
mouse->axis[INPUT_AXIS_X] = 0;
@@ -78,14 +79,16 @@ static void msmouse_queue_event(MouseState *mouse)
/* Buttons */
bytes[0] |= (mouse->btns[INPUT_BUTTON_LEFT] ? 0x20 : 0x00);
bytes[0] |= (mouse->btns[INPUT_BUTTON_RIGHT] ? 0x10 : 0x00);
- bytes[3] |= (mouse->btns[INPUT_BUTTON_MIDDLE] ? 0x20 : 0x00);
+ if (mouse->btns[INPUT_BUTTON_MIDDLE] ||
+ mouse->btnc[INPUT_BUTTON_MIDDLE]) {
+ bytes[3] |= (mouse->btns[INPUT_BUTTON_MIDDLE] ? 0x20 : 0x00);
+ mouse->btnc[INPUT_BUTTON_MIDDLE] = false;
+ count = 4;
+ }
- if (mouse->outlen <= sizeof(mouse->outbuf) - 4) {
- /* We always send the packet of, so that we do not have to keep track
- of previous state of the middle button. This can potentially confuse
- some very old drivers for two button mice though. */
- memcpy(mouse->outbuf + mouse->outlen, bytes, 4);
- mouse->outlen += 4;
+ if (mouse->outlen <= sizeof(mouse->outbuf) - count) {
+ memcpy(mouse->outbuf + mouse->outlen, bytes, count);
+ mouse->outlen += count;
} else {
/* queue full -> drop event */
}
@@ -107,6 +110,7 @@ static void msmouse_input_event(DeviceState *dev,
QemuConsole *src,
case INPUT_EVENT_KIND_BTN:
btn = evt->u.btn.data;
mouse->btns[btn->button] = btn->down;
+ mouse->btnc[btn->button] = true;
break;
default:
--
1.8.3.1
- [Qemu-devel] [PULL 0/8] input patch queue, Gerd Hoffmann, 2016/07/12
- [Qemu-devel] [PULL 4/8] msmouse: send short messages if possible.,
Gerd Hoffmann <=
- [Qemu-devel] [PULL 2/8] msmouse: fix buffer handling, Gerd Hoffmann, 2016/07/12
- [Qemu-devel] [PULL 3/8] msmouse: switch to new input interface, Gerd Hoffmann, 2016/07/12
- [Qemu-devel] [PULL 1/8] msmouse: add MouseState, unregister handler on close, Gerd Hoffmann, 2016/07/12
- [Qemu-devel] [PULL 8/8] input-linux: better capability checks, merge input_linux_event_{mouse, keyboard}, Gerd Hoffmann, 2016/07/12
- [Qemu-devel] [PULL 7/8] input-linux: factor out input_linux_handle_keyboard, Gerd Hoffmann, 2016/07/12
- [Qemu-devel] [PULL 5/8] input: add trace events for full queues, Gerd Hoffmann, 2016/07/12
- [Qemu-devel] [PULL 6/8] input-linux: factor out input_linux_handle_mouse, Gerd Hoffmann, 2016/07/12
- Re: [Qemu-devel] [PULL 0/8] input patch queue, Peter Maydell, 2016/07/12