From owner-cvs-all Sat Sep 1 0:43:39 2001 Delivered-To: cvs-all@freebsd.org Received: from peter3.wemm.org (c1315225-a.plstn1.sfba.home.com [24.14.150.180]) by hub.freebsd.org (Postfix) with ESMTP id 6DEBD37B407; Sat, 1 Sep 2001 00:43:25 -0700 (PDT) Received: from overcee.netplex.com.au (overcee.wemm.org [10.0.0.3]) by peter3.wemm.org (8.11.0/8.11.0) with ESMTP id f817hPM21828; Sat, 1 Sep 2001 00:43:25 -0700 (PDT) (envelope-from peter@wemm.org) Received: from wemm.org (localhost [127.0.0.1]) by overcee.netplex.com.au (Postfix) with ESMTP id 28AE13807; Sat, 1 Sep 2001 00:43:25 -0700 (PDT) (envelope-from peter@wemm.org) X-Mailer: exmh version 2.3.1 01/18/2001 with nmh-1.0.4 To: Matt Dillon Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern subr_prof.c kern_ntptime.c kern_xxx.c In-Reply-To: <200109010547.f815lwK20565@freefall.freebsd.org> Date: Sat, 01 Sep 2001 00:43:25 -0700 From: Peter Wemm Message-Id: <20010901074325.28AE13807@overcee.netplex.com.au> Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Matt Dillon wrote: > dillon 2001/08/31 22:47:58 PDT > > Modified files: > sys/kern subr_prof.c kern_ntptime.c kern_xxx.c > Log: > Pushdown Giant for: profil(), ntp_adjtime(), ogethostname(), > osethostname(), ogethostid(), osethostid() Thanks, but can you please try to stop going out of your way to cause style(9) violations in previously compliant code? eg: +/* + * MPSAFE + */ int oquota(p, uap) struct proc *p; struct oquota_args *uap; { - return (ENOSYS); } #endif /* COMPAT_43 */ The specific section: static void usage() { /* Insert an empty line if the function has no local variables. */ Just about every one of these that you have touched like this has caused a conflict with the kse diff because of the following example: +/* + * MPSAFE (accept1() is MPSAFE) + */ int oaccept(p, uap) struct proc *p; struct accept_args *uap; { - return (accept1(p, uap, 1)); } #endif /* COMPAT_OLDSOCK */ rcsmerge implodes when two adjacent deltas like this touch. In KSE it was: #ifdef COMPAT_OLDSOCK int oaccept(td, uap) struct thread *td; struct accept_args *uap; { return (accept1(td, uap, 1)); } #endif /* COMPAT_OLDSOCK */ ie: your delta causes a conflict with the s/p/td/ change on the adjacent line. You are also occasionally adding unnecesary blank lines out of sync with the rest of the code, eg: @@ -118,14 +121,17 @@ int protocol; } */ *uap; { - struct filedesc *fdp = p->p_fd; + struct filedesc *fdp; struct socket *so; struct file *fp; int fd, error; + mtx_lock(&Giant); + + fdp = p->p_fd; error = falloc(p, &fp, &fd); if (error) - return (error); + goto done2; fhold(fp); error = socreate(uap->domain, &so, uap->type, uap->protocol, p); if (error) { The added blank one after the mtx_lock() is the only one in the function (socket()), for example. Dont get me wrong, I appreciate the effort you are going to. But it kinda irks me when I have to resolve merge conflicts where style bugs were added in the process. :-] Cheers, -Peter -- Peter Wemm - peter@FreeBSD.org; peter@yahoo-inc.com; peter@netplex.com.au "All of this is for nothing if we don't go to the stars" - JMS/B5 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message