From owner-freebsd-sparc64@FreeBSD.ORG  Sun Dec 16 21:16:26 2012
Return-Path: <owner-freebsd-sparc64@FreeBSD.ORG>
Delivered-To: sparc64@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
 by hub.freebsd.org (Postfix) with ESMTP id D0B98C95;
 Sun, 16 Dec 2012 21:16:26 +0000 (UTC)
 (envelope-from marius@alchemy.franken.de)
Received: from alchemy.franken.de (alchemy.franken.de [194.94.249.214])
 by mx1.freebsd.org (Postfix) with ESMTP id 464568FC19;
 Sun, 16 Dec 2012 21:16:26 +0000 (UTC)
Received: from alchemy.franken.de (localhost [127.0.0.1])
 by alchemy.franken.de (8.14.5/8.14.5/ALCHEMY.FRANKEN.DE) with ESMTP id
 qBGLGNN0049914; Sun, 16 Dec 2012 22:16:24 +0100 (CET)
 (envelope-from marius@alchemy.franken.de)
Received: (from marius@localhost)
 by alchemy.franken.de (8.14.5/8.14.5/Submit) id qBGLGNXu049913;
 Sun, 16 Dec 2012 22:16:23 +0100 (CET) (envelope-from marius)
Date: Sun, 16 Dec 2012 22:16:23 +0100
From: Marius Strobl <marius@alchemy.franken.de>
To: Jeff Roberson <jroberson@jroberson.net>
Subject: Re: Call for testing and review, busdma changes
Message-ID: <20121216211623.GA49699@alchemy.franken.de>
References: <alpine.BSF.2.00.1212080841370.4081@desktop>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <alpine.BSF.2.00.1212080841370.4081@desktop>
User-Agent: Mutt/1.5.21 (2010-09-15)
X-Mailman-Approved-At: Sun, 16 Dec 2012 23:04:54 +0000
Cc: powerpc@freebsd.org, marcel@freebsd.org, mips@freebsd.org, mav@freebsd.org,
 scottl@freebsd.org, attilio@freebsd.org, kib@freebsd.org, sparc64@freebsd.org,
 arm@freebsd.org
X-BeenThere: freebsd-sparc64@freebsd.org
X-Mailman-Version: 2.1.14
Precedence: list
List-Id: Porting FreeBSD to the Sparc <freebsd-sparc64.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-sparc64>, 
 <mailto:freebsd-sparc64-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-sparc64>
List-Post: <mailto:freebsd-sparc64@freebsd.org>
List-Help: <mailto:freebsd-sparc64-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-sparc64>,
 <mailto:freebsd-sparc64-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 16 Dec 2012 21:16:26 -0000

On Sat, Dec 08, 2012 at 08:51:12AM -1000, Jeff Roberson wrote:
> Hello,
> 
> http://people.freebsd.org/~jeff/physbio.diff
> 
> I have a relative large patch that reforms the busdma API so that new 
> types may be added without modifying every architecture's 
> busdma_machdep.c.  It does this by unifying the bus_dmamap_load_buffer() 
> routines so that they may be called from MI code.  The MD busdma is then 
> given a chance to do any final processing in the complete() callback. 
> This patch also contains cam changes to unify the bus_dmamap_load* 
> handling in cam drivers.
> 
> The arm and mips implementations saw the largest changes since they have 
> to track virtual addresses for sync().  Previously this was done in a type 
> specific way.  Now it is done in a generic way by recording the list of 
> virtuals in the map.
> 
> I have verified that this patch passes make universe which includes 
> several kernel builds from each architecture.  I suspect that if I broke 
> anything your machine simply won't boot or will throw I/O errors.  There 
> is little subtlety, it is mostly refactoring.
> 
> The next step is to allow for dma loading of physical addresses.  This 
> will permit unmapped I/O.  Which is a significant performance optimization 
> targeted for 10.0.
> 
> Many thanks for your assistance.  Any review feedback is also appreciated.
> 

Survives a buildworld on sparc64; tested with mpt(4) and sym(4).

Marius