From: Hans de Goede <hdegoede@redhat.com>
To: Pekka Paalanen <ppaalanen@gmail.com>
Cc: "Jonas Ådahl" <jadahl@redhat.com>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>
Subject: Re: Atomic KMS API lacks the ability to set cursor hot-spot coordinates
Date: Thu, 19 Mar 2020 19:18:39 +0100 [thread overview]
Message-ID: <217ab553-9c16-329d-bbbd-02067a2ccd6e@redhat.com> (raw)
In-Reply-To: <20200319171633.2ee4afa4@eldfell.localdomain>
Hi,
On 3/19/20 4:16 PM, Pekka Paalanen wrote:
> On Thu, 19 Mar 2020 15:30:03 +0100
> Hans de Goede <hdegoede@redhat.com> wrote:
<snip>
>>> The only way to fix that is to stop Weston from putting non-cursor
>>> content on the cursor plane.
>>
>> Correct.
>
> Is that something that should be done?
>
> If the hotspot property also had a "disabled" value, then Weston could
> set the hotspot to disabled when it is using the cursor plane for
> non-cursor content and not lose the feature. And of course set hotspot
> correctly when it in fact is a cursor (but for what input?).
I believe cursor planes in the affected virtual gfx-cards do not
really have a mode where they can actually be used as a generic overlay
plane, certainly not in a useful manner (if anything works it will all
be software emulation), implementing a hotspot disabled mode would be
tricky and this would needs to be duplicated in all virtual-gfx cards
kms drivers.
If I understood Daniel's proposal for how to deal with this properly,
then only cursor planes which actually need them would get the new
hotspot x/y properties. If we do that then Weston could use the
presence of the hotspot x/y properties to detect if it is dealing
with a proper hw plane which can also be used as a generic
plane; or a virtual-gfx cards cursor-plane, and then just not
bother with trying to use the plane as a generic hw plane.
Would that work?
<snip>
> I think one of the major reasons why Wayland pointer relative motion
> and confinement extensions were designed was VM- etc. viewers, and of
> course games.
ack.
>>> What if display servers stop using the cursor plane completely, because
>>> people may hit a case where a VM-viewer makes the wrong assumption about
>>> which input device is associated to which cursor plane inside the VM?
>>
>> The confine + warp trick is typically the default mode and only
>> if the guest indicates through e.g. a guest-agent process that
>> seamless mode is supported then seamless mode is enabled.
>>
>> IOW the VM is careful to not enable it when it might not work.
>
> How would the guest-agent know? Does it check that there is literally
> only one pointer input device and that comes from the VM-viewer?
> Or does it limit seamless to white-listed display servers perhaps?
This varies between different solutions. E.g. the spice vdagent has
a desktop component which is started with a /etc/xdg/autostart/foo.desktop
file, assuming that it will work with any desktops honoring
/etc/xdg/autostart.
VirtualBox OTOH assumes it can use seamless mode as soon as
the vboxvideo driver loads inside the guest.
Regards,
Hans
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2020-03-19 18:18 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-18 14:28 Atomic KMS API lacks the ability to set cursor hot-spot coordinates Hans de Goede
2020-03-18 14:38 ` Simon Ser
2020-03-18 15:04 ` Hans de Goede
2020-03-18 15:22 ` Simon Ser
2020-03-19 11:35 ` Michel Dänzer
2020-03-19 11:52 ` Hans de Goede
2020-03-19 12:54 ` Pekka Paalanen
2020-03-19 13:51 ` Michel Dänzer
2020-03-19 14:48 ` Pekka Paalanen
2020-03-19 15:01 ` Hans de Goede
2020-03-19 9:57 ` Pekka Paalanen
2020-03-19 10:00 ` Simon Ser
2020-03-18 14:39 ` Hans de Goede
2020-03-18 15:09 ` Daniel Vetter
2020-03-18 22:20 ` Hans de Goede
2020-03-19 10:00 ` Pekka Paalanen
2020-03-19 11:49 ` Hans de Goede
2020-03-19 12:58 ` Pekka Paalanen
2020-03-19 13:16 ` Daniel Vetter
2020-03-19 13:24 ` Simon Ser
2020-03-19 14:30 ` Hans de Goede
2020-03-19 15:16 ` Pekka Paalanen
2020-03-19 18:18 ` Hans de Goede [this message]
2020-03-19 20:14 ` Simon Ser
2020-03-19 20:49 ` Hans de Goede
2020-03-19 21:07 ` Simon Ser
2020-03-19 22:57 ` Thomas Hellström (VMware)
2020-03-20 9:13 ` Pekka Paalanen
2020-03-20 10:59 ` Thomas Hellström (VMware)
2020-03-20 11:27 ` Simon Ser
2020-03-20 11:47 ` Thomas Hellström (VMware)
2020-03-21 8:56 ` Thomas Hellström (VMware)
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=217ab553-9c16-329d-bbbd-02067a2ccd6e@redhat.com \
--to=hdegoede@redhat.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=jadahl@redhat.com \
--cc=ppaalanen@gmail.com \
/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