From owner-cvs-all@FreeBSD.ORG  Tue Dec 19 17:42:14 2006
Return-Path: <owner-cvs-all@FreeBSD.ORG>
X-Original-To: cvs-all@FreeBSD.org
Delivered-To: cvs-all@FreeBSD.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id 23D3216A403;
	Tue, 19 Dec 2006 17:42:14 +0000 (UTC) (envelope-from mb@imp.ch)
Received: from mx2.imp.ch (mx2.imp.ch [157.161.9.17])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 8643043CA0;
	Tue, 19 Dec 2006 17:42:13 +0000 (GMT) (envelope-from mb@imp.ch)
Received: from godot.imp.ch (godot.imp.ch [157.161.4.8])
	by pop.imp.ch (8.13.8/8.13.8/Submit_imp) with ESMTP id kBJHKIEf046664; 
	Tue, 19 Dec 2006 18:20:19 +0100 (CET) (envelope-from mb@imp.ch)
Date: Tue, 19 Dec 2006 18:20:18 +0100 (CET)
From: Martin Blapp <mb@imp.ch>
To: Sam Leffler <sam@errno.com>
In-Reply-To: <45881D46.8090201@errno.com>
Message-ID: <20061219181505.L53548@godot.imp.ch>
References: <200612191646.kBJGkEln071323@repoman.freebsd.org>
	<45881D46.8090201@errno.com>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed
X-Scanned-By: MIMEDefang 2.58 on 157.161.9.65
Cc: cvs-src@FreeBSD.org, Martin Blapp <mbr@FreeBSD.org>, cvs-all@FreeBSD.org,
	src-committers@FreeBSD.org
Subject: Re: cvs commit: src/sys/kern tty.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 19 Dec 2006 17:42:14 -0000


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?
>
> 	Sam
>