From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Daniel P. Berrange" Subject: Re: [PATCH][QEMU] Allow more IDE power management Date: Mon, 27 Aug 2007 20:19:48 +0100 Message-ID: <20070827191948.GG9043@redhat.com> References: <46D31BD4.3040802@virtualiron.com> Reply-To: "Daniel P. Berrange" Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <46D31BD4.3040802@virtualiron.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ben Guthro Cc: Dave Lively , xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org On Mon, Aug 27, 2007 at 02:45:40PM -0400, Ben Guthro wrote: > Made the QEMU IDE disk tolerate more power-mgmt commands, specifically: > WIN_SETFEATURE(EN_AAM) - enable automatic acoustic mgmt > WIN_SETFEATURE(DIS_AAM) - disable automatic acoustic mgmt > WIN_SETFEATURE(DIS_APM) - disable automatic power mgmt > WIN_STANDBY > WIN_SETIDLE1 > WIN_SLEEPNOW1 > WIN_STANDBY2 > WIN_SETIDLE2 > WIN_SLEEPNOW2 > WIN_STANDBYNOW2 > These are all essentially no-ops, like existing support for EN_RLA (enable > read lookahead) and WIN_STANDBYNOW1. > > This fixes a crash in the SLES9-SP3 64bit kernel when the powersaved was > started (with ACPI or ACM). This guest really only needs EN_AAM, DIS_APM, > and WIN_SETIDLE1 support, but the others seemed sensible to include. > > I've excluded EN_APM since I'm unsure of what that's agreeing to do. It's > probably ok to include. This and other QEMU patches should really be sent to upstream QEMU for review and inclusion first so we don't have to maintain extra patches in Xen forever. Dan. -- |=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=| |=- Perl modules: http://search.cpan.org/~danberr/ -=| |=- Projects: http://freshmeat.net/~danielpb/ -=| |=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=|