On Wed, Feb 09, 2022 at 11:57:38AM -0800, Dan Williams wrote: > On Wed, Feb 9, 2022 at 8:58 AM Jonathan Cameron > <Jonathan.Cameron@xxxxxxxxxx> wrote: > [..] > > > It just seems an unnecessary hunk of code for the core to carry when > > > it's trivial for a client of the core to do: > > > > > > task->private = &completion; > > > task->end_task = complete_completion; > > > submit_task() > > > wait_for_completion(&completion); > > > > OK, we can move this to the callers though function obviously will > > also need renaming - I guess to pci_doe_exchange() and now need to take a > > task rather than the exchange. > > > > I personally slightly prefer the layered approach, but don't care that > > strongly. > > Like I said, you and Ira are holding the pen, so if you decide to keep > the layering, just document the ontology somewhere and I'll let it go. I'm busy with the PKS series ATM but I should get back to reviewing all these comments soon. Ira