Re: [PATCH] Don't use libudev for glx/dri3

[Eric Anholt <eric@anholt.net>]

[-- Attachment #1.1: Type: text/plain, Size: 1461 bytes --]

Daniel Vetter <daniel@ffwll.ch> writes:

> On Sat, Dec 14, 2013 at 3:33 AM, Kenneth Graunke <kenneth@whitecape.org> wrote:
>> On 11/18/2013 12:58 PM, Emil Velikov wrote:
>>> On 18/11/13 01:08, Keith Packard wrote:
>>>> libudev doesn't have a stable API/ABI, and if the application wants to use one
>>>> version, we'd best not load another into libGL.
>>>>
>>>> Signed-off-by: Keith Packard <keithp@keithp.com>
>>>> ---
>>>>
>>> Hi Keith,
>>>
>>> Did you had the chance to look at src/gallium/targets/egl-static/egl.c?
>>> It has a different implementation of drm_fd_get_pci_id, whenever udev is
>>> not available.
>>>
>>> AFAICS it goes back to the kernel via the relevant ioctl to retrieve the
>>> deviceid/chipid. Currently all but nouveau provide such information. I'm
>>> thinking that this approach might be more reasonable for those concerned
>>> with portability of the udev bits (think on *BSD).
>>>
>>> I'm not nitpicking, just thought you might find this interesting.
>>>
>>> Cheers,
>>> Emil
>>
>> Possibly.  But looking at that code, it either:
>> 1. Uses libudev
>> 2. On Android only...strcmps the driver string to guess the family and
>> then uses kernel ioctls...
>> 3. Fails.
>>
>> The Android only nature makes me a bit wary.
>
> The string mapping of the kernel driver name to a dri.so is what I
> kinda expected to be used as the generic thing

[snip]

There is more than one userspace driver per kernel driver.  (i915 vs
i965, for example)

[-- Attachment #1.2: Type: application/pgp-signature, Size: 835 bytes --]

[-- Attachment #2: Type: text/plain, Size: 156 bytes --]

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev