cvs commit: src/sys/arm/xscale/ixp425 if_npe.c ixp425.c ixp425_pci.c ixp425var.h

Olivier Houchard cognet at ci0.org
Thu Jan 18 01:49:33 UTC 2007


On Wed, Jan 17, 2007 at 03:23:21PM -0700, Warner Losh wrote:
> From: John-Mark Gurney <gurney_j at 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


More information about the cvs-src mailing list