Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 22 Jun 2002 22:27:10 -0700
From:      Alfred Perlstein <bright@mu.org>
To:        Alan Cox <alc@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/vm vm_mmap.c
Message-ID:  <20020623052710.GK53232@elvis.mu.org>
In-Reply-To: <200206230148.g5N1mN795443@freefall.freebsd.org>
References:  <200206230148.g5N1mN795443@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
* Alan Cox <alc@FreeBSD.org> [020622 18:48] wrote:
> alc         2002/06/22 18:48:23 PDT
> 
>   Modified files:
>     sys/vm               vm_mmap.c 
>   Log:
>    o Remove the unnecessary acquisition and release of Giant around fdrop()
>      in mmap(2).

I think fdrop() may need Giant in case a race with a thread makes
it the instance of fdrop() that leads to fclose() being called which
can call down into the VFS code.

-- 
-Alfred Perlstein [alfred@freebsd.org]
'Instead of asking why a piece of software is using "1970s technology,"
 start asking why software is ignoring 30 years of accumulated wisdom.'
Tax deductible donations for FreeBSD: http://www.freebsdfoundation.org/

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020623052710.GK53232>