From owner-freebsd-current@FreeBSD.ORG  Tue Nov  9 02:39:58 2010
Return-Path: <owner-freebsd-current@FreeBSD.ORG>
Delivered-To: current@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 9FFF61065673
	for <current@FreeBSD.org>; Tue,  9 Nov 2010 02:39:58 +0000 (UTC)
	(envelope-from kevlo@FreeBSD.org)
Received: from ns.kevlo.org (kevlo.org [220.128.136.52])
	by mx1.freebsd.org (Postfix) with ESMTP id 29AD68FC1B
	for <current@FreeBSD.org>; Tue,  9 Nov 2010 02:39:57 +0000 (UTC)
Received: from [127.0.0.1] (kevlo@kevlo.org [220.128.136.52])
	by ns.kevlo.org (8.14.3/8.14.3) with ESMTP id oA91xTP3009279;
	Tue, 9 Nov 2010 09:59:29 +0800 (CST)
From: Kevin Lo <kevlo@FreeBSD.org>
To: John Baldwin <jhb@FreeBSD.org>
In-Reply-To: <201011081714.53637.jhb@freebsd.org>
References: <201011081714.53637.jhb@freebsd.org>
Content-Type: text/plain; charset="UTF-8"
Date: Tue, 09 Nov 2010 10:00:15 +0800
Message-ID: <1289268015.1982.3.camel@nsl>
Mime-Version: 1.0
X-Mailer: Evolution 2.30.3 
Content-Transfer-Encoding: 7bit
Cc: current@FreeBSD.org
Subject: Re: Only display ACPI bootmenu key if ACPI is present
X-BeenThere: freebsd-current@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Discussions about the use of FreeBSD-current
	<freebsd-current.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>, 
	<mailto:freebsd-current-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-current>
List-Post: <mailto:freebsd-current@freebsd.org>
List-Help: <mailto:freebsd-current-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>,
	<mailto:freebsd-current-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 09 Nov 2010 02:39:58 -0000

John Baldwin wrote:
> This patch changes the Forth code for the Beastie menu to only display the
> menu option to enable or disable ACPI if the loader detects ACPI.  This avoids
> displaying a menu item prompting to enable ACPI if the BIOS doesn't actually
> include ACPI.  Any objections?

I have no objection. This patch makes sense to me.

	Kevin