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

2019-10-31 Thread Drew DeVault
On Thu Oct 24, 2019 at 10:50 AM Drew DeVault wrote: > So, I'm not sure what the action items are here. It seems like we might > have uncovered a potentially icky race condition in Linux, but that this > protocol is not really affected. Bump. Happy halloween!

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

2019-10-24 Thread Drew DeVault
So, I'm not sure what the action items are here. It seems like we might have uncovered a potentially icky race condition in Linux, but that this protocol is not really affected. ___ wayland-devel mailing list wayland-devel@lists.freedesktop.org

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

2019-10-21 Thread Daniel Vetter
On Mon, Oct 21, 2019 at 10:48 AM Pekka Paalanen wrote: > > On Fri, 18 Oct 2019 17:47:49 +0200 > Daniel Vetter wrote: > > > On Fri, Oct 18, 2019 at 4:34 PM Pekka Paalanen wrote: > > > > > > On Fri, 18 Oct 2019 16:19:33 +0200 > > > Daniel Vetter wrote: > > > > > > > On Fri, Oct 18, 2019 at 3:43

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

2019-10-21 Thread Pekka Paalanen
On Fri, 18 Oct 2019 17:47:49 +0200 Daniel Vetter wrote: > On Fri, Oct 18, 2019 at 4:34 PM Pekka Paalanen wrote: > > > > On Fri, 18 Oct 2019 16:19:33 +0200 > > Daniel Vetter wrote: > > > > > On Fri, Oct 18, 2019 at 3:43 PM Pekka Paalanen > > > wrote: > > > > > > > > On Fri, 18 Oct 2019

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

2019-10-18 Thread Daniel Vetter
On Fri, Oct 18, 2019 at 4:34 PM Pekka Paalanen wrote: > > On Fri, 18 Oct 2019 16:19:33 +0200 > Daniel Vetter wrote: > > > On Fri, Oct 18, 2019 at 3:43 PM Pekka Paalanen wrote: > > > > > > On Fri, 18 Oct 2019 07:54:50 -0400 > > > "Drew DeVault" wrote: > > > > > > > On Fri Oct 18, 2019 at 12:21

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

2019-10-18 Thread Pekka Paalanen
On Fri, 18 Oct 2019 10:43:16 -0400 "Drew DeVault" wrote: > Regarding hotplugging, the Wayland compositor is probably keeping track > of hotplugs itself and withdrawing/offering connectors as appropriate. > Also, when the lease is issued, the compositor withdraws that connector. > For the client,

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

2019-10-18 Thread Philipp Zabel
On Fri, 2019-10-18 at 10:43 -0400, Drew DeVault wrote: > Regarding hotplugging, the Wayland compositor is probably keeping track > of hotplugs itself and withdrawing/offering connectors as appropriate. > > Also, when the lease is issued, the compositor withdraws that connector. > For the client,

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

2019-10-18 Thread Drew DeVault
Regarding hotplugging, the Wayland compositor is probably keeping track of hotplugs itself and withdrawing/offering connectors as appropriate. Also, when the lease is issued, the compositor withdraws that connector. For the client, upon hotplug I imagine the DRM asks start to fail, and it handles

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

2019-10-18 Thread Pekka Paalanen
On Fri, 18 Oct 2019 16:19:33 +0200 Daniel Vetter wrote: > On Fri, Oct 18, 2019 at 3:43 PM Pekka Paalanen wrote: > > > > On Fri, 18 Oct 2019 07:54:50 -0400 > > "Drew DeVault" wrote: > > > > > On Fri Oct 18, 2019 at 12:21 PM Pekka Paalanen wrote: > > > > One thing I did not know the last

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

2019-10-18 Thread Daniel Vetter
On Fri, Oct 18, 2019 at 3:43 PM Pekka Paalanen wrote: > > On Fri, 18 Oct 2019 07:54:50 -0400 > "Drew DeVault" wrote: > > > On Fri Oct 18, 2019 at 12:21 PM Pekka Paalanen wrote: > > > One thing I did not know the last time was that apparently > > > systemd-logind may not like to give out

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

2019-10-18 Thread Pekka Paalanen
On Fri, 18 Oct 2019 15:47:39 +0200 Philipp Zabel wrote: > On Fri, 2019-10-18 at 08:51 +, Simon Ser wrote: > > On Thursday, October 17, 2019 11:15 PM, Drew DeVault wrote: > > > > > On Thu Oct 17, 2019 at 6:08 PM Simon Ser wrote: > > > > > > > Should we keep the connector event, now

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

2019-10-18 Thread Ville Syrjälä
On Fri, Oct 18, 2019 at 04:43:29PM +0300, Pekka Paalanen wrote: > On Fri, 18 Oct 2019 07:54:50 -0400 > "Drew DeVault" wrote: > > > On Fri Oct 18, 2019 at 12:21 PM Pekka Paalanen wrote: > > > One thing I did not know the last time was that apparently > > > systemd-logind may not like to give out

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

2019-10-18 Thread Philipp Zabel
On Fri, 2019-10-18 at 08:51 +, Simon Ser wrote: > On Thursday, October 17, 2019 11:15 PM, Drew DeVault wrote: > > > On Thu Oct 17, 2019 at 6:08 PM Simon Ser wrote: > > > > > Should we keep the connector event, now that we have an unprivileged > > > DRM FD? > > > Alternatives include: > > >

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

2019-10-18 Thread Pekka Paalanen
On Fri, 18 Oct 2019 07:54:50 -0400 "Drew DeVault" wrote: > On Fri Oct 18, 2019 at 12:21 PM Pekka Paalanen wrote: > > One thing I did not know the last time was that apparently > > systemd-logind may not like to give out non-master DRM fds. That might > > need fixing in logind implementations. I

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

2019-10-18 Thread Drew DeVault
On Fri Oct 18, 2019 at 12:21 PM Pekka Paalanen wrote: > One thing I did not know the last time was that apparently > systemd-logind may not like to give out non-master DRM fds. That might > need fixing in logind implementations. I hope someone would step up to > look into that. > > This protocol

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

2019-10-18 Thread Pekka Paalanen
On Thu, 17 Oct 2019 13:45:27 -0400 Drew DeVault wrote: > From: Marius Vlad > > DRM leasing is a feature which allows the DRM master to "lease" a subset > of its DRM resources to another DRM master via drmModeCreateLease, which > returns a file descriptor for the new DRM master. We use this

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

2019-10-18 Thread Simon Ser
On Thursday, October 17, 2019 11:15 PM, Drew DeVault wrote: > On Thu Oct 17, 2019 at 6:08 PM Simon Ser wrote: > > > Should we keep the connector event, now that we have an unprivileged > > DRM FD? > > Alternatives include: > > > > - Let the client use the FD to get what it needs (EDID/a

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

2019-10-17 Thread Drew DeVault
On Thu Oct 17, 2019 at 6:08 PM Simon Ser wrote: > Should we keep the connector event, now that we have an unprivileged > DRM FD? > > Alternatives include: > > - Let the client use the FD to get what it needs (EDID/a configured > output/something else). Isn't this: > - Keep an event to

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

2019-10-17 Thread Simon Ser
Should we keep the connector event, now that we have an unprivileged DRM FD? Alternatives include: - Let the client use the FD to get what it needs (EDID/a configured output/something else). - Keep an event to advertise lease-able connector IDs ___

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

2019-10-17 Thread Drew DeVault
From: Marius Vlad DRM leasing is a feature which allows the DRM master to "lease" a subset of its DRM resources to another DRM master via drmModeCreateLease, which returns a file descriptor for the new DRM master. We use this protocol to negotiate the terms of the lease and transfer this file