Date: Tue, 19 Dec 2006 15:53:15 -0700 (MST) From: "M. Warner Losh" <imp@bsdimp.com> To: mb@imp.ch Cc: sam@errno.com, src-committers@FreeBSD.org, cvs-all@FreeBSD.org, cvs-src@FreeBSD.org Subject: Re: cvs commit: src/sys/kern tty.c Message-ID: <20061219.155315.756909997.imp@bsdimp.com> In-Reply-To: <20061219194842.M53548@godot.imp.ch> References: <20061219181505.L53548@godot.imp.ch> <4588251F.3040206@errno.com> <20061219194842.M53548@godot.imp.ch>
next in thread | previous in thread | raw e-mail | index | archive | help
In message: <20061219194842.M53548@godot.imp.ch>
Martin Blapp <mb@imp.ch> writes:
:
: Hi,
:
: > That is not what I was asking for. You said "there are races" but
: > didn't identify them. It would be helpful to know what these races are
: > so someone other than you doesn't need to guess why you made this change.
:
: The reason for this is currently unknown (it shouldn't happen, tegges patch
: should work) and I (and bde) didn't had the time to look into it. As promised
: I'll look why it happens and then I'll add a comment at the needed places.
If the races are unknown, how do you know that you've really fixed
things rather than introducing a random frob that does nothing but
confuse people who come along later?
Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20061219.155315.756909997.imp>