Re: [Regression] Crash in load_module() while freeing args

[Linus Torvalds]

On Thu, 27 May 2010, Rafael J. Wysocki wrote:
> 
> Since the commit has been reverted, do you still want me to test this patch?
> Quite frankly I'd prefer to test a complete replacement for that commit on top
> of current -git.

I'm not re-applying it with the pointless semantic changes that are 
visible to modules. It doesn't matter if they were informed, if it means 
that they'll then just have some odd version dependency and add crazy 
"#if LINUX_VERSION" tests that aren't even exact.

I also wonder exactly what that module_mutex() actually ends up 
protecting. 99% of load_module() seems to be stuff that is purely about 
local issues. Maybe we could tighten the actual lock section to just the
parts that actually expose the vmalloc'ed area to others?

It's generally pointless releasing a lock in the middle - that just makes 
the lock even less valid. If it's valid to just release the lock (without 
some retry logic starting everything from the beginning), it likely the 
lock shouldn't have been held there in the first place.

		Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/