Re: [PATCH rdma-core] configure: Check the existence of all needed DRM headers

[liweihang <liweihang@huawei.com>]

On 2021/3/23 4:05, Jianxin Xiong wrote:
> Some vendor specific DRM headers may be missing on systems with old
> kernels. Make sure that all headers needed by pyverbs/dmabuf_alloc.c
> are present before enabling that module.
> 
> Remove unused reference of "radeon_drm.h" from pyverbs/dmabuf_alloc.c.
> 
> Signed-off-by: Jianxin Xiong <jianxin.xiong@intel.com>
> ---
>  CMakeLists.txt         | 6 +++++-
>  pyverbs/dmabuf_alloc.c | 1 -
>  2 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/CMakeLists.txt b/CMakeLists.txt
> index e9a2f49..1208ab6 100644
> --- a/CMakeLists.txt
> +++ b/CMakeLists.txt
> @@ -526,7 +526,11 @@ if (NOT DRM_INCLUDE_DIRS)
>  endif()
>  
>  if (DRM_INCLUDE_DIRS)
> -  include_directories(${DRM_INCLUDE_DIRS})
> +  if (EXISTS ${DRM_INCLUDE_DIRS}/i915_drm.h AND EXISTS ${DRM_INCLUDE_DIRS}/amdgpu_drm.h)
> +    include_directories(${DRM_INCLUDE_DIRS})
> +  else()
> +    unset(DRM_INCLUDE_DIRS CACHE)
> +  endif()
>  endif()
>  
>  #-------------------------
> diff --git a/pyverbs/dmabuf_alloc.c b/pyverbs/dmabuf_alloc.c
> index 9978a5b..e3ea0a4 100644
> --- a/pyverbs/dmabuf_alloc.c
> +++ b/pyverbs/dmabuf_alloc.c
> @@ -14,7 +14,6 @@
>  #include <drm.h>
>  #include <i915_drm.h>
>  #include <amdgpu_drm.h>
> -#include <radeon_drm.h>
>  #include "dmabuf_alloc.h"
>  
>  /*
> 

Tested-by: Weihang Li <liweihang@huawei.com>

It solves the issue I met on my server with ubuntu 14.04 ,thank you.

Weihang