From owner-freebsd-stable@FreeBSD.ORG Fri Jun 23 17:29:03 2006 Return-Path: X-Original-To: freebsd-stable@freebsd.org Delivered-To: freebsd-stable@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 1A70D16A494; Fri, 23 Jun 2006 17:29:03 +0000 (UTC) (envelope-from max@love2party.net) Received: from moutng.kundenserver.de (moutng.kundenserver.de [212.227.126.188]) by mx1.FreeBSD.org (Postfix) with ESMTP id 0DDD143D45; Fri, 23 Jun 2006 17:29:01 +0000 (GMT) (envelope-from max@love2party.net) Received: from [88.64.181.119] (helo=amd64.laiers.local) by mrelayeu.kundenserver.de (node=mrelayeu2) with ESMTP (Nemesis), id 0MKwtQ-1FtpSl3zdd-0001SI; Fri, 23 Jun 2006 19:29:00 +0200 From: Max Laier Organization: FreeBSD To: freebsd-stable@freebsd.org Date: Fri, 23 Jun 2006 19:28:48 +0200 User-Agent: KMail/1.9.1 References: <20060621202508.S17514@godot.imp.ch> <20060623133915.S14714@godot.imp.ch> <1151078632.62769.30.camel@buffy.york.ac.uk> In-Reply-To: <1151078632.62769.30.camel@buffy.york.ac.uk> X-Face: ,,8R(x[kmU]tKN@>gtH1yQE4aslGdu+2]; R]*pL,U>^H?)gW@49@wdJ`H<=?utf-8?q?=25=7D*=5FBD=0A=09U=5For=3D=5CmOZf764=26nYj=3DJYbR1PW0ud?=>|!~,,CPC.1-D$FG@0h3#'5"k{V]a~.<=?utf-8?q?mZ=7D44=23Se=7Em=0A=09Fe=7E=5C=5DX5B=5D=5Fxj?=(ykz9QKMw_l0C2AQ]}Ym8)fU MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPart4714497.KMY0BdFFgL"; protocol="application/pgp-signature"; micalg=pgp-sha1 Content-Transfer-Encoding: 7bit Message-Id: <200606231928.58063.max@love2party.net> X-Provags-ID: kundenserver.de abuse@kundenserver.de login:61c499deaeeba3ba5be80f48ecc83056 Cc: "Wojciech A. Koszek" , csjp@freebsd.org, Martin Blapp , Robert Watson , Patrick Guelat Subject: Re: Crash with FreeBSD 6.1 STABLE of today X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 Jun 2006 17:29:03 -0000 --nextPart4714497.KMY0BdFFgL Content-Type: text/plain; charset="iso-8859-6" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline On Friday 23 June 2006 18:03, Gavin Atkinson wrote: > On Fri, 2006-06-23 at 13:46 +0200, Martin Blapp wrote: > > Hi, > > > > Maybe this is the solution ? IMHO there is a race window > > open between the first tp->t_session test and the locking > > of the proc tree. > > I'm not sure if t_session is supposed to be protected by the proctree > lock though. With an initial glance of the code, it would seem odd to > be protected by the proctree lock, although I can't see any other locks > Someone with more knowledge of this code will probably know the answer > to this. > > There does seem to be a worrying comment above tty_close (which is the > only place that t_session seems to be set to NULL): > > * XXX our caller should have done `spltty(); l_close(); tty_close();' > * and l_close() should have flushed, but we repeat the spltty() and > * the flush in case there are buggy callers. > > As I understand it, spltty() is now a no-op. Does this mean that this > code is now essentially running without any locks that were used to > serialise changes to struct tty in days gone by? Or is the whole tty > subsystem still running under Giant? Just as a random note. I remember looking at a somewhat similar problem=20 during the 6.0 release cycle and comming up with a patch[1]. Somehow the=20 problem went away and I lost track of the patch. Can't remember who was=20 testing it. I think I talked with Christan about it, though [CC-list++] Note, I am not sure if the patch still applies or is correct at all, but fr= om=20 looking at it (and the name of the file) I seem to remember that there was = a=20 problem with t_pgrp and t_session being accessed unlocked in some places. = =20 Maybe it helps, maybe it doesn't. [1] http://people.freebsd.org/~mlaier/tty.t_pgrp.diff =2D-=20 /"\ Best regards, | mlaier@freebsd.org \ / Max Laier | ICQ #67774661 X http://pf4freebsd.love2party.net/ | mlaier@EFnet / \ ASCII Ribbon Campaign | Against HTML Mail and News --nextPart4714497.KMY0BdFFgL Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.2.2 (FreeBSD) iD8DBQBEnCTaXyyEoT62BG0RAqicAJ99MNWRCj8xsQmFuUi2B304EyxpuACfTEsv V9ehBvanTSwgvdSN4UNKgRI= =y6go -----END PGP SIGNATURE----- --nextPart4714497.KMY0BdFFgL--