> -----Original Message----- > From: Jason Gunthorpe <jgg@xxxxxxxx> > Sent: Wednesday, November 25, 2020 4:00 PM > To: Xiong, Jianxin <jianxin.xiong@xxxxxxxxx> > Cc: Daniel Vetter <daniel@xxxxxxxx>; Leon Romanovsky <leon@xxxxxxxxxx>; linux-rdma@xxxxxxxxxxxxxxx; dri-devel@xxxxxxxxxxxxxxxxxxxxx; > Doug Ledford <dledford@xxxxxxxxxx>; Vetter, Daniel <daniel.vetter@xxxxxxxxx>; Christian Koenig <christian.koenig@xxxxxxx> > Subject: Re: [PATCH rdma-core 3/5] pyverbs: Add dma-buf based MR support > > On Wed, Nov 25, 2020 at 07:27:07PM +0000, Xiong, Jianxin wrote: > > > From: Jason Gunthorpe <jgg@xxxxxxxx> > > > Sent: Wednesday, November 25, 2020 4:15 AM > > > To: Daniel Vetter <daniel@xxxxxxxx> > > > Cc: Xiong, Jianxin <jianxin.xiong@xxxxxxxxx>; Leon Romanovsky > > > <leon@xxxxxxxxxx>; linux-rdma@xxxxxxxxxxxxxxx; dri- > > > devel@xxxxxxxxxxxxxxxxxxxxx; Doug Ledford <dledford@xxxxxxxxxx>; > > > Vetter, Daniel <daniel.vetter@xxxxxxxxx>; Christian Koenig > > > <christian.koenig@xxxxxxx> > > > Subject: Re: [PATCH rdma-core 3/5] pyverbs: Add dma-buf based MR > > > support > > > > > > On Wed, Nov 25, 2020 at 11:50:41AM +0100, Daniel Vetter wrote: > > > > > > > Yeah imo makes sense. It's a bunch more code for you to make it > > > > work on > > > > i915 and amd, but it's not terrible. And avoids the dependencies, > > > > and also avoids the abuse of card* and dumb buffers. Plus not > > > > really more complex, you just need a table or something to match > > > > from the drm driver name to the driver-specific buffer create > > > > function. Everything else stays the same. > > > > > > If it is going to get more complicated please write it in C then. We > > > haven't done it yet, but you can link a C function through cython to > > > the python test script > > > > > > If you struggle here I can probably work out the build system bits, > > > but it should not be too terrible > > > > Thanks Daniel and Jason. I have started working in this direction. > > There should be no technical obstacle here. > > Just to be clear I mean write some 'get dma buf fd' function in C, not the whole test > Yes, that's my understanding.
