[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:10:13 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
Il 05/03/2013 17:59, David Woodhouse ha scritto:
> On Tue, 2013-03-05 at 17:03 +0100, Paolo Bonzini wrote:
>> > Resuming from suspend-to-RAM should not reset all devices. Only the
>> > CPU should get a reset signal.
> Hm... on reflection, I don't actually know if this is true.
>
> Perhaps we *should* reset all devices. After all, in a real machine
> they'll all have been turned off and the RAM will have been in
> self-refresh. Surely they have to be reset?
>
> So maybe we should *let* the i440FX PAM registers get reset to point to
> ROM. And fix the firmware to *cope* with that, check to see if the
> shadow RAM already holds an image of a started-up firmware with the
> correct checksum, and jump back to it.
>
> 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?
Paolo
- [Qemu-devel] [PATCH 3/3] hw: correctly implement soft reset, (continued)
- [Qemu-devel] [PATCH 3/3] hw: correctly implement soft reset, Paolo Bonzini, 2013/03/05
- 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