From owner-cvs-src@FreeBSD.ORG Tue Apr 12 00:17:27 2005 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 41D7D16A4CE; Tue, 12 Apr 2005 00:17:27 +0000 (GMT) Received: from sccrmhc11.comcast.net (sccrmhc11.comcast.net [204.127.202.55]) by mx1.FreeBSD.org (Postfix) with ESMTP id 65C9F43D5C; Tue, 12 Apr 2005 00:17:26 +0000 (GMT) (envelope-from dougb@freebsd.org) Received: from [192.0.35.182] (g35-182.icann.org[192.0.35.182]) by comcast.net (sccrmhc11) with ESMTP id <20050412001721011001cq37e>; Tue, 12 Apr 2005 00:17:25 +0000 Message-ID: <425B1390.5030105@freebsd.org> Date: Mon, 11 Apr 2005 17:17:20 -0700 From: Doug Barton Organization: http://www.FreeBSD.org User-Agent: Mozilla Thunderbird 1.0.2 (X11/20050326) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Jung-uk Kim References: <200504111519.j3BFJTSx095645@repoman.freebsd.org> <425ACE7F.7050609@freebsd.org> <200504111634.03435.jkim@niksun.com> In-Reply-To: <200504111634.03435.jkim@niksun.com> X-Enigmail-Version: 0.91.0.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit 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 vnode_if.src X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 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: Tue, 12 Apr 2005 00:17:27 -0000 Jung-uk Kim wrote: > On Monday 11 April 2005 03:22 pm, Doug Barton wrote: > >>Jeff Roberson wrote: >> >>>jeff 2005-04-11 15:19:29 UTC >>> >>> FreeBSD src repository >>> >>> Modified files: >>> sys/kern vnode_if.src >>> Log: >>> - Mark the VOPs that require exclusive locks. Those that >>>aren't marked with E may be called with a shared lock held. This >>>list really could be made per filesystem if we had any >>>filesystems which differed from ffs in locking guarantees. VFS >>>itself is not sensitive to this except where vgone() etc. are >>>concerned. >>> >>> Sponsored by: Isilon Systems, Inc. >>> >>> Revision Changes Path >>> 1.77 +26 -26 src/sys/kern/vnode_if.src >>> >>>http://www.FreeBSD.org/cgi/cvsweb.cgi/src/sys/kern/vnode_if.src.d >>>iff?&r1=1.76&r2=1.77&f=h >> >>I now have a broken build: >> >>cc -c -O -pipe -march=pentium4m -Wall -Wredundant-decls >>-Wnested-externs -Wstric t-prototypes -Wmissing-prototypes >>-Wpointer-arith -Winline -Wcast-qual -fforma t-extensions -std=c99 >>-g -nostdinc -I- -I. -I/usr/local/src/sys -I/usr/local/sr >>c/sys/contrib/dev/acpica -I/usr/local/src/sys/contrib/altq >>-I/usr/local/src/sys/ contrib/ipfilter >>-I/usr/local/src/sys/contrib/pf -I/usr/local/src/sys/contrib/de >>v/ath -I/usr/local/src/sys/contrib/dev/ath/freebsd >>-I/usr/local/src/sys/contrib/ ngatm -D_KERNEL -include opt_global.h >>-fno-common -finline-limit=8000 --param in line-unit-growth=100 >>--param large-function-growth=1000 -mno-align-long-strings >>-mpreferred-stack-boundary=2 -mno-mmx -mno-3dnow -mno-sse >>-mno-sse2 -ffreestan >>ding -Werror vnode_if.c >>vnode_if.c: In function `VOP_CREATE_APV': >>vnode_if.c:205: warning: implicit declaration of function >>`ASSERT_VOP_ELOCKED' vnode_if.c:205: warning: nested extern >>declaration of `ASSERT_VOP_ELOCKED' vnode_if.c: In function >>`VOP_WHITEOUT_APV': >>vnode_if.c:261: warning: nested extern declaration of >>`ASSERT_VOP_ELOCKED' vnode_if.c:205: warning: redundant >>redeclaration of 'ASSERT_VOP_ELOCKED' vnode_if.c:205: warning: >>previous implicit declaration of 'ASSERT_VOP_ELOCKED' w as here >>vnode_if.c: In function `VOP_MKNOD_APV': >>vnode_if.c:315: warning: nested extern declaration of >>`ASSERT_VOP_ELOCKED' vnode_if.c:205: warning: redundant >>redeclaration of 'ASSERT_VOP_ELOCKED' vnode_if.c:205: warning: >>previous implicit declaration of 'ASSERT_VOP_ELOCKED' w as here >> >>and on and on. >> >>Doug > > > FYI, the following patch fixes it. > > Jung-uk Kim > > Index: vnode.h > =================================================================== > RCS file: /home/ncvs/src/sys/sys/vnode.h,v > retrieving revision 1.296 > diff -u -r1.296 vnode.h > --- src/sys/sys/vnode.h 11 Apr 2005 15:17:06 -0000 1.296 > +++ src/sys/sys/vnode.h 11 Apr 2005 20:31:15 -0000 > @@ -514,8 +514,8 @@ > > #define ASSERT_VI_LOCKED(vp, str) > #define ASSERT_VI_UNLOCKED(vp, str) > -#if 0 > #define ASSERT_VOP_ELOCKED(vp, str) > +#if 0 > #define ASSERT_VOP_ELOCKED_OTHER(vp, str) > #endif > #define ASSERT_VOP_LOCKED(vp, str) > Thank you, re-compiling now. Have you by chance confirmed that the kernel this produces is safe to use? Doug -- This .signature sanitized for your protection