Re: [PATCH 01/11] sysctl: Separate the binary sysctl logic into it's own file.

Arnd Bergmann
2009-11-06T13:11:29Z

On Friday 06 November 2009, Eric W. Biederman wrote:
> The primary proc path already doesn't need the lock_kernel(). My next
> patch winds up killing the entire binary path and rebuilding on top of
> /proc/sys. Which removes that lock_kernel().
>
> Which I think elegantly solves all of the sysctl BKL lock issues.

Yes, that sounds like an excellent plan, but I'm not completely sure
if the lack of the BKL in the procfs case is intentional. As a
particular case that I stumbled over, 'core_pattern' is read
with the BKL held to protect against sysctl changing it, but
it is changed with proc_dostring without the BKL.

Most uses of intvec or string seem to be racy and probably need
a proper serialization method anyway.