cvs commit: src/sys/kern tty.c

Sam Leffler sam at errno.com
Tue Dec 19 09:45:10 PST 2006


Martin Blapp wrote:
> 
> Hi Sam,
> 
> I'll have got again a setup box in the next week and work hard
> with bde, gib and tegge to solve them. For RELENG_6_2 it's
> too late. I hope to get that small workaround again committed,
> but for more serios work we would have to delay 6.2 for half a
> year ;-)
> 
> The full tty locking stuff is also pendent, maybe I'll grab that
> and make it work. Unfortunatly this will open again a new 'can of worms':
> new races will appear.
> 
> Martin
> 
>>>   Modified files:
>>>     sys/kern             tty.c
>>>   Log:
>>>   Add the tp->t_refcnt validity check back. There are still some race
>>>   conditions where tp->t_refcnt can go to zero.
>>
>> Can you please identify those races?

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.

	Sam


More information about the cvs-all mailing list