Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-07-02 Thread Drew DeVault
On Fri Jun 28, 2019 at 10:37 AM Simon Ser wrote: > I'm now wondering if DRM leasing is that much different from xdg-shell > set_fullscreen requests. > > Is DRM leasing that much of a big deal regarding security? (Especially > if the compositor has a policy to lease only non-desktop outputs) > > O

Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-06-28 Thread Philipp Zabel
Hi Pekka, On Fri, 2019-06-28 at 12:58 +0300, Pekka Paalanen wrote: > On Fri, 28 Jun 2019 11:23:53 +0200 > Philipp Zabel wrote: > > On Thu, 2019-06-27 at 20:36 +, Simon Ser wrote: > > > What would be other use-cases for DRM leases? Probably fullscreen games? > > > > One use case I'd like wo

Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-06-28 Thread Simon Ser
On Thursday, June 27, 2019 11:36 PM, Simon Ser wrote: > > Previously there were some musings about the security considerations. > > This version of the protocol allows the compositor to consider the lease > > request in its own time, perhaps presenting the user with a dialog to > > consent to the

Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-06-28 Thread Simon Ser
On Friday, June 28, 2019 12:58 PM, Pekka Paalanen wrote: > On Fri, 28 Jun 2019 11:23:53 +0200 > Philipp Zabel p.za...@pengutronix.de wrote: > > > On Thu, 2019-06-27 at 20:36 +, Simon Ser wrote: > > > > > What would be other use-cases for DRM leases? Probably fullscreen games? > > > > One use c

Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-06-28 Thread Simon Ser
On Friday, June 28, 2019 12:23 PM, Philipp Zabel wrote: > > I'm not sure the client is in the best position to decide which connectors > > to > > pick here. > > The client may be the only one with enough knowledge to pick, see below. > > > Maybe it would be better for the client to say why it nee

Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-06-28 Thread Simon Ser
On Friday, June 28, 2019 10:51 AM, Marius Vlad wrote: > >> + > >> + > >> +The compositor may choose to advertise 0 or more connectors which > >> may be > >> +leased to clients, and will use this event to do so. This object > >> may be > >> +passed into a lease

Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-06-28 Thread Pekka Paalanen
On Fri, 28 Jun 2019 11:23:53 +0200 Philipp Zabel wrote: > On Thu, 2019-06-27 at 20:36 +, Simon Ser wrote: > > What would be other use-cases for DRM leases? Probably fullscreen games? > > One use case I'd like would be presentation software, where tagging the > projector as non-desktop befo

Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-06-28 Thread Philipp Zabel
On Thu, 2019-06-27 at 20:36 +, Simon Ser wrote: [...] > > + > > + > > +The compositor may choose to advertise 0 or more connectors which > > may be > > +leased to clients, and will use this event to do so. This object > > may be > > +passed into a lease reque

Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-06-28 Thread Pekka Paalanen
On Thu, 27 Jun 2019 15:46:27 -0400 Drew DeVault wrote: > From: Marius Vlad > > Simple protocol extension to manage DRM lease. Based on the work by Keith > Packard in [1], respectively [2]. > > [1] > https://cgit.freedesktop.org/mesa/drm/commit/?id=c4171535389d72e9135c9615cecd07b346fd6d7e > [2

Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-06-28 Thread Marius Vlad
Hi Drew, Thanks for taking the time to reboot the lease protocol! On 6/27/19 11:36 PM, Simon Ser wrote: > Hi, > > Thanks for your work! Here is a first round of comments. > > On Thursday, June 27, 2019 10:46 PM, Drew DeVault wrote: >> From: Marius Vlad >> >> Simple protocol extension to manag

Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-06-27 Thread Simon Ser
Cc'ing Marius Vlad again, hopefully with a working e-mail address now. is defunct. > From: Marius Vlad > > Simple protocol extension to manage DRM lease. Based on the work by Keith > Packard in [1], respectively [2]. > > [1] > https://cgit.freedesktop.org/mesa/drm/commit/?id=c4171535389d72e9135

Re: [PATCH] unstable/drm-lease: DRM lease protocol support

2019-06-27 Thread Simon Ser
Hi, Thanks for your work! Here is a first round of comments. On Thursday, June 27, 2019 10:46 PM, Drew DeVault wrote: > From: Marius Vlad > > Simple protocol extension to manage DRM lease. Based on the work by Keith > Packard in [1], respectively [2]. > > [1] > https://cgit.freedesktop.org/mes