From owner-cvs-all@FreeBSD.ORG Wed Jul 23 09:15:16 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 046AF37B401; Wed, 23 Jul 2003 09:15:16 -0700 (PDT) Received: from phk.freebsd.dk (phk.freebsd.dk [212.242.86.175]) by mx1.FreeBSD.org (Postfix) with ESMTP id E9FAF43FBD; Wed, 23 Jul 2003 09:15:14 -0700 (PDT) (envelope-from phk@phk.freebsd.dk) Received: from critter.freebsd.dk (critter.freebsd.dk [212.242.86.163]) by phk.freebsd.dk (8.12.8/8.12.8) with ESMTP id h6NGFCV3045893; Wed, 23 Jul 2003 16:15:12 GMT (envelope-from phk@phk.freebsd.dk) Received: from critter.freebsd.dk (localhost [127.0.0.1]) by critter.freebsd.dk (8.12.9/8.12.9) with ESMTP id h6NGFB5H023601; Wed, 23 Jul 2003 18:15:11 +0200 (CEST) (envelope-from phk@phk.freebsd.dk) To: Paul Richards From: "Poul-Henning Kamp" In-Reply-To: Your message of "23 Jul 2003 16:34:20 BST." <1058974459.31173.17.camel@localhost> Date: Wed, 23 Jul 2003 18:15:11 +0200 Message-ID: <23600.1058976911@critter.freebsd.dk> cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern init_main.c kern_malloc.c md5c.c subr_autoconf.c subr_mbuf.c subr_prf.c tty_subr.c vfs_cluster.c vfs_subr.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: Wed, 23 Jul 2003 16:15:16 -0000 In message <1058974459.31173.17.camel@localhost>, Paul Richards writes: >On Wed, 2003-07-23 at 06:40, Poul-Henning Kamp wrote: > >> >> The ones I took out yesterday are the onces which GCC had ignored and >> which therefore ipso facto were "unproven" _and_ added significant >> amounts of object code if respected. > >That's just untrue. The inline you removed from lnc had *ZERO* impact on >code size. It would _really_ help if you would bother to listen to what people tell you about how the new GCC warnings work. GCC rejects inlining things above a certain limit, and while the limit is arguably measured in "elbonian mud gallons", it is nontheless something we have to address: If we ever want to be able to use those warnings to tell us when something is amiss, we need to set the limit reasonably. The two inlines in the lnc driver were among the largest in the system. If we had set the GCC limit to allow those inlines, we would never get a warning when inline mistakes were made. And when you consider that that the inlines in lnc were purely ornamental, and that GCC previously had silently refused the inline request, you will realize that by removing them, I merely preserved the status-quo. -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD committer | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence.