On Fri Feb 16 17:18:37 2024 +0000, Jinoh Kang wrote:
I think this will keep the test intact. Do you have any specific use case against this?
if (thread != current && process && (candidate = find_thread_in_apc_wait( process )))
I don't immediately see how this helps. It will remove the effect of the patch when async IO is going to be queued by the same thread which queued original async OP which will undesirably limit optimization scope. Also, in the view of I described, it would "fix" only the limited, not the most practically interesting aspect of the problem. I suppose the choice here is between: - Go without such a limitation; - Even if it looks like a practically least important case, don't touch it turning the things into todo until we have some way to solve the problem in general (fwiw I tried thinking of how to solve that minor sync problem but didn't have immediate good ideas how that can be synchronized without introducing convoluted and perf impacting synchronization).