On Sat Feb 17 00:24:56 2024 +0000, Paul Gofman wrote:
FWIW my own major concern with the patch is that (hopefully) upcoming ntsync changes should render it useless (as it essentially is already with esync / fsync out of tree patches). In fact, adding a facility to request APC processing from the calling thread upon server call exit (as I suggested in another comment) also won't help much as in many practically interesting cases the async completion is triggered by server fd poll without any calling thread present (named pipes with IO fully performed through server are an exception but probably less interesting perf wise than sockets).
A faster sync will speed up the wait operation for sure, but I don't yet understand how the faster sync will make this patch unnecessary.