[PATCH v2 00/14] Use systemd-logind to manage /dev/... nodes for us
Hans de Goede
hdegoede at redhat.com
Fri Feb 21 04:41:56 PST 2014
Hi,
On 02/18/2014 11:37 PM, Dave Airlie wrote:
>>
>> Thanks for the review!
>>
>>> I'm not systemd-logind aware enough to know about the rest yet,
>>>
>>> Well we already have a number of flags from video drivers probing,
>>> like the needs hw one etc, do you need
>>> to have a separate capabilities field in the driver?
>>
>
> sorry about delay, I'm not really sure I've a good opinion here on how to go,
>
> but what about driverFunc, can't you use that to get flags back from
> the driver before probe?
Ah yes, that would work, I did not know about that one.
So I see 2 options here, either add a new HW_SUPPORT_SERVER_FD flag, which
would be a bit weird API wise, or add a new
SUPPORTS_SERVER_FDS xorgDriverFuncOp, which simply returns TRUE if the
driver supports server fds.
I've a preference for the latter, so if I don't hear anything from you
that is what I'll go and implement.
> We use that now to send some flags from the drivers back to the server.
>
> though I'm quite happy to accept that everyone should handle server-fds,
> just make sure the driver authors know what to change to make it work.
I think it would be best for drivers to opt-in to server-fds, esp. taking
things like the blobs into consideration.
With your driverFunc idea that should be a simple follow-up patch on top of
my existing patches.
I'm going vacation for a week next-week, after that I'll post a pull-req
for my existing already reviewed patches (seems poor style todo that
just before going on vacation in case my work causes any issues), and
post a new patch adding support for drivers to advertise that they
support server-fds.
I'll also throw some docs on what drivers need to do to support server-fds
into that patch.
> and port -modesetting :)
Yes I've already dug-up my usb displaylink laptop dock to test this :)
And I'll also take care of adding support for server fds to -intel, -ati
and -nouveau.
Regards,
Hans
More information about the xorg-devel
mailing list