[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 05/12] input: convert virtio-input-hid device
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH v5 05/12] input: convert virtio-input-hid device to keycodemapdb |
Date: |
Tue, 12 Sep 2017 16:04:52 +0200 |
Hi,
> From what I can see the guest OS will read this bitmap when it first
> probes the virtio device, and never updates it. If so, then is it
> actually a problem if we don't preserve the same map across save/
> restore/migrate ?
virtio-input config space size isn't fixed, instead it'll take up as
many bytes as actually needed. So, when adding keys it can happen that
the bitmap needed grows, therefore the config space grows. Not sure
whenever live migration trips over that, needs checking.
> If we added a key to the bitmap, we would potentially be sending
> a key to the guest that we hadn't previously declared support for.
> Either the guest OS will handle this normally, despite it not
> being in the declared bitmap, or it will silently drop the event.
> Neither behaviour seems problematic.
Yes, in theory the guest drivers should be robust enough.
We should check though whenever that is actually the case.
> If we did need to preserve ABI for this bitmap, then it seems we're
> going to create alot of work for ourselves, having to maintain
> many historical copies of the keycode maps tables, adding a new
> copy every time support for a new key is added in QEMU.
That's why I've suggested to not touch that for now, and once the dust
has settled and QKeyCode got all the keys still missing create a v2 of
the device, so we don't end up with many different versions.
cheers,
Gerd
- [Qemu-devel] [PATCH v5 01/12] ui: add keycodemapdb repository as a GIT submodule, (continued)
- [Qemu-devel] [PATCH v5 01/12] ui: add keycodemapdb repository as a GIT submodule, Daniel P. Berrange, 2017/09/12
- [Qemu-devel] [PATCH v5 03/12] ui: convert key events to QKeyCodes immediately, Daniel P. Berrange, 2017/09/12
- [Qemu-devel] [PATCH v5 04/12] ui: don't export qemu_input_event_new_key, Daniel P. Berrange, 2017/09/12
- [Qemu-devel] [PATCH v5 12/12] display: convert XenInput keyboard to keycodemapdb, Daniel P. Berrange, 2017/09/12
- [Qemu-devel] [PATCH v5 10/12] ui: convert the SDL2 frontend to keycodemapdb, Daniel P. Berrange, 2017/09/12
- [Qemu-devel] [PATCH v5 08/12] char: convert the escc device to keycodemapdb, Daniel P. Berrange, 2017/09/12
- [Qemu-devel] [PATCH v5 07/12] input: convert the adb device to keycodemapdb, Daniel P. Berrange, 2017/09/12
- [Qemu-devel] [PATCH v5 05/12] input: convert virtio-input-hid device to keycodemapdb, Daniel P. Berrange, 2017/09/12
- [Qemu-devel] [PATCH v5 09/12] ui: convert cocoa frontend to keycodemapdb, Daniel P. Berrange, 2017/09/12
- [Qemu-devel] [PATCH v5 06/12] input: convert ps2 device to keycodemapdb, Daniel P. Berrange, 2017/09/12
- [Qemu-devel] [PATCH v5 02/12] ui: convert common input code to keycodemapdb, Daniel P. Berrange, 2017/09/12
- [Qemu-devel] [PATCH v5 11/12] ui: convert GTK and SDL1 frontends to keycodemapdb, Daniel P. Berrange, 2017/09/12
- Re: [Qemu-devel] [PATCH v5 00/12] Convert over to use keycodemapdb, no-reply, 2017/09/12
- Re: [Qemu-devel] [PATCH v5 00/12] Convert over to use keycodemapdb, no-reply, 2017/09/12
- Re: [Qemu-devel] [PATCH v5 00/12] Convert over to use keycodemapdb, no-reply, 2017/09/12
- Re: [Qemu-devel] [PATCH v5 00/12] Convert over to use keycodemapdb, Gerd Hoffmann, 2017/09/12