[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 05/11] suspend: add infrastructure
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH v4 05/11] suspend: add infrastructure |
Date: |
Tue, 14 Feb 2012 15:49:13 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.26) Gecko/20120130 Red Hat/3.1.18-1.el6_2 Thunderbird/3.1.18 |
Hi,
>> I see. I've expeced the the guest os putting them into a hlt loop or
>> some simliar idle state. Play save and expliticly pausing them all is
>> certainly good from a robustness perspective.
> Yes. We should not trust a guest to do the "right thing".
Updated patch attached.
>>> I think it is also important to reset all device
>>> immediately to ensure that no device will do DMA into main memory after
>>> suspend.
>>
>> Didn't investigate yet, but I suspect this could break wakeup from pci
>> devices (nic, usb-tablet via uhci) ...
>
> Yes. Can't say I fully understand how this works on real HW. I know
> that there are separate "power planes" for different system sates
> (this is defined in ACPI spec). So in S3 some devices (or even part of
> a device?) may be powered down, but others still have power. Not sure
> we should dive into emulating that in this patch series.
We'll do that later, one step at a time.
cheers,
Gerd
0001-suspend-add-infrastructure.patch
Description: Text document
- [Qemu-devel] [PATCH v4 04/11] acpi: add acpi_pm1_evt_write_en, (continued)
- [Qemu-devel] [PATCH v4 04/11] acpi: add acpi_pm1_evt_write_en, Gerd Hoffmann, 2012/02/09
- [Qemu-devel] [PATCH v4 01/11] acpi: move around structs, Gerd Hoffmann, 2012/02/09
- [Qemu-devel] [PATCH v4 09/11] suspend: make serial ports wakeup the guest., Gerd Hoffmann, 2012/02/09
- [Qemu-devel] [PATCH v4 10/11] suspend: make rtc alarm wakeup the guest., Gerd Hoffmann, 2012/02/09
- [Qemu-devel] [PATCH v4 05/11] suspend: add infrastructure, Gerd Hoffmann, 2012/02/09
[Qemu-devel] [PATCH v4 08/11] suspend: make ps/2 devices wakeup the guest, Gerd Hoffmann, 2012/02/09
[Qemu-devel] [PATCH v4 11/11] suspend: pmtimer s3 wakeup, Gerd Hoffmann, 2012/02/09
[Qemu-devel] [PATCH v4 06/11] suspend: switch acpi s3 to new infrastructure., Gerd Hoffmann, 2012/02/09
[Qemu-devel] [PATCH 12/11] suspend: add qmp events, Gerd Hoffmann, 2012/02/10