[Mesa-dev] [PATCH 0/2] anv/i965: drop libdrm dependency completely

[Eric Anholt]

Emil Velikov <emil.l.velikov at gmail.com> writes:

> On 25 April 2017 at 23:56, Lionel Landwerlin
> <lionel.g.landwerlin at intel.com> wrote:
>> Hi,
>>
>> While working with changes that span from kernel to user space, I've
>> been wondering whether we need to depend on libdrm at all for the anv
>> & i965 drivers. Indeed with Ken's recent changes, we only depend on
>> libdrm for its kernel header files which we could just embed
>> ourselves.
>>
>> I've only included the minimal set of header files we need from the
>> kernel for anv & i965. Maybe other drivers would be interested and
>> maybe we should put all the kernel drm uapi headers into include?
>>
> AFAICT the goal behind the libdrm_intel fold was to allow rapid and
> seamless rework of the interface.
> With a potential goal to make it a shared one, as it gets stabilised.
>
> Currently ANV uses more than just the UAPI headers. But if we ignore
> that, coping them is a bad idea.
>
> Why - adds a, yet another, copy and making synchronisation more
> annoying. First example - you blindly copied the files rather than
> using `make headers_install' first ;-)
> Not to mention that it makes the chicken and egg problem* even more
> confusing and error prone.
>
> Emil
> *  Which patches land first - kernel or userspace

I don't see how it does that at all.  It simplifies the process: Now you
send out your proposed new userspace to one repo, instead of two.

And, after the first commit, it'll be obvious when you screw up using
make headers_install because there will be surprising diff.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170427/21f6e161/attachment.sig>