From owner-cvs-all@FreeBSD.ORG  Sat Jul 29 22:01:58 2006
Return-Path: <owner-cvs-all@FreeBSD.ORG>
X-Original-To: cvs-all@FreeBSD.org
Delivered-To: cvs-all@FreeBSD.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 1894F16A508;
	Sat, 29 Jul 2006 22:01:58 +0000 (UTC)
	(envelope-from jb@what-creek.com)
Received: from what-creek.com (what-creek.com [66.111.37.70])
	by mx1.FreeBSD.org (Postfix) with ESMTP id C9E1043D49;
	Sat, 29 Jul 2006 22:01:57 +0000 (GMT)
	(envelope-from jb@what-creek.com)
Received: by what-creek.com (Postfix, from userid 102)
	id 0D58078C1F; Sat, 29 Jul 2006 22:01:55 +0000 (GMT)
Date: Sat, 29 Jul 2006 22:01:54 +0000
From: John Birrell <jb@what-creek.com>
To: Pyun YongHyeon <yongari@FreeBSD.org>
Message-ID: <20060729220154.GA23870@what-creek.com>
References: <200607270043.k6R0hY8g088353@repoman.freebsd.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <200607270043.k6R0hY8g088353@repoman.freebsd.org>
User-Agent: Mutt/1.4.2.1i
Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/dev/em if_em.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
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: Sat, 29 Jul 2006 22:01:58 -0000

On Thu, Jul 27, 2006 at 12:43:34AM +0000, Pyun YongHyeon wrote:
> yongari     2006-07-27 00:43:34 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/dev/em           if_em.c 
>   Log:
>   Prepending an mbuf after loading a DMA map results in unexpected
>   result. So, modify mbuf chains before loading a DMA map.
>   
>   Revision  Changes    Path
>   1.122     +28 -31    src/sys/dev/em/if_em.c

The change immediately after the one that I tested on sun4v brought the
bus_dmamap_unload diagnostics back (worse than before). So much so,
I had to back the change out of my P4 tree.

This is the change that causes the grief:

<http://perforce.freebsd.org/fileDiffView.cgi?FSPC=//depot/projects/dtrace/src/sys/dev/em/if%5fem.c&REV=6&ACT=integrate>

--
John Birrell