Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 18 Jan 2007 02:46:27 +0100
From:      Olivier Houchard <cognet@ci0.org>
To:        Warner Losh <imp@bsdimp.com>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/arm/xscale/ixp425 if_npe.c ixp425.c ixp425_pci.c ixp425var.h
Message-ID:  <20070118014627.GB2941@ci0.org>
In-Reply-To: <20070117.152321.85348370.imp@bsdimp.com>
References:  <200701170058.l0H0wQM3088464@repoman.freebsd.org> <20070117201642.GS92003@funkthat.com> <20070117.152321.85348370.imp@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Jan 17, 2007 at 03:23:21PM -0700, Warner Losh wrote:
> From: John-Mark Gurney <gurney_j@resnet.uoregon.edu>
> Subject: Re: cvs commit: src/sys/arm/xscale/ixp425 if_npe.c ixp425.c ixp425_pci.c ixp425var.h
> Date: Wed, 17 Jan 2007 12:16:42 -0800
> 
> > Olivier Houchard wrote this message on Wed, Jan 17, 2007 at 00:58 +0000:
> > >   IXP425 boards have 128MB or more, and most of the PCI drivers do not bother
> > >   providing the parent dma tag.
> > 
> > You should start fixing this by panicing on devices that don't properly
> > call bus_get_dma_tag...  That's what I did for sun4v so that we can get
> > the devices converted...
> 
> I have patches that will implement this :-)
> 

I do agree, I remember discussing this with Warner seom times ago. However,
due to the weak exposure of the arm code, it won't get a lot of devices
converted. Maybe a good start would be to add a big warning in i386/amd64
code in -CURRENT ?

Olivier



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070118014627.GB2941>