On Fri, Sep 30, 2022 12:59 AM Bob Pearson wrote: > On 9/26/22 20:38, matsuda-daisuke@xxxxxxxxxxx wrote: > > On Tue, Sep 27, 2022 8:14 AM, Jason Gunthorpe wrote: > >> On Fri, Sep 16, 2022 at 10:10:51PM -0500, Bob Pearson wrote: > >>> This patch series implements the xrc transport for the rdma_rxe driver. > >>> It is based on the current for-next branch of rdma-linux. > >>> The first two patches in the series do some cleanup which is helpful > >>> for this effort. The remaining patches implement the xrc functionality. > >>> There is a matching patch set for the user space rxe provider driver. > >>> The communications between these is accomplished without making an > >>> ABI change by taking advantage of the space freed up by a recent > >>> patch called "Remove redundant num_sge fields" which is a reprequisite > >>> for this patch series. > >>> > >>> The two patch sets have been tested with the pyverbs regression test > >>> suite with and without each set installed. This series enables 5 of > >>> the 6 xrc test cases in pyverbs. The ODP case does is currently skipped > >>> but should work once the ODP patch series is accepted. > >> > >> The ODP patch isn't even on patchworks any more, so it needs > >> resending. I can't remember why it needed respin now. > > > > The ODP patch series is the one I posted for rxe this month: > > [RFC PATCH 0/7] RDMA/rxe: On-Demand Paging on SoftRoCE > > https://lore.kernel.org/lkml/cover.1662461897.git.matsuda-daisuke@xxxxxxxxxxx/ > > https://patchwork.kernel.org/project/linux-rdma/list/?series=674699&state=%2A&archive=both > > > > We had an argument about the way to use workqueues instead of tasklets. > > Some prefer to get rid of tasklets, but others prefer finding a way to switch > > between the bottom halves for performance reasons. I am currently taking > > some data to continue the discussion while waiting for Bob to post their(HPE's) > > implementation that enables the switching. I think I can resend the ODP patches > > without an RFC tag once we reach an agreement on this point. > > I tried to get Ian Ziemba, who wrote the patch series, to send it in but he is very busy > and finally after a long while he asked me to do that. I have to rebase it from > an older kernel to head of tree. I hope to have that done in a day or two. > > Bob Thank you for the update. I am glad to hear your work is in progress just now. I am looking forward to seeing your work! Daisuke > > > > Thanks, > > Daisuke Matsuda > > > >> > >> I'm inclined to apply this without really looking closely at the rxe > >> code. If someone has other ideas please chime in. It looks like it > >> needs rebasing, yes? > >> > >> Jason >
