On Thu, 8 Aug 2024 at 22:19, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote: > > End result: I now have a new plan - I'll make the > wait_for_completion(&idem.complete) be interruptible and return -EINTR > (and I'll have to clean up the wait-queues etc). .. and that seems to have been pretty straightforward, and creating a test-module that just recursively does a "request_module()" of itself shows that it all seems to work. I've committed it and marked it as Fixes: 9b9879fc0327 ("modules: catch concurrent module loads, treat them as idempotent") but it shouldn't actually matter for any non-buggy module situation. Linus