From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from [140.186.70.92] (port=38896 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1PKy9j-0000Pq-SH
	for qemu-devel@nongnu.org; Tue, 23 Nov 2010 14:04:07 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <weil@mail.berlios.de>) id 1PKy9R-0002tp-Qh
	for qemu-devel@nongnu.org; Tue, 23 Nov 2010 14:03:48 -0500
Received: from moutng.kundenserver.de ([212.227.17.9]:60393)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <weil@mail.berlios.de>) id 1PKy9R-0002sA-Fa
	for qemu-devel@nongnu.org; Tue, 23 Nov 2010 14:03:37 -0500
Message-ID: <4CEC1004.4090105@mail.berlios.de>
Date: Tue, 23 Nov 2010 20:03:32 +0100
From: Stefan Weil <weil@mail.berlios.de>
MIME-Version: 1.0
Subject: Re: [Qemu-devel] [Bug 427612] Re: kvm sends caps lock key up event
	twice
References: <20090910235343.14184.29717.malonedeb@gangotri.canonical.com>
	<20101123004254.824.65473.malone@gandwana.canonical.com>
In-Reply-To: <20101123004254.824.65473.malone@gandwana.canonical.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Bug 427612 <427612@bugs.launchpad.net>
Cc: Benjamin Drung <benjamin.drung@gmail.com>, qemu-devel@nongnu.org

Am 23.11.2010 01:42, schrieb Benjamin Drung:
> Attached the patch for qemu-kvm 0.13. This patch is tested on natty with
> qemu-kvm 0.13.0+noroms-0ubuntu7 and the German and NEO2 keyboard layout.
>
> ** Patch added: "caps-lock-key-up-event.patch"
> https://bugs.launchpad.net/ubuntu/+source/qemu-kvm/+bug/427612/+attachment/1742131/+files/caps-lock-key-up-event.patch
>
> ** Changed in: qemu
> Status: Invalid => New
>

The patch might fix part of the problem, but there remain more issues:

* SDL also sends an SDL_KEYUP event for caps lock when the
   environment variable SDL_DISABLE_LOCK_KEYS is set.
   This mode is very useful but currently unsupported by qemu/kvm.

* Num lock and caps lock are handled in a similar way by SDL.
   The patch only handles caps lock. Maybe this is less important
   because keyboard layouts which remap num lock are rare
   (I don't know any).

* The keyboard status LEDs and the qemu client's keyboard status
   can become unsynchronized if the input focus changes from qemu
   to other applications.

Regards,

Stefan Weil