qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 1/1] ich9: add disable_s3, disable_s4, s4_val pr


From: Amit Shah
Subject: Re: [Qemu-devel] [PATCH 1/1] ich9: add disable_s3, disable_s4, s4_val properties
Date: Mon, 5 Jan 2015 17:53:15 +0530

On (Wed) 17 Dec 2014 [15:08:36], Marcel Apfelbaum wrote:
> On 12/16/2014 01:23 PM, Amit Shah wrote:
> >PIIX4 has disable_s3 and disable_s4 properties to enable or disable PM
> >functions.  Add such properties to the ICH9 chipset as well for the Q35
> >machine type.
> >
> >S3 / S4 are not guaranteed to always work (needs work in the guest as
> >well as QEMU for things to work properly), and disabling advertising of
> >these features ensures guests don't go into zombie state if something
> >isn't working right.
> >
> >The defaults are kept the same as in PIIX4: both S3 and S4 are enabled
> >by default.
> >
> >These can be disabled via the cmdline:
> >
> >   ... -global ICH9-LPC,disable_s3=1 -global ICH9-LPC,disable_s4=1
> Maybe we can add them to Q35 Machines instead?
> - machine q35,disable_s3=1,disable_s4=1

The PM properties are chipset properties, aren't they?  To make the
change you requested, q35 will have to query the chipsets it supports,
and set the properties for the chipset in use - not sure how that
happens.

I modeled this based on how i440fx works, so there's also the case for
keeping things similar to an existing implementation...

                Amit



reply via email to

[Prev in Thread] Current Thread [Next in Thread]