From owner-cvs-all Wed Dec 23 20:50:21 1998 Return-Path: Received: (from majordom@localhost) by hub.freebsd.org (8.8.8/8.8.8) id UAA07133 for cvs-all-outgoing; Wed, 23 Dec 1998 20:50:21 -0800 (PST) (envelope-from owner-cvs-all@FreeBSD.ORG) Received: from feral-gw.feral.com (feral.com [192.67.166.1]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id UAA07120; Wed, 23 Dec 1998 20:50:17 -0800 (PST) (envelope-from mjacob@feral.com) Received: from localhost (mjacob@localhost) by feral-gw.feral.com (8.8.7/8.8.7) with ESMTP id UAA12880; Wed, 23 Dec 1998 20:49:57 -0800 Date: Wed, 23 Dec 1998 20:49:57 -0800 (PST) From: Matthew Jacob X-Sender: mjacob@feral-gw Reply-To: mjacob@feral.com To: Bruce Evans cc: cvs-all@FreeBSD.ORG, cvs-committers@FreeBSD.ORG Subject: Re: cvs commit: src/sys/cam cam_xpt.c In-Reply-To: <199812240443.PAA28211@godzilla.zeta.org.au> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk On Thu, 24 Dec 1998, Bruce Evans wrote: > > Modified files: > > sys/cam cam_xpt.c > > Log: > > Unstaticize swi_camnet && swi_cambio so Alpha kernels can build again. > > This is a regression. It is a bug for swi_camnet and swi_cambio to > be hard-configured in the alpha ihandlers[]. It is another bug for > cam to access ihandlers[] directly instead of using register_swi(). > register_swi() apparently works on alphas since it is used in sio.c. > It's possibly a bug for people to gratuitously break other architectures so they don't build and therefore don't run, no matter how pure the motivation. That trumps the purity argument, IMHO. I spent about 10 minutes trying to fix it a better way but couldn't. I'll spend some more time to try and clean up what is clearly some incompletely thought out code- but if I don't make it tonite prior to going off for a couple days with the family- feel free to break the alpha builds again. -matt To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message