From owner-cvs-all@FreeBSD.ORG Fri Nov 14 08:56:07 2003 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 E143A16A4CE; Fri, 14 Nov 2003 08:56:07 -0800 (PST) Received: from harmony.village.org (rover.bsdimp.com [204.144.255.66]) by mx1.FreeBSD.org (Postfix) with ESMTP id B0DD843F85; Fri, 14 Nov 2003 08:53:57 -0800 (PST) (envelope-from imp@bsdimp.com) Received: from localhost (warner@rover2.village.org [10.0.0.1]) by harmony.village.org (8.12.9p2/8.12.9) with ESMTP id hAEGrveG094915; Fri, 14 Nov 2003 09:53:57 -0700 (MST) (envelope-from imp@bsdimp.com) Date: Fri, 14 Nov 2003 09:53:39 -0700 (MST) Message-Id: <20031114.095339.121107822.imp@bsdimp.com> To: green@FreeBSD.org From: "M. Warner Losh" In-Reply-To: <200311141635.hAEGZqiT031464@green.bikeshed.org> References: <200311141635.hAEGZqiT031464@green.bikeshed.org> X-Mailer: Mew version 2.1 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/conf kern.post.mk kmod.mk 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: Fri, 14 Nov 2003 16:56:08 -0000 In message: <200311141635.hAEGZqiT031464@green.bikeshed.org> "Brian F. Feldman" writes: : "M. Warner Losh" wrote: : > In message: <200311141604.hAEG4BCg041862@repoman.freebsd.org> : > Brian Feldman writes: : > : Include opt_global.h in the modules build, when building from a normal : > : kernel build. This makes it possible for me not to get pissed off that : > : random.ko crashes the system trying to rdtsc() when the i386/cpu.h : > : support code decides it's okay to call that op when neither I386_CPU or : > : I486_CPU is defined. I guess it also makes WITNESS/INVARIANTS defines : > : get picked up by the modules. : > : > I've been polishing similar things that I'd hoped to get into this : > release. I'll try to finish them up today. You should really talk to : > people before doing these things. : : I know that you have been working on that, and I'm very happy to see that. : If this makes things difficult of course there's no reason not to remove it : other than being an interim solution for crashes people might see now. If I : didn't know you were working on the smarter module build, I would have just : gone through all the headers and tried to remove all the "harmful" uses of : constants defined by opt_global.h (which I feel is very much not a : "solution", but either way... getting rid of the problem is important, and I : don't think that the importance is particularly diminished just because the : bugs have existed for a while, IMHO). This actually is a good part of the 'first step' that I've been wanting to get done. It just suprised me, that's all, to see it committed the day we're trying to freeze for 5.2. Warner