From owner-cvs-all@FreeBSD.ORG  Thu Oct  7 22:43:08 2004
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from green.homeunix.org (freefall.freebsd.org [216.136.204.21])
	by hub.freebsd.org (Postfix) with ESMTP
	id D98A416A4CE; Thu,  7 Oct 2004 22:43:07 +0000 (GMT)
Received: from green.homeunix.org (green@localhost [127.0.0.1])
	by green.homeunix.org (8.13.1/8.13.1) with ESMTP id i97Mh7Cb098562;
	Thu, 7 Oct 2004 18:43:07 -0400 (EDT)
	(envelope-from green@green.homeunix.org)
Received: (from green@localhost)
	by green.homeunix.org (8.13.1/8.13.1/Submit) id i97Mh0t5098561;
	Thu, 7 Oct 2004 18:43:00 -0400 (EDT)
	(envelope-from green)
Date: Thu, 7 Oct 2004 18:43:00 -0400
From: Brian Fundakowski Feldman <green@FreeBSD.org>
To: Scott Long <scottl@FreeBSD.org>
Message-ID: <20041007224300.GF73261@green.homeunix.org>
References: <200410071621.i97GL3lu029620@repoman.freebsd.org>
	<20041007175206.GA82275@ns1.xcllnt.net> <4165C120.7040005@root.org>
	<4165C1EF.7020407@FreeBSD.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4165C1EF.7020407@FreeBSD.org>
User-Agent: Mutt/1.5.6i
cc: src-committers@FreeBSD.org
cc: cvs-src@FreeBSD.org
cc: cvs-all@FreeBSD.org
cc: Marcel Moolenaar <marcel@xcllnt.net>
cc: Nate Lawson <nate@root.org>
cc: Warner Losh <imp@FreeBSD.org>
Subject: Re: cvs commit: src/sys/sys pbioio.h src/sys/i386/isa pbio.c
	src/sys/conf files.i386 src/sys/i386/conf NOTES
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 07 Oct 2004 22:43:08 -0000

On Thu, Oct 07, 2004 at 04:23:43PM -0600, Scott Long wrote:
> Nate Lawson wrote:
> >Marcel Moolenaar wrote:
> >
> >>On Thu, Oct 07, 2004 at 04:21:03PM +0000, Warner Losh wrote:
> >>
> >>>imp         2004-10-07 16:21:03 UTC
> >>>
> >>> FreeBSD src repository
> >>>
> >>> Modified files:
> >>>   sys/conf             files.i386    sys/i386/conf        NOTES 
> >>> Added files:
> >>>   sys/sys              pbioio.h    sys/i386/isa         pbio.c  Log:
> >>> Port pbio to HEAD.
> >>
> >>
> >>
> >>I appreciate your speed, but don't you think that pbioio.h is pretty
> >>MD given that the driver only exists on i386. Wouldn't <machine/pbioio.h>
> >>be a better place?
> >
> >
> >Also, I think our policy for both RELENG_4 and -current is new inb/outb 
> >in new drivers.  The bus_space stuff is pretty easy to use so this isn't 
> >too bad a requirement.
> >
> 
> I agree that new code should _not_ be using unportable primitives unless
> there is very good reason.  FWIW, I plan to make vtophys(),
> rman_get_virtual(), and other evil and i386-specific primitives very
> hard to use in 6-CURRENT, and I will strongly oppose importing new
> code that tries to abuse them.  I was just hoping that 5.3 would pass
> before people started testing the boundaries.

Maybe first we should make the busdma API usable?  The BUS_DMASYNC_*
macros are named positively terribly, and the documentation really
isn't any better.

-- 
Brian Fundakowski Feldman                           \'[ FreeBSD ]''''''''''\
  <> green@FreeBSD.org                               \  The Power to Serve! \
 Opinions expressed are my own.                       \,,,,,,,,,,,,,,,,,,,,,,\