qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Daniel Henrique Barboza <danielhb413@gmail.com>
To: Mario <ml@kernelobjects.org>,
	qemu-devel@nongnu.org, "Daniel P. Berrange" <berrange@redhat.com>
Subject: Re: [Qemu-devel] Request for comment - dynamic VNC keyboard mapping
Date: Thu, 9 May 2019 15:02:58 -0300	[thread overview]
Message-ID: <2fb7615e-0899-665c-e7e9-71aa5f9b6648@gmail.com> (raw)
In-Reply-To: <8C02D03E-96CD-414F-BC2C-C59972B26F94@kernelobjects.org>

Hi Mario,

Sorry for the delayed answer.  As Daniel already said, any solution based
on how "-k <keymap>" works are too daunting to be maintained - there
are a lot of known keymaps to be aware of.

There's also the case of changing keymaps during the same session.
Speaking from experience: my laptop has a regular en-us keyboard, but
I also use it with an external Brazilian keyboard (which has the "Ç" key and
other punctuation keys). I happen to use some VMs in this hardware too.
It is not uncommon in my usage, in the same VNC session, a change in the
current keyboard layout en_us -> pt_br back and forth.

This is only possible because gtk-vnc does not care about my layout, but for
the keycodes being pressed. In my case, instead of sending the "Ç" symbol,
it would send the keycode of the 11th keyboard key located at the forth 
line.
The translation of keycode to keysym happens up to the OS layer.

gtk-vnc implements the RFB scancode extension that allows for the keycode to
be sent to VNC. This implementation isn't complicated to pull it off, as 
Daniel
also mentioned in his reply. Many VNC clients already supports it. If 
you want
something to start from, I've implemented this extension in a browser VNC
client called noVNC a few years ago and I talked about it here:

https://danielhb.github.io/article/2019/05/06/noVNC-QEMU-RFB.html

You may also want to check Daniel's article about all this scancode/keycode/
keysym theory here:

https://www.berrange.com/tags/rfb/



Thanks,


DHB

On 5/3/19 8:47 AM, Mario wrote:
> Hi all,
>
> I have a question related to the VNC server keyboard settings. Currently the user of qemu has to decide before VM startup which language is used for VNC keyboard mapping. If no keyboard is configured, the en-us keyboard will be loaded for keysym to scancode conversion. A later reconfiguration of the keyboard is not possible at present.
>
> So here is my idea:
>
> The VNC Display context stores a list of loaded keyboards (like vs->vd->kbds). If no keyboard was specified, at least the default will be added to the list ("en-us"?). If a client connects, a copy of the (latest configured) pointer will be stored within the VncState structure. For any keys sent by this client the own keyboard reference will be used.
>
> As (in theory) every VNC client may have a differen keyboard layout, in my point of view it makes sense to attach the keyboard used for keysym2scancode() to the client context rather to the VNC server context. However as (most likely) all clients do have the same keyboard mapping it would be an overhead to parse and store the mapping list each time the client connects.
>
> Now in my idea two mechanisms may change the behavior of the client keyboard.
>
> 1.) a qmp command was sent, that sets /configures for a particular/default keyboard.
> 2.) within the VNC protocol a "VNC_MSG_CLIENT_QEMU/VNC_MSG_CLIENT_QEMU_EXT_KEYBOARD_LAYOUT" message arrives
>
> If the new keyboard was not already loaded and stored inside vs->vd->kbds , qemu initializes the new mapping. The affected VNC client (VncState) again stores the reference only.
>
> This enables switching the keyboard layout during runtime as well as using multiple different keyboard layouts at atime.
>
> Together with a corresponding patch I would also add a patch for libvncclient to have a reference implmentation for the integrated keyboard layout switching.
>
> What are your thoughts about this kind of improvement?
>
> Thanks,
> Mario



      parent reply	other threads:[~2019-05-09 18:04 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-03 11:47 [Qemu-devel] Request for comment - dynamic VNC keyboard mapping Mario
2019-05-03 11:47 ` Mario
2019-05-03 12:08 ` Daniel P. Berrangé
2019-05-03 12:08   ` Daniel P. Berrangé
2019-05-09 18:02 ` Daniel Henrique Barboza [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2fb7615e-0899-665c-e7e9-71aa5f9b6648@gmail.com \
    --to=danielhb413@gmail.com \
    --cc=berrange@redhat.com \
    --cc=ml@kernelobjects.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).