From owner-cvs-all Mon Dec 30 7:47:47 2002 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 1DF2237B401; Mon, 30 Dec 2002 07:47:46 -0800 (PST) Received: from elvis.mu.org (elvis.mu.org [192.203.228.196]) by mx1.FreeBSD.org (Postfix) with ESMTP id CFA4A43EB2; Mon, 30 Dec 2002 07:47:45 -0800 (PST) (envelope-from mux@freebsd.org) Received: by elvis.mu.org (Postfix, from userid 1920) id AF617AE359; Mon, 30 Dec 2002 07:47:45 -0800 (PST) Date: Mon, 30 Dec 2002 07:47:45 -0800 From: Maxime Henrion To: Bruce Evans Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/pci if_xl.c Message-ID: <20021230154745.GE27086@elvis.mu.org> References: <200212301246.gBUCk8VW075195@repoman.freebsd.org> <20021230235804.I46568-100000@gamplex.bde.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20021230235804.I46568-100000@gamplex.bde.org> User-Agent: Mutt/1.4i 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 Bruce Evans wrote: > On Mon, 30 Dec 2002, Maxime Henrion wrote: > > > mux 2002/12/30 04:46:08 PST > > > > Modified files: > > sys/pci if_xl.c > > Log: > > Leave only one of the two $FreeBSD$ tag, and use __FBSDID(). > > > > Revision Changes Path > > 1.114 +3 -6 src/sys/pci/if_xl.c > > This removes the wrong tag IMO. Id strings are not normally used in the > kernel. The places that do use them mostly have duplicate $FreeBSD$'s. > The main places are netatm and wpaul's network drivers. if_xl.c was one > of the latter. In netatm files, one of the tags should be the vendor > tag and the other should be the FreeBSD tag. In the network drivers, > there should be only one tag. I agree that there should only be one tag, and that's why I did this change in the first place. I don't understand why it would be bad to have Id strings in the kernel though. I find it a very useful feature when you want to know what file revisions the kernel was compiled with. Moreover, style(9) doesn't seem to say that we shouldn't use __FBSDID in the kernel. If there is a consensus that __FBSDID shouldn't be used in the kernel, I would like to know why and it also should be documented in style(9). Cheers, Maxime To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message