-----Original Message----- > From: Yishai Hadas <yishaih@xxxxxxxxxx> > Sent: Tuesday, January 12, 2021 4:49 AM > To: Xiong, Jianxin <jianxin.xiong@xxxxxxxxx>; Alex Deucher <alexdeucher@xxxxxxxxx> > Cc: Jason Gunthorpe <jgg@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>; Yishai > Hadas <yishaih@xxxxxxxxxx> > Subject: Re: [PATCH v16 0/4] RDMA: Add dma-buf support > > On 1/11/2021 7:55 PM, Xiong, Jianxin wrote: > >> -----Original Message----- > >> From: Alex Deucher <alexdeucher@xxxxxxxxx> > >> Sent: Monday, January 11, 2021 9:47 AM > >> To: Xiong, Jianxin <jianxin.xiong@xxxxxxxxx> > >> Cc: Jason Gunthorpe <jgg@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 v16 0/4] RDMA: Add dma-buf support > >> > >> On Mon, Jan 11, 2021 at 12:44 PM Xiong, Jianxin <jianxin.xiong@xxxxxxxxx> wrote: > >>>> -----Original Message----- > >>>> From: Jason Gunthorpe <jgg@xxxxxxxx> > >>>> Sent: Monday, January 11, 2021 7:43 AM > >>>> To: Xiong, Jianxin <jianxin.xiong@xxxxxxxxx> > >>>> Cc: linux-rdma@xxxxxxxxxxxxxxx; dri-devel@xxxxxxxxxxxxxxxxxxxxx; > >>>> Doug Ledford <dledford@xxxxxxxxxx>; Leon Romanovsky > >>>> <leon@xxxxxxxxxx>; Sumit Semwal <sumit.semwal@xxxxxxxxxx>; > >>>> Christian Koenig <christian.koenig@xxxxxxx>; Vetter, Daniel > >>>> <daniel.vetter@xxxxxxxxx> > >>>> Subject: Re: [PATCH v16 0/4] RDMA: Add dma-buf support > >>>> > >>>> On Mon, Jan 11, 2021 at 03:24:18PM +0000, Xiong, Jianxin wrote: > >>>>> Jason, will this series be able to get into 5.12? > >>>> I was going to ask you where things are after the break? > >>>> > >>>> Did everyone agree the userspace stuff is OK now? Is Edward OK with > >>>> the pyverbs changes, etc > >>>> > >>> There is no new comment on the both the kernel and userspace series. > >>> I assume silence means no objection. I will ask for opinions on the userspace thread. > >> Do you have a link to the userspace thread? > >> > > https://www.spinics.net/lists/linux-rdma/msg98135.html > > > Any reason why the 'fork' comment that was given few times wasn't not handled / answered ? > > Specifically, > > ibv_reg_dmabuf_mr() doesn't call ibv_dontfork_range() but ibv_dereg_mr does call its opposite API (i.e. ibv_dofork_range()) > Sorry, that part was missed. Strangely enough, a few of your replies didn't reach my inbox and I just found them in the web archives: https://www.spinics.net/lists/linux-rdma/msg97973.html, and https://www.spinics.net/lists/linux-rdma/msg98133.html I will add check to ibv_dereg_mr() to avoid calling ibv_ibv_dofork_range() for dmabuf case. Thanks a lot for bring this up again. Jianxin
