From owner-cvs-all@FreeBSD.ORG Tue Mar 29 10:23:12 2005 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id D70C116A4CE; Tue, 29 Mar 2005 10:23:12 +0000 (GMT) Received: from mail.chesapeake.net (chesapeake.net [208.142.252.6]) by mx1.FreeBSD.org (Postfix) with ESMTP id 5814F43D4C; Tue, 29 Mar 2005 10:23:12 +0000 (GMT) (envelope-from jroberson@chesapeake.net) Received: from mail.chesapeake.net (localhost [127.0.0.1]) by mail.chesapeake.net (8.12.10/8.12.10) with ESMTP id j2TANB9P094160; Tue, 29 Mar 2005 05:23:11 -0500 (EST) (envelope-from jroberson@chesapeake.net) Received: from localhost (jroberson@localhost)j2TANBg8094157; Tue, 29 Mar 2005 05:23:11 -0500 (EST) (envelope-from jroberson@chesapeake.net) X-Authentication-Warning: mail.chesapeake.net: jroberson owned process doing -bs Date: Tue, 29 Mar 2005 05:23:11 -0500 (EST) From: Jeff Roberson To: David Malone In-Reply-To: <20050329102154.GA20842@walton.maths.tcd.ie> Message-ID: <20050329052244.J54623@mail.chesapeake.net> References: <200503290958.j2T9wfq0025519@repoman.freebsd.org> <20050329102154.GA20842@walton.maths.tcd.ie> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: cvs-src@FreeBSD.org cc: Jeff Roberson cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern vfs_cache.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 29 Mar 2005 10:23:13 -0000 On Tue, 29 Mar 2005, David Malone wrote: > On Tue, Mar 29, 2005 at 09:58:41AM +0000, Jeff Roberson wrote: > > Log: > > - Invalidate the childrens v_dd pointers when we cache_purge() a directory. > > Otherwise the stale pointer may be accessed after a vnode is freed. > > I guess this has no impact on how likely we are to have a call to > realpath work because in the old version the generation number of > the vnode would have changed anyway? Yes, that is correct. This change eliminates a dependency on type-stable vnode storage that I missed. > > David. >