module: move find_module check to end
authorLinus Torvalds <torvalds@linux-foundation.org>
Sat, 5 Jun 2010 17:17:36 +0000 (11:17 -0600)
committerRusty Russell <rusty@rustcorp.com.au>
Sat, 5 Jun 2010 01:47:37 +0000 (10:47 +0930)
commit3bafeb6247042dcbb72b0141ec7c7107de9f0b99
treef9b8f2437455332cd64d94bedb0836ac0f2b6fd9
parent75676500f8298f0ee89db12db97294883c4b768e
module: move find_module check to end

I think Rusty may have made the lock a bit _too_ finegrained there, and
didn't add it to some places that needed it. It looks, for example, like
PATCH 1/2 actually drops the lock in places where it's needed
("find_module()" is documented to need it, but now load_module() didn't
hold it at all when it did the find_module()).

Rather than adding a new "module_loading" list, I think we should be able
to just use the existing "modules" list, and just fix up the locking a
bit.

In fact, maybe we could just move the "look up existing module" a bit
later - optimistically assuming that the module doesn't exist, and then
just undoing the work if it turns out that we were wrong, just before
adding ourselves to the list.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
kernel/module.c