<div dir="ltr"><div>Hi Peter,</div><div><br></div><div>Thanks for sharing this proposal. I have some little comments for a later stage, but would rather discuss some big items first.</div><div><br></div><div>The feedback I will share comes from a couple different angles as we are interested in Wayland for various different use cases.</div><div><br></div><div>Originally we had the gamepad wayland thread originally started by Jingkui. The proposal there was to provide knowledge of axes, buttons and everything making up a gamepad to the compositor. He proposed this direction as a way to inject events from Chrome into Android as at that level there is no file descriptor anymore. Ourselves we were quite interested in this approach as well. Among the things we do is roughly remote streamable desktops. Currently on X you can inject keyboard/mouse events through XTest, but the same applies to a custom Wayland compositor into which you can easily inject keyboard/mouse data. Clients don't have to be aware of the custom virtual devices and protocols behind the scenes.</div><div><br></div><div>When using inputfd for such a use case, either we would need to fake an input device through uinput (not ideal) or add a new 'fd_type' and have a custom protocol across this. Clients or input libraries would need to be able to deal with this other protocol, which would be a big pain. For such remote desktop use cases and even the Android one inputfd is not ideal.</div><div><br></div><div>Now back to general inputfd feedback and requirements. I want to make sure basic gamepad functionality is handled and also more complex functionality. As the maintainer of hid-sony for ds3/ds4 controllers, I want to make sure our gamepads are supported well. The same feedback would also apply to xbox360, switch or steam controllers as all devices have become more complex.</div><div><br></div><div>The current proposal could work for exposing basic gamepad functionality for the axes/buttons. Taking hid-sony as an example ds3 and ds4 devices are now threated as composite devices utilizing multiple device nodes. The ds4 is probably the best example using 3 evdev nodes, 1 for gamepad, 1 for touchpad (reported as a pointer device) and 1 for motion sensors. I expect the Switch Pro controller and the Joypads for the Switch to use at minimum 2 nodes as well.</div><div><br></div><div>How would composite devices be handled? There would need to be a way to tie multiple devices together. So far composite devices use all the same ids (product/vendor) and same unique id (EVIOCGUNIQ) and even physical location (EVIOCGPHYS). At minimum I think the compositor needs to tie these together with some sort of shared id. Another issue again taking ds4 as an example it has a touchpad, which is already picked up as a pointer device elsewhere. How to handle this? Unflag it as a pointer and share this inputfd as well or use it really as a pointer and gamepad state the other way?</div><div><br></div><div>In addition next to evdev nodes there are sysfs nodes for which fd passing won't work. The best example would be LEDs as found on ds3/ds4/xbox360, but I'm sure there will be GPIO as well. In case of ds4 it is not uncommon to update the LEDs many times a second (often used for light effects). How would such features be exposed? Is the idea to leverage properties or something like this? Passing these nodes to clients is probably not a good idea.</div><div><br></div><div>This is some initial feedback. I'm not sure how much I like the inputfd proposal. For a part because it doesn't fit some of our use cases well (remote desktop + custom protocols). In addition I'm also a bit worried about the complexity in handling composite devices.</div><div><br></div><div><div>Thanks,</div><div>Roderick<br class="gmail-Apple-interchange-newline"><br></div>-- <br><div class="gmail_signature">Roderick Colenbrander<br>Senior Manager of Software Engineering<br>Gaikai, a Sony Interactive Entertainment Company<br><a href="mailto:roderick@gaikai.com" target="_blank">roderick@gaikai.com</a></div></div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 31, 2017 at 12:29 AM, Peter Hutterer <span dir="ltr"><<a href="mailto:peter.hutterer@who-t.net" target="_blank">peter.hutterer@who-t.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This is the first draft for a protocol to enable direct access to input<br>
devices. The basic premise is: instead of protocols to handle gaming<br>
devices, 3D mice, sensors, etc. we just hand you (the client) an fd and hope<br>
you're happy with it. You get to play with that fd until it's revoked, i.e.<br>
right now we only provide it for evdev fds where we have EVIOCMUTE.<br>
<br>
Focus management is straightforward and follows compositor policy. The<br>
protocol is quite simple:<br>
<br>
clients get a wp_inputfd_device for each device<br>
  this device sends some static information (name, usb ids, ...)<br>
  this device sends focus-in with an fd<br>
  this device sends focus-out, the fd is revoked<br>
  ... repeat focus in/out as necessary ...<br>
  this device sends removed on unplug<br>
<br>
The devices are wrapped up into seats and those seats are provided by a<br>
global manager object - largely because that's what we need for the protocol<br>
to work.<br>
<br>
The notable bits are:<br>
* devices are divided into categories, so instead of getting all devices,<br>
  you say "give me an inputfd-seat for gaming devices". That object then<br>
  sends you all devices. In the future, we'll have an equivalent "give me<br>
  the seat for 3D mice", etc., but right now the gaming devices is<br>
  all that exists<br>
* there's a generic key/value event called "property". This one is largely<br>
  so the compositor can tag devices with some capabilities to make it easier<br>
  for the client to identify what it wants. I'll punt to Bastien for<br>
  details because he requested that, but I can imagine something like<br>
  "joystick type" is "gamepad". This big deal with this is that the<br>
  dictionary of tags is something that compositor and clients will have to<br>
  agree on, and it's not something defined by the protocol. This enables us<br>
  to change/extend the tags independent of protocol bumps.<br>
  It's quite similar to having access to the udev device to get at udev<br>
  properties, but some devices may not have a udev device, so...<br>
<br>
What is *not* in the protocol is how the compositor decides what is a gaming<br>
devices. The ID_INPUT_JOYSTICK udev property is too crude with many false<br>
positives. For tablets, we get around this by having libwacom spit out udev<br>
rules to tag things correctly. A similar approach is the best solution<br>
here: have some database that knows about various joystick devices and which<br>
category they fall into and install the udev rules/hwdb entries<br>
accordingly. There was talk of a gaming-device-oriented library<br>
(libgamingdev), this library could provide that information. Such a library<br>
would also be a prime spot for that property dictionary. Volunteers to start<br>
working on this, please step forward!<br>
<br>
/me takes a big step back<br>
<br>
Again, this is a draft, so feel free to pick it apart or request changes so<br>
it matches your use-case. Please do add CC's as required, I am by no means a<br>
game developer so I'm not aware of use-cases beyond the immediately obvious.<br>
<br>
Cheers,<br>
   Peter<br>
<br>
---<br>
<br>
diff --git a/Makefile.am b/Makefile.am<br>
index e693afa..e46910a 100644<br>
--- a/Makefile.am<br>
+++ b/Makefile.am<br>
@@ -4,6 +4,7 @@ unstable_protocols =                                                            \<br>
        unstable/linux-dmabuf/linux-<wbr>dmabuf-unstable-v1.xml                      \<br>
        unstable/text-input/text-<wbr>input-unstable-v1.xml                          \<br>
        unstable/input-method/input-<wbr>method-unstable-v1.xml                      \<br>
+       unstable/inputfd/inputfd-<wbr>unstable-v1.xml                                \<br>
        unstable/xdg-shell/xdg-shell-<wbr>unstable-v5.xml                            \<br>
        unstable/xdg-shell/xdg-shell-<wbr>unstable-v6.xml                            \<br>
        unstable/relative-pointer/<wbr>relative-pointer-unstable-v1.<wbr>xml              \<br>
diff --git a/unstable/inputfd/README b/unstable/inputfd/README<br>
new file mode 100644<br>
index 0000000..a24d858<br>
--- /dev/null<br>
+++ b/unstable/inputfd/README<br>
@@ -0,0 +1,4 @@<br>
+Input device fd passing protocol<br>
+<br>
+Maintainers:<br>
+Peter Hutterer <<a href="mailto:peter.hutterer@who-t.net">peter.hutterer@who-t.net</a>><br>
diff --git a/unstable/inputfd/inputfd-<wbr>unstable-v1.xml b/unstable/inputfd/inputfd-<wbr>unstable-v1.xml<br>
new file mode 100644<br>
index 0000000..3b83378<br>
--- /dev/null<br>
+++ b/unstable/inputfd/inputfd-<wbr>unstable-v1.xml<br>
@@ -0,0 +1,279 @@<br>
+<?xml version="1.0" encoding="UTF-8"?><br>
+<protocol name="inputfd_unstable_v1"><br>
+  <copyright><br>
+    Copyright 2017 © Red Hat, Inc.<br>
+<br>
+    Permission is hereby granted, free of charge, to any person<br>
+    obtaining a copy of this software and associated documentation files<br>
+    (the "Software"), to deal in the Software without restriction,<br>
+    including without limitation the rights to use, copy, modify, merge,<br>
+    publish, distribute, sublicense, and/or sell copies of the Software,<br>
+    and to permit persons to whom the Software is furnished to do so,<br>
+    subject to the following conditions:<br>
+<br>
+    The above copyright notice and this permission notice (including the<br>
+    next paragraph) shall be included in all copies or substantial<br>
+    portions of the Software.<br>
+<br>
+    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,<br>
+    EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF<br>
+    MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND<br>
+    NONINFRINGEMENT.  IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS<br>
+    BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN<br>
+    ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN<br>
+    CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE<br>
+    SOFTWARE.<br>
+  </copyright><br>
+<br>
+  <description summary="Wayland protocol for direct fd access to input devices"><br>
+    This description provides a high-level overview of the interfaces<br>
+    in this protocol. For details, see the protocol specification.<br>
+<br>
+    Some input devices do not interact with the windowing system. Examples<br>
+    of such input devices are gaming controllers or 3D mice. In many cases,<br>
+    a client requires direct access to the device to access or interpret<br>
+    device-specific functionality.<br>
+<br>
+    This interface provides the ability for a compositor to pass a file<br>
+    descriptor to the client. The compositor may restrict the type of device<br>
+    designated as compatible device and it may restrict specific events from<br>
+    being sent to the client (e.g. by masking the Home button on a gamepad).<br>
+    Otherwise, a client should treat the device as if opened manually.<br>
+<br>
+    This interface divides devices into type-based categories, clients are<br>
+    expected to only request access to devices that match the category they<br>
+    require. As of version 1, this interface provides only a single<br>
+    category: "gaming devices". Future categories of devices are expected to<br>
+    re-use the same wp_inputfd_device interface, but may differ in the way<br>
+    how to request them.<br>
+<br>
+    Multiple input devices of the same type may exists and assigned to<br>
+    different seats. The top-level object of this protocol is a<br>
+    wp_inputfd_manager. Dependent on the device type sought after, a client<br>
+    may request a type-specific seat interface (e.g.<br>
+    wp_inputfd_manager.get_seat_<wbr>gamedev). This seat then provides the list<br>
+    of devices for that category.<br>
+<br>
+    Once a compositor deems a device to be focused on the client or on a<br>
+    client's surface it sends a wp_inputfd_device.focus_in event with a file<br>
+    descriptor for this device. A compositor may arbitrarily revoke access<br>
+    to the device by sending a wp_inputfd_gamedev.focus_out. Additionally, a<br>
+    compositor may invoke system functionality to restrict access to the<br>
+    file descriptor, e.g. by using EVIOCMUTE on an evdev fd.<br>
+<br>
+    Otherwise, a client should treat the file descriptor as direct access to<br>
+    the device for the duration of it having access.<br>
+  </description><br>
+<br>
+  <interface name="zwp_inputfd_manager_v1" version="1"><br>
+    <description summary="controller object for direct fd access input devices"><br>
+      An object that provides access to the input devices available for<br>
+      direct fd access on this system. All input devices are associated with<br>
+      a seat, to get access to the actual devices, use one of the get_seat<br>
+      requests, e.g. wp_inputfd_manager.get_seat_<wbr>gamedev.<br>
+    </description><br>
+<br>
+    <request name="get_seat_gamedev"><br>
+      <description summary="get the seat for gaming devices"><br>
+       Get the wp_inputfd_seat object for the given seat. This object<br>
+       provides access to all gaming devices in this seat.<br>
+<br>
+       The decision which device is a gaming device is made by the<br>
+       compositor. The protocol makes no guarantees whether a particular<br>
+       device is available through this interface.<br>
+      </description><br>
+      <arg name="inputfd_seat" type="new_id" interface="zwp_inputfd_seat_<wbr>v1"/><br>
+      <arg name="seat" type="object" interface="wl_seat" summary="The wl_seat object to retrieve the input devices for" /><br>
+    </request><br>
+<br>
+    <request name="destroy" type="destructor"><br>
+      <description summary="release the memory for the inputfd manager object"><br>
+       Destroy the wp_inputfd_manager object. Objects created from this<br>
+       object are unaffected and should be destroyed separately.<br>
+      </description><br>
+    </request><br>
+  </interface><br>
+<br>
+  <interface name="zwp_inputfd_seat_v1" version="1"><br>
+    <description summary="controller object for input devices of a seat"><br>
+      An object that provides access to the input devices available on this<br>
+      seat for the requested type of device. After binding to this<br>
+      interface, the compositor sends a set of wp_inputfd_seat.device_added<br>
+      events for currently available devices and whenever a new device<br>
+      becomes available.<br>
+    </description><br>
+<br>
+    <request name="destroy" type="destructor"><br>
+      <description summary="release the memory for the inputfd seat object"><br>
+       Destroy the wp_inputfd_seat object. Objects created from this<br>
+       object are unaffected and should be destroyed separately.<br>
+      </description><br>
+    </request><br>
+<br>
+    <event name="device_added"><br>
+      <description summary="new device notification"><br>
+       This event is sent whenever a new device becomes available on<br>
+       this seat. This event only provides the object id of the devices,<br>
+       any static information about the device (device name,<br>
+       vid/pid, etc.) is sent through the wp_inputfd_device interface.<br>
+<br>
+       Which devices are compatible input devices for this seat is a<br>
+       decision made by the compositor, the protocol makes no guarantee<br>
+       that any specific device becomes available as inputfd device to a<br>
+       client.<br>
+      </description><br>
+      <arg name="id" type="new_id" interface="zwp_inputfd_device_<wbr>v1" summary="the newly added device"/><br>
+    </event><br>
+  </interface><br>
+<br>
+  <interface name="zwp_inputfd_device_v1" version="1"><br>
+    <description summary="input fd device"><br>
+      The wp_inputfd_device interface represents one device accessible<br>
+      directly by an fd passed to the client.<br>
+<br>
+      A device has a number of static characteristics, e.g. device<br>
+      name and pid/vid. These capabilities are sent in an event sequence<br>
+      immediately after the wp_inputfd_seat.device_added event. This initial<br>
+      event sequence is terminated by a wp_inputfd_device.done event. This<br>
+      sequence is sent only once and always before the first<br>
+      wp_inputfd_device.focus_in event.<br>
+    </description><br>
+<br>
+    <request name="destroy" type="destructor"><br>
+      <description summary="destroy the inputfd object"><br>
+       This destroys the client's resource for this inputfd object.<br>
+      </description><br>
+    </request><br>
+<br>
+    <event name="name"><br>
+      <description summary="device name"><br>
+       The name is a UTF-8 encoded string with the device's name, intended<br>
+       for presentation to the user.<br>
+<br>
+       This event is sent in the initial burst of events before the<br>
+       wp_inputfd_device.done event.<br>
+<br>
+       This event is optional, if the required information is not available<br>
+       for this device the event is omitted.<br>
+      </description><br>
+      <arg name="name" type="string" summary="the device name"/><br>
+    </event><br>
+<br>
+    <event name="usb_id"><br>
+      <description summary="device USB vendor/product id"><br>
+       This event is sent in the initial burst of events before the<br>
+       wp_inputfd_device.done event.<br>
+<br>
+       This event is optional, if the required information is not available<br>
+       for this device the event is omitted.<br>
+      </description><br>
+      <arg name="vid" type="uint" summary="USB vendor id"/><br>
+      <arg name="pid" type="uint" summary="USB product id"/><br>
+    </event><br>
+<br>
+    <event name="property"><br>
+      <description summary="device capability notification"><br>
+       This event is sent to notify the client of a custom property that<br>
+       applies to this device. The property is a standard key/value store<br>
+       in UTF-8 format, interpretation of both strings is left to the<br>
+       client. The wayland protocol makes no guarantees about the content<br>
+       of each string beyond its text encoding.<br>
+<br>
+       Compositors and clients need to agree on a dictionary of properties.<br>
+       For example, a compositor may designate the device to be of<br>
+       'joystick-type' 'gamepad'. This dictionary is out of the scope of<br>
+       this protocol.<br>
+      </description><br>
+      <arg name="property" type="string" summary="A UTF-8 encoded property name"/><br>
+      <arg name="value" type="string" summary="A UTF-8 encoded property value"/><br>
+    </event><br>
+<br>
+    <event name="done"><br>
+      <description summary="device description events sequence complete"><br>
+       This event is sent immediately to signal the end of the initial<br>
+       burst of descriptive events. A client may consider the static<br>
+       description of the device to be complete and finalize<br>
+       initialization of the device.<br>
+      </description><br>
+    </event><br>
+<br>
+    <event name="removed"><br>
+      <description summary="device removed event"><br>
+       Sent when the device has been removed from the system.<br>
+<br>
+       If the client currently has the device focus, a<br>
+       wp_inputfd_device.focus_out event is sent before the removed event.<br>
+       See wp_inputfd_device.focus_in for more details.<br>
+<br>
+       When this event is received, the client must wp_inputfd_device.destroy<br>
+       the object.<br>
+      </description><br>
+    </event><br>
+<br>
+    <enum name="fd_type"><br>
+      <description summary="Input fd device file descriptor types"><br>
+       This enum specifies the format of the file descriptor passed to<br>
+       clients with the wp_inputfd_device.focus_in event.<br>
+      </description><br>
+      <entry name="evdev" value="0" summary="An evdev file descriptor" /><br>
+    </enum><br>
+<br>
+    <event name="focus_in"><br>
+      <description summary="input fd device focus in event"><br>
+       Notification that this client now has the focus and/or access to<br>
+       this device. The decision what consitutes focus left to the<br>
+       compositor. For example, a compositor may tie joystick focus to the<br>
+       wl_pointer focus of this seat. The protocol does not guarantee that<br>
+       any specific client ever receives the focus for a device.<br>
+<br>
+       The client is passed a file descriptor with access to this<br>
+       device. This file descriptor is valid until a subsequent<br>
+       wp_inputfd_device.focus_out event. Upon wp_inputfd_device.focus_out, the<br>
+       compositor may revoke the fd and further operations will fail.<br>
+<br>
+       However, due to potential race conditions a client must be able to<br>
+       handle errors as if it opened the fd itself. No guarantee is<br>
+       given that the wp_inputfd_device.focus_out event or wp_inputfd_device.removed<br>
+       event are sent before the client encounters an error on the file<br>
+       descriptor.<br>
+<br>
+       A compositor guarantees that the underlying device does not change<br>
+       until a wp_inputfd_device.removed event. In other words, if the fd<br>
+       type allows querying capabilities through the fd, a client needs to<br>
+       do so only once at the first focus_in. Subsequent focus_in events<br>
+       will provide the same capabilities.<br>
+<br>
+       If applicable, this event contains the surface that has the focus.<br>
+       In some cases, the focus may not be tied to a specific client surface<br>
+       but is given to the client independent of any surface. In that case,<br>
+       the surface is null.<br>
+<br>
+       The protocol guarantees that focus_in and focus_out always come in<br>
+       pairs. If the client currently has the focus and the device is<br>
+       removed, a focus_out event is sent to the client before the<br>
+       wp_inputfd_device.removed event.<br>
+      </description><br>
+      <arg name="serial" type="uint"/><br>
+      <arg name="fd" type="fd" summary="file descriptor to the device"/><br>
+      <arg name="fd_type" type="uint" enum="fd_type" summary="fd type" /><br>
+      <arg name="surface" type="object" interface="wl_surface" summary="The current surface that has the device's focus" allow-null="true"/><br>
+    </event><br>
+<br>
+    <event name="focus_out"><br>
+      <description summary="input fd device focus out event"><br>
+       Notification that this client no longer has focus and/or access to<br>
+       this device. Further reads from this device's file descriptor<br>
+       will fail. The client must close(2) the file descriptor received in<br>
+       the wp_inputfd_device.focus_in event.<br>
+<br>
+       This event does not mean the device was removed, merely that the<br>
+       device is focused elsewhere. For device removal, see<br>
+       wp_inputfd_device.removed.<br>
+<br>
+       See wp_inputfd_device.focus_in for more details.<br>
+      </description><br>
+    </event><br>
+  </interface><br>
+</protocol><br>
+<br>
+<br>
</blockquote></div><div class="gmail_signature"><br></div>
</div></div>