From: Pekka Paalanen <ppaalanen@gmail.com>
To: Simon Ser <contact@emersion.fr>
Cc: Dennis Filder <d.filder@web.de>,
dri-devel@lists.freedesktop.org,
Hans de Goede <hdegoede@redhat.com>
Subject: Re: Handling DRM master transitions cooperatively
Date: Tue, 7 Sep 2021 15:52:14 +0300 [thread overview]
Message-ID: <20210907155214.285b3561@eldfell> (raw)
In-Reply-To: <Kh2hM9odvPcbqNUxhSEwQbnUCLt6nyrAzGhxnQ0JO1CcAzI6y31enf8_AmgAynFxWo3E1QUlW1m8UvKPDAq0gFLQi2CF1b3FFNMRfiUzOAI=@emersion.fr>
[-- Attachment #1: Type: text/plain, Size: 1381 bytes --]
On Tue, 07 Sep 2021 10:19:03 +0000
Simon Ser <contact@emersion.fr> wrote:
> FWIW, I've just hit a case where a compositor leaves a "rotation" KMS
> prop set behind, then Xorg tries to startup and fails because it doesn't
> reset this prop. So none of this is theoretical.
>
> I still think a "reset all KMS props to an arbitrary default value" flag
> in drmModeAtomicCommit is the best way forward. I'm not sure a user-space
> protocol would help too much.
Hi Simon,
for the "reset KMS state" problem, sure. Thanks for confirming the
problem, too.
The hand-off problem does need userspace protocol though, so that the
two parties can negotiate what part of KMS state can be inherited by
the receiver and who will do the animation from the first to the second
state in case you want to avoid abrupt changes. It would also be useful
for a cross-fade as a perhaps more flexible way than the current "leak
an FB, let the next KMS client scrape it via ioctls and copy it so it
can be textured from".
Userspace protocol is also useful for starting the next KMS client
first and handing off only later once it's actually running. I'm not
sure if that is already possible with the session switching stuff, but
I have a feeling it might be fragile or miss pieces like the next KMS
client signalling ready before actually switching to it.
Thanks,
pq
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2021-09-07 12:52 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-03 19:08 Handling DRM master transitions cooperatively Dennis Filder
2021-09-07 10:07 ` Pekka Paalanen
2021-09-07 10:19 ` Simon Ser
2021-09-07 12:52 ` Pekka Paalanen [this message]
2021-09-07 15:52 ` Sebastian Wick
2021-09-08 16:21 ` Dennis Filder
2021-09-09 8:20 ` Pekka Paalanen
2021-09-08 9:51 ` Simon Ser
2021-09-08 11:13 ` Pekka Paalanen
2021-09-08 16:14 ` Dennis Filder
2021-09-07 12:42 ` Hans de Goede
2021-09-08 7:36 ` Pekka Paalanen
2021-09-08 7:49 ` Hans de Goede
2021-09-08 16:27 ` Daniel Vetter
2021-09-09 7:37 ` Pekka Paalanen
2021-09-14 13:45 ` Daniel Vetter
2021-09-22 8:56 ` Pekka Paalanen
2021-09-22 9:21 ` Hans de Goede
2021-09-23 8:23 ` Pekka Paalanen
2021-09-23 8:52 ` Hans de Goede
2021-09-30 9:26 ` Daniel Vetter
2021-10-01 16:33 ` Simon Ser
2021-10-02 17:27 ` Hans de Goede
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=20210907155214.285b3561@eldfell \
--to=ppaalanen@gmail.com \
--cc=contact@emersion.fr \
--cc=d.filder@web.de \
--cc=dri-devel@lists.freedesktop.org \
--cc=hdegoede@redhat.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 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.