From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keith Packard Subject: Re: Proposal for RandR version 1.6, Leases and EDID-based output grabs Date: Tue, 04 Apr 2017 08:53:45 -0700 Message-ID: <867f30gody.fsf@hiro.keithp.com> References: <86fuhrka4t.fsf@hiro.keithp.com> <20170402154302.zd7nmqf7vtcvgssu@phenom.ffwll.local> <86y3viinti.fsf@hiro.keithp.com> <20170403074528.c7vwoi3mg7yeojdr@phenom.ffwll.local> <86d1ctigu9.fsf@hiro.keithp.com> <20170403220749.5ujhdzuy6dnikwry@phenom.ffwll.local> <86h925gl6u.fsf@hiro.keithp.com> <20170404070242.rphtgg4yopek2sf7@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0311620845==" Return-path: In-Reply-To: <20170404070242.rphtgg4yopek2sf7-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: xorg-devel-bounces-go0+a7rfsptAfugRpC6u6w@public.gmane.org Sender: "xorg-devel" Cc: xorg-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, Daniel Vetter List-Id: dri-devel@lists.freedesktop.org --===============0311620845== Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha256; protocol="application/pgp-signature" --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Daniel Vetter writes: > The multi-seat thing sounds like vapourware, I think we should care about > the vr use-case for now, and only that one. Ok, I can live with that, even if I like the idea of a slightly more general solution. > For VR itself I'd go as far as saying that probably our "create lease" > ioctl should have only the semantics we need to pass one crtc+primary > plane for pageflipping in a VR compositor, expressed in a flag. Yeah, we can't express planes through X anyways. I'll leave the kernel API with multiple planes as that's actually simpler than having it validate that only a single plane is in the lease. > All the details about additional corner cases are just so unclear to > me (and there's not even a clear use case that will materialize) that > I don't think having the uapi is worth it. Too close to the "I'll > regret this immediately" bucket :-) Removing the 'ChangeLease' ioctl eliminates a bunch of complexity in the code, and means I don't even have to think about sending events. I'll also go ahead and remove the ability to hide resources from the lessor. Thanks, as always, for your thoughtful review. ps -- Any thoughts on whether the X request should include the mode to use? Doing that would let us restrict the lessee from setting modes, and avoid potential resource issues with the window system. However, it would also require providing a scanout buffer in the request. =2D-=20 =2Dkeith --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEw4O3eCVWE9/bQJ2R2yIaaQAAABEFAljjwYoACgkQ2yIaaQAA ABGHtw//S03Br2a14OEYX9tMAaMlgcHsc3+F7PRYXmbyAB/jUT8kuh29ibj49MMT 6cQNs7eOLnRiNBLQQ8dJTxMhWNiJLpPDkAGBJydehO5mLezMwMy4Z+j8pcm0lP2j S9/5xpYYf0Dc82W2bIpNE1vOl6Qn4y8AEIkTE/oMwhIhquG/HfS8jXw5iRdQxP2m m6flnoHRt80pLd4aEm8sT3DZ8N5wgR0c6evbsiuOFrJCuhEsyLJXPslLkaC5KBLc ixFqpnN0W2TN1Vo2tlIeKw4QhA/nU8A6A+KkF4GsN9VFL+5dUz/mJhAQtbIuuNy2 Kqlc6bLhAQGEMxUiBi2RR5l0mehwQXnpdy83FkJPwYOPrObn4wOQh1ufQXFXWPnm S1lbPIeZS2rWD39ZJmfQth4DUQE49BMyLj9AfHD6cqXjSHWoWRUYeZNgeJLHtHb1 VKuSSDRtH5TvpOVRSaBjN1DRVIw12LXGffApSHpQnwBWRdJhpKtaWYH8+mbQ13ZS foRBP+PyISvw9kmbn4R3DJJZJxkqQOpwuILc+gLAckRAjH+iajlD6by9qNOloRjU 6hMQ1b0AbsCjiqhU7Xn79Nludf8LeYLiagrF0wgjMvy1BCumppksK27VPI5IN3Q/ HGVBxlu1Wcsg0AAmD1IEMDHWSmV2XlrryGeZnPN4VMZtrJdw4Nk= =oymu -----END PGP SIGNATURE----- --=-=-=-- --===============0311620845== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KeG9yZy1kZXZl bEBsaXN0cy54Lm9yZzogWC5PcmcgZGV2ZWxvcG1lbnQKQXJjaGl2ZXM6IGh0dHA6Ly9saXN0cy54 Lm9yZy9hcmNoaXZlcy94b3JnLWRldmVsCkluZm86IGh0dHBzOi8vbGlzdHMueC5vcmcvbWFpbG1h bi9saXN0aW5mby94b3JnLWRldmVs --===============0311620845==--