Re: [Qemu-devel] [PATCH] input: limit kbd queue depth

[Gerd Hoffmann]

  Hi,

> Drive-by comment, feel free to ignore: I'd be tempted to lower the limit
> to something comparable to actual hardware, then dumb down the queue to
> an array.

Well, the point of this queuing feature is to make automated tests a bit
easier, i.e. a script passes something longish like a install URL to
qemu, and qemu forwards it to the guest with small delays so the
keyboard hardware buffers don't overflow.

So I left he limit relatively high.  Even 1024 isn't as much as it
sounds, each typed char is at least four queue items (keydown, delay,
keyup, delay), in case modifiers are needed even more.  So this gives
buffering room for roughly 150-250 chars typed.

Also lowering the limit to something hardware-comparable is rather
pointless, we have those buffers in the hardware emulation, so we could
just drop the queuing support at input layer altogether.  I don't think
this is a good idea though.

cheers,
  Gerd