From owner-freebsd-current Wed Oct 9 11:45:46 2002 Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 7CFC837B401 for ; Wed, 9 Oct 2002 11:45:44 -0700 (PDT) Received: from rootlabs.com (root.org [67.118.192.226]) by mx1.FreeBSD.org (Postfix) with SMTP id E027343E42 for ; Wed, 9 Oct 2002 11:45:43 -0700 (PDT) (envelope-from nate@rootlabs.com) Received: (qmail 14718 invoked by uid 1000); 9 Oct 2002 18:45:44 -0000 Date: Wed, 9 Oct 2002 11:45:44 -0700 (PDT) From: Nate Lawson To: Andrew Gallatin Cc: current@freebsd.org Subject: Re: alpha tinderbox failure In-Reply-To: <15780.18620.457682.641298@grasshopper.cs.duke.edu> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Wed, 9 Oct 2002, Andrew Gallatin wrote: > Jeff Roberson writes: > > > > On Tue, 8 Oct 2002, Dag-Erling Smorgrav wrote: > > > "Makefile", line 4194: warning: duplicate script for target "geom_bsd.o" ignored > > > cc1: warnings being treated as errors > > > /h/des/src/sys/dev/advansys/adv_pci.c: In function `adv_pci_attach': > > > /h/des/src/sys/dev/advansys/adv_pci.c:197: warning: overflow in implicit constant conversion > > > *** Error code 1 > > > > Any progress on this? > > This particular message is caused by alpha's > > #define BUS_SPACE_UNRESTRICTED (~0UL) > > Clashing with int nsegments: > > Sparc64 has the same problem. ia64 gets around it by just making > BUS_SPACE_UNRESTRICTED an int: > > #define BUS_SPACE_UNRESTRICTED (~0) > > I'd like to do the same for alpha. I think this is valid, as > BUS_SPACE_UNRESTRICTED seems to be used exlusively as an argument > to bus_dma_tag_create(... nsegments = BUS_SPACE_UNRESTRICTED...) Yes, I looked into this before and agree this is a valid approach. It's likely the number of segments never exceeds 32, let alone 2^31. > I'd also like to add a bus_space_subregion(). Please review the > appended patch. I'm running it with no ill effects, and it makes > alpha get a bit further on in LINT. (until it dies on printf format > warnings). Its going to be a bear to get a clean lint with Werror. > All those crusty old isa drivers casting pointers to integers make > me feel a bit overwhelmed. I can spend a little time cleaning that up as things progress. The patch looks fine. > It would be nice if des could remove LINT from the tinderbox builds > until LINT has a chance of compiling. Right now people are ignoring > the alpha tinderbox failure messages. Eventually, it will fail for a > reason besides useless LINT garbage, and everybody will ignore it. Ok by me. -Nate To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message