[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/3] wakeup: only reset the CPU
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 4/3] wakeup: only reset the CPU |
Date: |
Tue, 05 Mar 2013 18:42:56 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
Il 05/03/2013 18:26, Peter Stuge ha scritto:
>>> > > That is: perhaps it's a *SeaBIOS* bug that suspend/resume doesn't
>>> > > work if the PAM configuration is reset?
>> >
>> > Yeah, it sounded a bit weird when I wrote that commit message.
>> > This could be the case. How does it work on Coreboot?
> Yes all hardware except RAM in self-refresh and registers explicitly
> hooked to battery sleep power rail has reset state on resume from S3.
> Those registers seem to grow over time, the i440 probably doesn't
> have very many. The memory controller configuration would be stored
> there.
Ok, so it's not correct either with this patch or without. :)
But without this patch it is more correct; things such as USB queue
heads must be reset, or you'll corrupt random memory when the VM is
restarted.
Paolo
- Re: [Qemu-devel] [PATCH 3/3] hw: correctly implement soft reset, (continued)
- Re: [Qemu-devel] [PATCH 0/3] Implement x86 soft reset, David Woodhouse, 2013/03/05
- [Qemu-devel] [PATCH 4/3] wakeup: only reset the CPU, Paolo Bonzini, 2013/03/05
- Re: [Qemu-devel] [PATCH 4/3] wakeup: only reset the CPU, Laszlo Ersek, 2013/03/05
- Re: [Qemu-devel] [PATCH 4/3] wakeup: only reset the CPU, Paolo Bonzini, 2013/03/05
- Re: [Qemu-devel] [PATCH 4/3] wakeup: only reset the CPU, Laszlo Ersek, 2013/03/05
- Re: [Qemu-devel] [PATCH 4/3] wakeup: only reset the CPU, Kevin O'Connor, 2013/03/05
- Re: [Qemu-devel] [PATCH 4/3] wakeup: only reset the CPU, David Woodhouse, 2013/03/06