On Sun, Mar 21, 2021 at 02:06:55PM +0200, Leon Romanovsky wrote:
> On Fri, Mar 19, 2021 at 04:50:53PM +0000, Xiong, Jianxin wrote:
> >
> > > From: liweihang <liweihang@xxxxxxxxxx>
> > > Sent: Friday, March 19, 2021 1:44 AM
> > > To: Xiong, Jianxin <jianxin.xiong@xxxxxxxxx>; linux-rdma@xxxxxxxxxxxxxxx
> > > Cc: Jason Gunthorpe <jgg@xxxxxxxx>; Leon Romanovsky <leon@xxxxxxxxxx>; Linuxarm <linuxarm@xxxxxxxxxx>
> > > Subject: [rdma-core] Compile issue with DRM headers
> > >
> > > Hi Jianxin,
> > >
> > > I met a compile error with recent version of rdma-core on my server with Ubuntu
> > > 14.04:
> > >
> > > ../pyverbs/dmabuf_alloc.c:16:24: fatal error: amdgpu_drm.h: No such file or directory #include <amdgpu_drm.h>
> > > ^
> > > compilation terminated.
> > >
> > > I found it is related with dma-buf based commits. And the commit 3788aa843b4b
> > > ("configure: Add check for DRM headers") adds a check for libdrm headers. I have installed it but my version(2.4.67-1ubuntu0.14.04.2) isn't
> > > new enough, there is no 'amdgpu_drm.h' in DRM_INCLUDE_DIRS(/usr/include/drm).
> > >
> > > So I think we may need some check for the the version of libdrm in CMakeList.txt or something else :) Could you please give me some
> > > suggestions?
> > >
> > > Thanks
> > > Weihang
> >
> > Hi Weihang,
> >
> > The simplest way is to replace the check of "drm.h" with "amdgpu_drm.h". This is
> > reasonable since dma-buf based MR won't work with old kernel anyway.
> >
> > Alternatively, we can add a check for "amdgpu_drm.h" in CMakeLists.txt and add
> > some #ifdef's to dmabuf_alloc.c around the code related to amdgpu.
>
> Let's add compilation test that checks all those files at the same time:
> 14 #include <drm.h>
> 15 #include <i915_drm.h>
> 16 #include <amdgpu_drm.h>
> 17 #include <radeon_drm.h>
Yes please
Jason
