This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: [Xen-devel] [PATCH][QEMU] Allow more IDE power management

To: Ben Guthro <bguthro@xxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH][QEMU] Allow more IDE power management
From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
Date: Mon, 27 Aug 2007 20:19:48 +0100
Cc: Dave Lively <dlively@xxxxxxxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 27 Aug 2007 12:21:02 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <46D31BD4.3040802@xxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <46D31BD4.3040802@xxxxxxxxxxxxxxx>
Reply-to: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.4.1i
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_SETIDLE1
>       WIN_STANDBY2
>       WIN_SETIDLE2
> 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.

|=- 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  -=| 

Xen-devel mailing list

<Prev in Thread] Current Thread [Next in Thread>