On Fri, Oct 14, 2022 2:18 AM Bob Pearson wrote: > On 10/12/22 02:41, matsuda-daisuke@xxxxxxxxxxx wrote: > > Hi Bob, > > > > I am ready and willing to review your workqueue implementation. > > Could you inform me of the current status of the patch series? > > Are you having trouble rebasing it? > > > > Daisuke > > > >>>> 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 > > > > I know. Thanks. It's ready to send in. I am testing the performance and was surprised that the tasklet performance has > gotten better in 6.0. Not sure why. I'll send it today. > > Bob Thank you for the reply. The performance change sounds interesting. BTW, would you please send it in as soon as possible? I would like to see if I can easily rebase my ODP patches on it, and also I will start testing and reviewing your change so that it will be merged safely and quickly. Daisuke
