From owner-cvs-src@FreeBSD.ORG  Thu Jan 18 01:49:32 2007
Return-Path: <owner-cvs-src@FreeBSD.ORG>
X-Original-To: cvs-src@FreeBSD.org
Delivered-To: cvs-src@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id D978A16A407;
	Thu, 18 Jan 2007 01:49:32 +0000 (UTC)
	(envelope-from doginou@dong.ci0.org)
Received: from dong.ci0.org (cognet.ci0.org [80.65.224.102])
	by mx1.freebsd.org (Postfix) with ESMTP id 02D4913C442;
	Thu, 18 Jan 2007 01:49:31 +0000 (UTC)
	(envelope-from doginou@dong.ci0.org)
Received: from dong.ci0.org (localhost.ci0.org [127.0.0.1])
	by dong.ci0.org (8.13.7/8.13.4) with ESMTP id l0I1kSD6012467;
	Thu, 18 Jan 2007 02:46:28 +0100 (CET)
	(envelope-from doginou@dong.ci0.org)
Received: (from doginou@localhost)
	by dong.ci0.org (8.13.7/8.13.4/Submit) id l0I1kSiV012466;
	Thu, 18 Jan 2007 02:46:28 +0100 (CET) (envelope-from doginou)
Date: Thu, 18 Jan 2007 02:46:27 +0100
From: Olivier Houchard <cognet@ci0.org>
To: Warner Losh <imp@bsdimp.com>
Message-ID: <20070118014627.GB2941@ci0.org>
References: <200701170058.l0H0wQM3088464@repoman.freebsd.org>
	<20070117201642.GS92003@funkthat.com>
	<20070117.152321.85348370.imp@bsdimp.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20070117.152321.85348370.imp@bsdimp.com>
User-Agent: Mutt/1.4.1i
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
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 18 Jan 2007 01:49:33 -0000

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