From: John Baboval <baboval@spineless.org>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] console muti-head some more design input
Date: Wed, 20 Nov 2013 09:32:43 -0500 [thread overview]
Message-ID: <528CC80B.7080601@spineless.org> (raw)
In-Reply-To: <1384935158.2005.34.camel@nilsson.home.kraxel.org>
On 11/20/2013 03:12 AM, Gerd Hoffmann wrote:
> Hi,
>
>> I think you are only considering output here, for input we definitely
>> need some idea of screen layout, and this needs to be stored
>> somewhere.
> Oh yea, input. That needs quite some work for multihead / multiseat.
>
> I think we should *not* try to hack that into the ui. We should extend
> the input layer instead.
This would be a contrast to how a real system works. IMO, the UI is the
appropriate place for this sort of thing. A basic UI is going to be
sending relative events anyway.
I think a "seat" should be a UI construct as well.
> The functions used to notify qemu about mouse + keyboard events should
> get an additional parameter to indicate the source of the event. I
> think we can use a QemuConsole here.
>
> Then teach the input layer about seats, where a seat is a group of input
> devices (kbd, mouse, tablet) and a group of QemuConsoles. With x+y for
> each QemuConsole. The input layer will do the event routing: Translate
> coordinates, send to the correct device.
>
> I think initially we just can handle all existing QemuConsole and input
> devices implicitly as "seat 0". Stick x+y into QemuConsole for now, and
> have the input layer get it from there. At some point in the future we
> might want move this to a QemuSeat when we actually go multiseat.
>
> Bottom line: please do the coordinates math in input.c not sdl2.c so we
> don't run into roadblocks in the future.
>
> cheers,
> Gerd
>
>
>
>
next prev parent reply other threads:[~2013-11-20 14:33 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-19 6:24 [Qemu-devel] console muti-head some more design input Dave Airlie
2013-11-19 8:11 ` Gerd Hoffmann
2013-11-20 2:59 ` Dave Airlie
2013-11-20 4:06 ` John Baboval
2013-11-20 5:17 ` Dave Airlie
2013-11-20 5:18 ` Dave Airlie
2013-11-20 8:12 ` Gerd Hoffmann
2013-11-20 14:32 ` John Baboval [this message]
2013-11-20 15:14 ` Gerd Hoffmann
2013-11-20 15:49 ` John Baboval
2013-11-22 8:36 ` Gerd Hoffmann
2013-11-21 0:45 ` Dave Airlie
2013-11-22 8:41 ` Gerd Hoffmann
2013-11-27 4:29 ` Dave Airlie
2013-11-27 7:11 ` Gerd Hoffmann
2013-11-27 14:23 ` John Baboval
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=528CC80B.7080601@spineless.org \
--to=baboval@spineless.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.