From owner-freebsd-current@FreeBSD.ORG Fri May 14 09:58:21 2004 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 61FEF16A4D0 for ; Fri, 14 May 2004 09:58:21 -0700 (PDT) Received: from root.org (root.org [67.118.192.226]) by mx1.FreeBSD.org (Postfix) with SMTP id CD1F443D55 for ; Fri, 14 May 2004 09:58:19 -0700 (PDT) (envelope-from nate@root.org) Received: (qmail 97292 invoked by uid 1000); 14 May 2004 16:58:20 -0000 Date: Fri, 14 May 2004 09:58:20 -0700 (PDT) From: Nate Lawson To: current@freebsd.org In-Reply-To: <20040514165308.1938B16A516@hub.freebsd.org> Message-ID: <20040514095537.L97216@root.org> References: <20040514165308.1938B16A516@hub.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: acpi@freebsd.org Subject: Re: cvs commit: src/usr.sbin/acpi/acpidump acpi_user.c acpidump.h X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 14 May 2004 16:58:21 -0000 On Fri, 14 May 2004, Nate Lawson wrote: > njl 2004/05/14 09:52:39 PDT > > FreeBSD src repository > > Modified files: > usr.sbin/acpi/acpidump acpi_user.c acpidump.h > Log: > Instead of scanning the entire lower 1 MB of RAM, only scan locations > where the RSD PTR can actually occur. According to section 5.2.2 > of the ACPI spec, we only consider two regions for the base address: > > 1. EBDA (0x0 - 0x3FF) > 2. High memory (0xE0000 - 0xFFFFF) > > I don't know whether this fixes any actual problems but is more correct. > > Revision Changes Path > 1.8 +42 -29 src/usr.sbin/acpi/acpidump/acpi_user.c > 1.14 +11 -0 src/usr.sbin/acpi/acpidump/acpidump.h Please recompile and test acpidump by running acpidump -t. Also, it might be nice to test it on systems with acpi disabled[*] since I think this should work now. -Nate * Disabled in FreeBSD, not in the BIOS. It is not possible to run acpidump on a machine with ACPI disabled in the BIOS since the tables will not be present. This commit brought to you by slow bridge traffic this morning. :)