[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: CVS commit: src/sys/kern
On Mon, Jul 26, 2010 at 06:33:57PM +0300, Antti Kantee wrote:
> On Mon Jul 26 2010 at 15:22:17 +0000, Juergen Hannken-Illjes wrote:
> > Module Name: src
> > Committed By: hannken
> > Date: Mon Jul 26 15:22:16 UTC 2010
> > Modified Files:
> > src/sys/kern: vfs_subr.c
> > Log Message:
> > When both vget() and vrelel() call vn_lock() we know VI_XLOCK is clear.
> > No need to use LK_INTERLOCK or LK_RETRY here.
> I'd like the commit message to include the essential bit about them
> examining the return value and not blindly plowing on after vn_lock().
> Purely from the commit message I get the impression that this introduced
> a race condition.
Commit message changed as:
When both vget() and vrelel() call vn_lock() we know VI_XLOCK is clear.
No need to use LK_INTERLOCK or LK_RETRY here.
The return value of vn_lock() is already examined here.
Ok: Antti Kantee <pooka%netbsd.org@localhost>
Juergen Hannken-Illjes - hannken%eis.cs.tu-bs.de@localhost - TU Braunschweig
Main Index |
Thread Index |