Cc'ing Marius Vlad again, hopefully with a working e-mail address now. <[email protected]> is defunct.
> From: Marius Vlad <[email protected]> > > 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] > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v4.15-rc9&id=62884cd386b876638720ef88374b31a84ca7ee5f > > Signed-off-by: Marius Vlad <[email protected]> > Signed-off-by: Drew DeVault <[email protected]> > --- > This updates Marius's original patch series implementing DRM leasing for > Wayland. This cleans up the XML style, reworks resource lifetimes, adds > a little link to xdg-output, and a few other changes. > > A server-side implementation of this protocol is under development and > available here: > > https://github.com/swaywm/wlroots/pull/1730 > > I've also rigged up Keith Packard's kmscube fork to support leasing from > Wayland instead of X: > > https://git.sr.ht/~sircmpwn/kmscube > > Run `./kmscube -l` to test it. While doing the research for this > protocol there was also some discussions in wlroots which may be > insightful: > > https://github.com/swaywm/wlroots/issues/1723 > > Background: > > 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 descriptor to > clients. > > In less DRM-specific terms: this protocol allows Wayland compositors to > give over their GPU resources (like displays) to a Wayland client to > exclusively control. > > The primary use-case for this is Virtual Reality headsets, which via the > non-desktop DRM property are generally not used as desktop displays by > Wayland compositors, and for latency reasons (among others) are most > useful to games et al if they have direct control over the DRM resources > associated with it. Basically, these are peripherals which are of no use > to the compositor and may be of use to a client, but since they are tied > up in DRM we need to use DRM leasing to get them into client's hands. > > 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 lease. Additionally, leased connectors can be added and > removed at the compositor's whim, and race conditions have been > considered to avoid disagreement between the client and compositor as to > which connectors are available for lease - the compositor being the > ultimate authority. > > In the coming weeks I intend to work on patches for Vulkan and Xwayland > adding support for this protocol, respectively to allow Vulkan clients > to utilize DRM leasing on Wayland and to allow X11 clients utilizing the > xrandr lease request[0] to fulfill their leases through Xwayland. > > https://gitlab.freedesktop.org/xorg/proto/xcbproto/blob/master/src/randr.xml#L909-938 > > Makefile.am | 1 + > unstable/drm-lease/README | 4 + > unstable/drm-lease/drm-lease-unstable-v1.xml | 203 +++++++++++++++++++ > 3 files changed, 208 insertions(+) > create mode 100644 unstable/drm-lease/README > create mode 100644 unstable/drm-lease/drm-lease-unstable-v1.xml > > diff --git a/Makefile.am b/Makefile.am > index 345ae6a..d9fff89 100644 > --- a/Makefile.am > +++ b/Makefile.am > @@ -4,6 +4,7 @@ unstable_protocols = > \ > unstable/pointer-gestures/pointer-gestures-unstable-v1.xml > \ > unstable/fullscreen-shell/fullscreen-shell-unstable-v1.xml > \ > unstable/linux-dmabuf/linux-dmabuf-unstable-v1.xml > \ > + unstable/drm-lease/drm-lease-unstable-v1.xml > \ > unstable/text-input/text-input-unstable-v1.xml > \ > unstable/text-input/text-input-unstable-v3.xml > \ > unstable/input-method/input-method-unstable-v1.xml > \ > diff --git a/unstable/drm-lease/README b/unstable/drm-lease/README > new file mode 100644 > index 0000000..a25600c > --- /dev/null > +++ b/unstable/drm-lease/README > @@ -0,0 +1,4 @@ > +Linux DRM lease > + > +Maintainers: > +Marius Vlad <[email protected]> > diff --git a/unstable/drm-lease/drm-lease-unstable-v1.xml > b/unstable/drm-lease/drm-lease-unstable-v1.xml > new file mode 100644 > index 0000000..8f52021 > --- /dev/null > +++ b/unstable/drm-lease/drm-lease-unstable-v1.xml > @@ -0,0 +1,203 @@ > +<?xml version="1.0" encoding="UTF-8"?> > +<protocol name="drm_lease_unstable_v1"> > + <copyright> > + Copyright © 2018 NXP > + Copyright © 2019 Status Research & Development GmbH. > + > + Permission is hereby granted, free of charge, to any person obtaining a > + copy of this software and associated documentation files (the > "Software"), > + to deal in the Software without restriction, including without limitation > + the rights to use, copy, modify, merge, publish, distribute, sublicense, > + and/or sell copies of the Software, and to permit persons to whom the > + Software is furnished to do so, subject to the following conditions: > + > + The above copyright notice and this permission notice (including the next > + paragraph) shall be included in all copies or substantial portions of the > + Software. > + > + THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS > OR > + IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR > OTHER > + LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING > + FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER > + DEALINGS IN THE SOFTWARE. > + </copyright> > + > + <interface name="zwp_drm_lease_manager_v1" version="1"> > + <description summary="lease manager"> > + This protocol is used by Wayland compositors which act as Direct > + Renderering Manager (DRM) masters to lease DRM resources to Wayland > + clients. Once leased, the compositor will not use the leased resources > + until the lease is revoked or the client closes the file descriptor. > + > + The lease manager is used to advertise connectors which are available > for > + leasing, and by the client to negotiate a lease request. > + > + Warning! The protocol described in this file is experimental and > + backward incompatible changes may be made. Backward compatible changes > + may be added together with the corresponding interface version bump. > + Backward incompatible changes are done by bumping the version number in > + the protocol and interface names and resetting the interface version. > + Once the protocol is to be declared stable, the 'z' prefix and the > + version number in the protocol and interface names are removed and the > + interface version number is reset. > + </description> > + > + <request name="destroy" type="destructor"> > + <description summary="destroys the manager"> > + Destroys this zwp_drm_lease_manager_v1 object. > + > + Destroying a bound zwp_drm_lease_manager_v1 while there are lease > + requests still alive is illegal and will result in a protocol error. > + </description> > + </request> > + > + <request name="create_lease_request"> > + <description summary="create a lease request object"> > + Creates a lease request object. > + > + See the documentation for zwp_drm_lease_request_v1 for details. > + </description> > + <arg name="id" type="new_id" interface="zwp_drm_lease_request_v1" /> > + </request> > + > + <event name="connector"> > + <description summary="advertise connectors available for leases"> > + 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 request to lease that connector. See > + zwp_drm_lease_request_v1.add_connector for details. > + > + When this global is bound, the compositor will send all connectors > + available for lease, but may send additional connectors at any time. > + </description> > + <arg name="id" type="new_id" interface="zwp_drm_lease_connector_v1" /> > + </event> > + </interface> > + > + <interface name="zwp_drm_lease_connector_v1" version="1"> > + <description summary="a leasable DRM connector"> > + Represents a DRM connector which is available for lease. These objects > are > + created via zwp_drm_lease_manager_v1.connector, and should be passed > into > + lease requests via zwp_drm_lease_request_v1.add_connector. > + </description> > + > + <event name="name"> > + <description summary="name"> > + Sent to indicate the name of this connector. This will not change for > + the duration of the Wayland session, but is not guaranteed to be > + consistent between sessions. > + > + If the compositor also supports zxdg_output_manager_v1, this name > will > + match the name of a zxdg_output_v1 object. > + </description> > + <arg name="name" type="string" summary="connector name" /> > + </event> > + > + <event name="withdrawn"> > + <description summary="lease offer withdrawn"> > + Sent to indicate that the compositor will no longer honor requests > for > + DRM leases which include this connector. The client may still issue a > + lease request including this connector, but the connector may not be > + included in the issued lease. > + </description> > + </event> > + > + <request name="release"> > + <description summary="release connector"> > + The client may send this request to indicate that it will not issue a > + lease request for this connector. Clients are encouraged to send this > + after receiving the "withdrawn" request so that the server can > release > + the resources associated with this connector offer. > + > + It is a protocol error to call > + zwp_drm_lease_request_v1.request_connector with this object, or to > call > + zwp_drm_lease_request_v1.submit on a lease which includes this > + connector, after the connector has been released. > + </description> > + </request> > + </interface> > + > + <interface name="zwp_drm_lease_request_v1" version="1"> > + <description summary="DRM lease request"> > + A client that wishes to lease DRM resources will attach the list of > + connectors advertised with zwp_drm_lease_manager_v1.connector that they > + wish to lease, then use zwp_drm_lease_request_v1.submit to submit the > + request. > + </description> > + > + <request name="destroy" type="destructor"> > + <description summary="destroys the lease request object"> > + Indicates that the client will no longer use this lease request. It > is a > + protocol error to send any further requests for this object after > + sending destroy. > + </description> > + </request> > + > + <request name="request_connector"> > + <description summary="request a connector for this lease"> > + Indicates that the client would like to lease the given connector. > + This must be called before zwp_drm_lease_request_v1.create. This is > + only used as a suggestion, the compositor may choose to include any > + resources in the lease it issues, or change the set of leased > + resources at any time. > + </description> > + <arg name="connector" type="object" > + interface="zwp_drm_lease_connector_v1" /> > + </request> > + > + <request name="submit"> > + <description summary="submit the lease request"> > + Submits the lease request and creates a new zwp_drm_lease_v1 object. > + After calling submit, issuing any other request than destroy is a > + protocol error. Submitting a lease request with no connectors is a > + protocol error. > + </description> > + <arg name="id" type="new_id" interface="zwp_drm_lease_v1" /> > + </request> > + </interface> > + > + <interface name="zwp_drm_lease_v1" version="1"> > + <description summary="a DRM lease"> > + A DRM lease object is used to transfer the DRM file descriptor to the > + client and manage the lifetime of the lease. > + </description> > + > + <event name="lease_fd"> > + <description summary="shares the DRM file descriptor"> > + This event returns a file descriptor suitable for use with > DRM-related > + ioctls. The client should use drmModeGetLease to enumerate the DRM > + objects which have been leased to them, which may not be the objects > + they requested. The lease may have zero DRM objects. > + > + The compositor may also issue and immediately revoke the lease if no > + connectors are leasable. > + > + It is a protocol error for the compositor to send this event more > than > + once for a given lease. > + </description> > + <arg name="leased_fd" type="fd" summary="leased DRM file descriptor" /> > + </event> > + > + <event name="finish"> > + <description summary="sent when the lease has been revoked"> > + When the compositor revokes the lease, it will issue this event to > + notify clients of the change. If the client requires a new lease, > they > + should destroy this resource and submit a new lease request. The > + compositor will send no further events for this object after sending > + the finish event. > + </description> > + </event> > + > + <request name="destroy" type="destructor"> > + <description summary="destroys the lease object"> > + The client should send this to indicate that it no longer wishes to > use > + this lease. The compositor should use drmModeRevokeLease on the > + appropriate file descriptor, if necessary, then release this object. > It > + is a protocol error for the client to send any further requests for > + this object after sending a destroy request. > + </description> > + </request> > + </interface> > +</protocol> > -- > 2.22.0 > > _______________________________________________ > wayland-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
