lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Fri, 17 Apr 2009 02:01:42 +0200
From:	Ingo Molnar <mingo@...e.hu>
To:	Al Viro <viro@...IV.linux.org.uk>
Cc:	Alessio Igor Bogani <abogani@...ware.it>,
	Alexander Viro <viro@....linux.org.uk>,
	Frederic Weisbecker <fweisbec@...il.com>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	LKML <linux-kernel@...r.kernel.org>,
	Jonathan Corbet <corbet@....net>
Subject: Re: [PATCH -tip] remove the BKL: Replace BKL in mount/umount
	syscalls with a mutex


* Al Viro <viro@...IV.linux.org.uk> wrote:

> remount is potentially nastier, but then it *is* nasty.  Again, 
> it's only per-fs stuff, so the obvious first step is taking BKL 
> down into the instances. It doesn't protect anything in VFS; all 
> uses are fs internal, so that'll take review of individual 
> filesystems.
> 
> NOTE: do not assume that code in fs/foo/* is correct; "it doesn't 
> take BKL elsewhere" does _not_ mean that we don't have races.  
> IOW, the same review ought to look for such beasts and deal with 
> them.  Mere "oh, no BKL anywhere in that fs" is not enough to 
> discard the ->remount_fs() instance.

what kind of races do you mean? Timing sensitive ones that are there 
just are not easy to trigger with the BKL held?

Or actual locking interaction between that body of BKL code and all 
other BKL using code?

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

Powered by blists - more mailing lists