On Mon, Nov 30, 2020 at 05:53:39PM +0000, Xiong, Jianxin wrote: > > From: Jason Gunthorpe <j...@ziepe.ca> > > Sent: Monday, November 30, 2020 8:08 AM > > To: Xiong, Jianxin <jianxin.xi...@intel.com> > > Cc: linux-r...@vger.kernel.org; dri-devel@lists.freedesktop.org; Doug > > Ledford <dledf...@redhat.com>; Leon Romanovsky > > <l...@kernel.org>; Sumit Semwal <sumit.sem...@linaro.org>; Christian Koenig > > <christian.koe...@amd.com>; Vetter, Daniel > > <daniel.vet...@intel.com> > > Subject: Re: [PATCH rdma-core v3 4/6] pyverbs: Add dma-buf based MR support > > > > On Fri, Nov 27, 2020 at 12:55:41PM -0800, Jianxin Xiong wrote: > > > > > > +function(rdma_multifile_module PY_MODULE MODULE_NAME LINKER_FLAGS) > > > > I think just replace rdma_cython_module with this? No good reason I can see > > to have two APIs? > > rdma_cython_module can handle many modules, but this one is for a single > module. > If you agree, I can merge the two by slightly tweaking the logic: each module > starts > with a .pyx file, followed by 0 or more .c and .h files.
Then have rdma_cython_module call some rdam_single_cython_module() multiple times that has this code below? > > Here too? You probably don't need to specify h files at all, at > > worst they should only be used with publish_internal_headers > > Without the .h link, the compiler fail to find the header file (both > dmabuf_alloc.c and the generated "dmabuf.c" contain #include > "dmabuf_alloc.h"). Header files are made 'cross module' using the "publish_internal_headers" command But we could also hack in a -I directive to fix up the "" include for the cython outupt.. But it should not be handled here in the cython module command Jason _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
