From owner-cvs-src@FreeBSD.ORG Sun Dec 31 13:24:20 2006 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 1D8F816A40F; Sun, 31 Dec 2006 13:24:20 +0000 (UTC) (envelope-from itetcu@FreeBSD.org) Received: from it.buh.tecnik93.com (it.buh.tecnik93.com [81.196.204.98]) by mx1.freebsd.org (Postfix) with ESMTP id CA3B613C461; Sun, 31 Dec 2006 13:24:19 +0000 (UTC) (envelope-from itetcu@FreeBSD.org) Received: from it.buh.tecnik93.com (localhost [127.0.0.1]) by it.buh.tecnik93.com (Postfix) with ESMTP id 6CA031711B; Sun, 31 Dec 2006 14:53:02 +0200 (EET) Date: Sun, 31 Dec 2006 14:53:00 +0200 From: Ion-Mihai "IOnut" Tetcu To: Robert Watson Message-ID: <20061231145300.49313fe8@it.buh.tecnik93.com> In-Reply-To: <200612301603.kBUG3iRp078184@repoman.freebsd.org> References: <200612301603.kBUG3iRp078184@repoman.freebsd.org> X-Mailer: Sylpheed-Claws 2.6.0 (GTK+ 2.10.6; i386-portbld-freebsd6.2) Mime-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_7G7PwxXm=dO4Rm4/dCIBWHc"; protocol="application/pgp-signature"; micalg=PGP-SHA1 Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/netinet tcp_subr.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 31 Dec 2006 13:24:20 -0000 --Sig_7G7PwxXm=dO4Rm4/dCIBWHc Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Sat, 30 Dec 2006 16:03:44 +0000 (UTC) Robert Watson wrote: > rwatson 2006-12-30 16:03:44 UTC >=20 > FreeBSD src repository >=20 > Modified files: (Branch: RELENG_6) > sys/netinet tcp_subr.c=20 > Log: > Merge tcp_subr.c:1.251 from HEAD to RELENG_6: > =20 > Fix race conditions on enumerating pcb lists by moving the > initialization (and where appropriate the destruction) of the pcb > mutex to the init/finit functions of the pcb zones. This allows > locking of the pcb entries and race condition free comparison of the > generation count. Rearrange locking a bit to avoid extra locking > operation to update the generation count in in_pcballoc(). > (in_pcballoc now returns the pcb locked) > =20 > I am planning to convert pcb list handling from a type safe to a > reference count model soon. (As this allows really freeing the > PCBs)=20 > Reviewed by: rwatson, mohans > =20 > This was missed in jhb's earlier MFC, and lead to a panic when > creating TCP sockets. Thanks you ! I lost my last night plus part of the day trying to understand why a server suddenly panic'ed on boot :( Now I know :-) --=20 IOnut - Un^d^dregistered ;) FreeBSD "user" "Intellectual Property" is nowhere near as valuable as "Intellect" BOFH excuse #152: My pony-tail hit the on/off switch on the power strip --Sig_7G7PwxXm=dO4Rm4/dCIBWHc Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.6 (FreeBSD) iD8DBQFFl7KtBX6fi0k6KXsRAn5WAKCoyHyGa91vSJwKRoDjecvQTpdtTQCcCnhN GjJyEj3yt+cJmPg5ORWO5tg= =xbST -----END PGP SIGNATURE----- --Sig_7G7PwxXm=dO4Rm4/dCIBWHc--