From owner-freebsd-net@FreeBSD.ORG Tue Feb 28 03:03:07 2006 Return-Path: X-Original-To: freebsd-net@freebsd.org Delivered-To: freebsd-net@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 2436316A420 for ; Tue, 28 Feb 2006 03:03:07 +0000 (GMT) (envelope-from jinmei@isl.rdc.toshiba.co.jp) Received: from shuttle.wide.toshiba.co.jp (shuttle.wide.toshiba.co.jp [202.249.10.124]) by mx1.FreeBSD.org (Postfix) with ESMTP id A5F3D43D48 for ; Tue, 28 Feb 2006 03:03:06 +0000 (GMT) (envelope-from jinmei@isl.rdc.toshiba.co.jp) Received: from impact.jinmei.org (unknown [2001:200:0:8002:fc4c:87b0:3965:cfb3]) by shuttle.wide.toshiba.co.jp (Postfix) with ESMTP id 2F7BD15228; Tue, 28 Feb 2006 12:03:05 +0900 (JST) Date: Tue, 28 Feb 2006 12:03:03 +0900 Message-ID: From: JINMEI Tatuya / =?ISO-2022-JP?B?GyRCP0BMQEMjOkgbKEI=?= To: Craig Boston In-Reply-To: <20060227200301.GB247@nowhere> References: <20060125152032.GA40581@nowhere> <20060127020528.GA18728@nowhere> <20060227200301.GB247@nowhere> User-Agent: Wanderlust/2.14.0 (Africa) Emacs/21.3 Mule/5.0 (SAKAKI) Organization: Research & Development Center, Toshiba Corp., Kawasaki, Japan. MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Cc: freebsd-net@freebsd.org Subject: Re: Race condition in ip6_getpmtu (actually gif)? X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 28 Feb 2006 03:03:07 -0000 >>>>> On Mon, 27 Feb 2006 14:03:01 -0600, >>>>> Craig Boston said: >> > Attached is a quick hack to protect the cached route with a mutex. A >> > better fix with less overhead would be to allocate the route in a local >> > variable on the stack, and only copy it to the softc if route caching is >> > enabled. I'll run for a couple weeks with the patch and file a PR if >> > that fixes it. >> >> I guess this problem was fixed with the following changes: >> >> http://www.jp.freebsd.org/cgi/cvsweb.cgi/src/sys/net/if_gif.c.diff?r1=1.57&r2=1.58 >> http://www.freebsd.org/cgi/cvsweb.cgi/src/sys/netinet6/in6_gif.c.diff?r1=1.23&r2=1.24 > Yes, it was. I saw the changes in those files when I attempted to > re-merge my patch after a cvsup. I've currently been running with no > panics for 12 days with rev 1.52.2.4 of if_gif.c and rev 1.22.2.2 of > in6_gif.c. Okay, thanks for the confirmation. JINMEI, Tatuya Communication Platform Lab. Corporate R&D Center, Toshiba Corp. jinmei@isl.rdc.toshiba.co.jp