[Mesa-dev] [PATCH] loader: add a libdrm case for loader_get_device_name_for_fd

[Emil Velikov]

On 19 April 2016 at 04:03, Jonathan Gray <jsg at jsg.id.au> wrote:
> Any objections to this?
>
Absolutely none. I simply missed it.

> On Mon, Dec 21, 2015 at 04:39:55PM +1100, Jonathan Gray wrote:
>> Use dev_node_from_fd() with HAVE_LIBDRM to provide an implmentation
>> of loader_get_device_name_for_fd() for non-linux systems that
>> use libdrm but don't have udev or sysfs.
>>
>> Signed-off-by: Jonathan Gray <jsg at jsg.id.au>
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

Jonathan, apart from the patches that I've replied to (and the llvm
one in a second) do you have any more, even if they're
nasty/subjective/etc. ? Please send them over and ping me/us is
there's no reply within a week or so.

On the topic of the loader I hope you managed to take a look at the
libdrm drmDevice API. It does (almost) the same thing as the below
patch, but my goal is to keep it separate so that one can reuse it
from other places, plus we can re-spin a libdrm release as often
and/fast as we need to. In other words - most/all of this nasty code
will get moved from mesa to libdrm. Note current libdrm implementation
is Linux only. Please throw in a patch when you can spare some time.

Thanks
Emil