> On Jul 14, 2022, at 7:46 PM, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote: > > On Fri, 15 Jul 2022 02:04:33 +0000 > Song Liu <songliubraving@xxxxxx> wrote: > >>> What I'm suggesting is that a DIRECT ops will never set IPMODIFY. >> >> Aha, this the point I misunderstood. I thought DIRECT ops would always >> set IPMODIFY (as it does now). > > My fault. I was probably not being clear when I was suggesting that > DIRECT should *act* like an IPMODIFY, but never explicitly stated that > it should not set the IPMODIFY flag. > > The only reason it does today was to make it easy to act like an > IPMODIFY (because it set the flag). But I'm now suggesting to get rid > of that and just make DIRECT act like an IPMDOFIY as there can only be > one of them on a function, but now we have some cases where DIRECT can > work with IPMODIFY via the callbacks. Thanks for the clarification. I think we are finally on the same page on this. :) Song
