From owner-cvs-all Sun Sep 22 23:19:19 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 CC91537B401; Sun, 22 Sep 2002 23:19:17 -0700 (PDT) Received: from elvis.mu.org (elvis.mu.org [192.203.228.196]) by mx1.FreeBSD.org (Postfix) with ESMTP id 89DA043E6A; Sun, 22 Sep 2002 23:19:17 -0700 (PDT) (envelope-from bright@elvis.mu.org) Received: by elvis.mu.org (Postfix, from userid 1192) id 5D3CDAE211; Sun, 22 Sep 2002 23:19:17 -0700 (PDT) Date: Sun, 22 Sep 2002 23:19:17 -0700 From: Alfred Perlstein To: Peter Wemm Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/alpha/pci tsunamireg.h src/sys/boot/ia64/skiload main.c src/sys/i386/include npx.h src/sys/ia64/include reg.h signal.h src/sys/sys cdefs.h Message-ID: <20020923061917.GG6262@elvis.mu.org> References: <200209230555.g8N5tBCP031090@freefall.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200209230555.g8N5tBCP031090@freefall.freebsd.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 * Peter Wemm [020922 22:55] wrote: > peter 2002/09/22 22:55:11 PDT > > Modified files: > sys/alpha/pci tsunamireg.h > sys/boot/ia64/skiload main.c > sys/i386/include npx.h > sys/ia64/include reg.h signal.h > sys/sys cdefs.h > Log: > At great personal risk, add a __packed and __aligned(x) define that > expand to __attribute__((packed)) and __attribute__((aligned(x))) > respectively. Replace the handful of gcc-ism's that use > __attribute__((aligned(16))) etc around the kernel with __aligned(16). > > There are over 400 __attribute__((packed)) to deal with, that can come > later. I just want to use __packed in new code rather than add more > gcc-ism's. I really like what you've done, however don't you think that the places that do not have __packed/__aligned shouldn't have it defined to nothing... therefore you can add: #ifndef __packed #error you must define __packed or bad things will happen #endif #ifndef __aligned #error you must define __aligned or bad things will happen #endif Otherwise people may get compiled but very broken code which most likely will be in critical things like device drivers. -- -Alfred Perlstein [alfred@freebsd.org] 'Instead of asking why a piece of software is using "1970s technology," start asking why software is ignoring 30 years of accumulated wisdom.' To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message