[Mesa-dev] [PATCH] configure.ac: describe what all the LIBDRM_*REQUIRED macros mean
plamena.manolova at intel.com
Thu Feb 2 12:01:11 UTC 2017
LGTM Reviewed-by: Plamena Manolova <plamena.manolova at intel.com>
On Thu, Feb 2, 2017 at 2:32 AM, Emil Velikov <emil.l.velikov at gmail.com>
> From: Emil Velikov <emil.velikov at collabora.com>
> They are versions of the respective libdrm package. They are _not_ the
> version of libdrm[.so] required for driver X.
> Doing the latter will lead to combinatoric explosion and in all fairness
> things will likely be broken most of the time.
> To make things even more confusing the kernel UAPI is provided by libdrm
> Cc: Vinson Lee <vlee at freedesktop.org>
> Cc: Kenneth Graunke <kenneth at whitecape.org>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> Ken, you/Chad have things spot on. Yet semes like other people struggle
> deeply with these.
> configure.ac | 7 +++++++
> 1 file changed, 7 insertions(+)
> diff --git a/configure.ac b/configure.ac
> index 92339b4a3f..e8e154b412 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -67,6 +67,13 @@ OPENCL_VERSION=1
> dnl Versions for external dependencies
> +dnl The LIBDRM instances reference either the generic libdrm or the
> +dnl specific libdrm-$hw.
> +dnl They are _not_ "the min. libdrm required for dri/gallium/other driver
> +dnl Note that the "basic" ABI for $hw -> $hw_drm.h is provided by libdrm
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the mesa-dev