[PATCH v2 09/14] hotplug: Extend OdevAttributes for server-managed fd support

Peter Hutterer peter.hutterer at who-t.net
Tue Feb 11 06:56:02 CET 2014


On Tue, Feb 04, 2014 at 12:49:15PM +0100, Hans de Goede wrote:
> With systemd-logind support, the xserver, rather than the drivers will be
> responsible for opening/closing the fd for drm nodes.
> 
> This commit adds a fd member to OdevAttributes to store the fd to pass it
> along to the driver.
> 
> systemd-logind tracks devices by their chardev major + minor numbers, so
> also add OdevAttributes to store the major and minor.
> 
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
> ---
>  hw/xfree86/common/xf86platformBus.c | 14 ++++++++++++++
>  hw/xfree86/common/xf86platformBus.h |  1 +
>  include/hotplug.h                   | 10 ++++++++++
>  3 files changed, 25 insertions(+)
> 
> diff --git a/hw/xfree86/common/xf86platformBus.c b/hw/xfree86/common/xf86platformBus.c
> index 97d606a..e1d7bd0 100644
> --- a/hw/xfree86/common/xf86platformBus.c
> +++ b/hw/xfree86/common/xf86platformBus.c
> @@ -128,6 +128,20 @@ xf86_get_platform_device_unowned(int index)
>          TRUE : FALSE;
>  }
>  
> +struct xf86_platform_device *
> +xf86_find_platform_device_by_devnum(int major, int minor)
> +{
> +    int i, attr_major, attr_minor;
> +
> +    for (i = 0; i < xf86_num_platform_devices; i++) {
> +        attr_major = xf86_get_platform_int_attrib(i, ODEV_ATTRIB_MAJOR, 0);
> +        attr_minor = xf86_get_platform_int_attrib(i, ODEV_ATTRIB_MINOR, 0);
> +        if (attr_major == major && attr_minor == minor)
> +            return &xf86_platform_devices[i];
> +    }
> +    return NULL;
> +}
> +
>  /*
>   * xf86IsPrimaryPlatform() -- return TRUE if primary device
>   * is a platform device and it matches this one.
> diff --git a/hw/xfree86/common/xf86platformBus.h b/hw/xfree86/common/xf86platformBus.h
> index 9a49260..3e27d34 100644
> --- a/hw/xfree86/common/xf86platformBus.h
> +++ b/hw/xfree86/common/xf86platformBus.h
> @@ -35,6 +35,7 @@ struct xf86_platform_device {
>  
>  /* xf86_platform_device flags */
>  #define XF86_PDEV_UNOWNED       0x01
> +#define XF86_PDEV_SERVER_FD     0x02

this seems to belong to a different patch?

Cheers,
   Peter
  
>  #ifdef XSERVER_PLATFORM_BUS
>  int xf86platformProbe(void);
> diff --git a/include/hotplug.h b/include/hotplug.h
> index 70d80f8..676e647 100644
> --- a/include/hotplug.h
> +++ b/include/hotplug.h
> @@ -78,6 +78,12 @@ config_odev_free_attributes(struct OdevAttributes *attribs);
>  #define ODEV_ATTRIB_SYSPATH 2
>  /* DRI-style bus id */
>  #define ODEV_ATTRIB_BUSID 3
> +/* Server managed FD */
> +#define ODEV_ATTRIB_FD 4
> +/* Major number of the device node pointed to by ODEV_ATTRIB_PATH */
> +#define ODEV_ATTRIB_MAJOR 5
> +/* Minor number of the device node pointed to by ODEV_ATTRIB_PATH */
> +#define ODEV_ATTRIB_MINOR 6
>  
>  typedef void (*config_odev_probe_proc_ptr)(struct OdevAttributes *attribs);
>  void config_odev_probe(config_odev_probe_proc_ptr probe_callback);
> @@ -88,4 +94,8 @@ void DeleteGPUDeviceRequest(struct OdevAttributes *attribs);
>  #endif
>  
>  #define ServerIsNotSeat0() (SeatId && strcmp(SeatId, "seat0"))
> +
> +struct xf86_platform_device *
> +xf86_find_platform_device_by_devnum(int major, int minor);
> +
>  #endif                          /* HOTPLUG_H */
> -- 
> 1.8.5.3
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
> 


More information about the xorg-devel mailing list