Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 21 Jan 2003 12:31:05 -0800
From:      Alfred Perlstein <bright@mu.org>
To:        Jeffrey Hsu <hsu@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/kern kern_descrip.c
Message-ID:  <20030121203105.GJ33821@elvis.mu.org>
In-Reply-To: <200301212020.h0LKKmlk016779@repoman.freebsd.org>
References:  <200301212020.h0LKKmlk016779@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
* Jeffrey Hsu <hsu@FreeBSD.org> [030121 12:21] wrote:
> hsu         2003/01/21 12:20:48 PST
> 
>   Modified files:
>     sys/kern             kern_descrip.c 
>   Log:
>   Add missing SMP file locks around read-modify-write operations on
>   the flag field.

You shouldn't need these, if the user code races against itself to
set flags it shoudn't matter.  You can't prevent user races inside
the kernel.

>   Reviewed by:    rwatson

huh?  Sorry, but since when is Robert an SMP guru or filedesc guru?

-- 
-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.'

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?20030121203105.GJ33821>