From owner-cvs-src@FreeBSD.ORG  Wed Apr 25 17:10:38 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 0D81716A404
	for <cvs-src@freebsd.org>; Wed, 25 Apr 2007 17:10:38 +0000 (UTC)
	(envelope-from nate@root.org)
Received: from root.org (root.org [67.118.192.226])
	by mx1.freebsd.org (Postfix) with ESMTP id D343D13C480
	for <cvs-src@freebsd.org>; Wed, 25 Apr 2007 17:10:37 +0000 (UTC)
	(envelope-from nate@root.org)
Received: (qmail 17150 invoked from network); 25 Apr 2007 16:48:56 -0000
Received: from ppp-71-139-34-102.dsl.snfc21.pacbell.net (HELO ?10.0.0.235?)
	(nate-mail@71.139.34.102)
	by root.org with ESMTPA; 25 Apr 2007 16:48:56 -0000
Message-ID: <462F8672.7040200@root.org>
Date: Wed, 25 Apr 2007 09:48:50 -0700
From: Nate Lawson <nate@root.org>
User-Agent: Thunderbird 2.0.0.0 (X11/20070424)
MIME-Version: 1.0
To: John Baldwin <jhb@FreeBSD.org>
References: <20070425162233.8CCFC16A59E@hub.freebsd.org>
In-Reply-To: <20070425162233.8CCFC16A59E@hub.freebsd.org>
X-Enigmail-Version: 0.95.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org
Subject: Re: cvs commit: src/sys/dev/acpica acpi.c
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: Wed, 25 Apr 2007 17:10:38 -0000

John Baldwin wrote:
> jhb         2007-04-25 16:22:18 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/dev/acpica       acpi.c 
>   Log:
>   Use a tighter check to see if a resource allocation request is for a
>   specific request and thus should first try to be allocated from the
>   sys_resource pool.  This avoids using the sys_resource pool for wildcard
>   requests that have bounded ranges coming from cbb(4) and Host-PCI pcib(4)
>   drivers.
>   
>   Tested by:      Andrea Bittau <a.bittau of cs.ucl.ac.uk fame>
>   Sleuthing by:   Andrea Bittau as well
>   
>   Revision  Changes    Path
>   1.235     +1 -1      src/sys/dev/acpica/acpi.c
> 
> 
> Index: src/sys/dev/acpica/acpi.c
> diff -u src/sys/dev/acpica/acpi.c:1.234 src/sys/dev/acpica/acpi.c:1.235
> --- src/sys/dev/acpica/acpi.c:1.234	Thu Mar 22 18:16:40 2007
> +++ src/sys/dev/acpica/acpi.c	Wed Apr 25 16:22:18 2007
> @@ -1034,7 +1034,7 @@
>       * the request from our system resource regions.  If we can't, pass the
>       * request up to the parent.
>       */
> -    if (!(start == 0UL && end == ~0UL) && rm != NULL)
> +    if (start + count - 1 == end && rm != NULL)
>  	res = rman_reserve_resource(rm, start, end, count, flags & ~RF_ACTIVE,
>  	    child);
>      if (res == NULL) {

I think I'll test this to see if it helps my via 8235 ata survive boot.

-- 
Nate