cvs commit: src/sys/kern tty.c

M. Warner Losh imp at bsdimp.com
Tue Dec 19 14:55:19 PST 2006


In message: <20061219194842.M53548 at godot.imp.ch>
            Martin Blapp <mb at 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


More information about the cvs-src mailing list