From: Peter Maydell <peter.maydell@linaro.org>
To: Markus Armbruster <armbru@redhat.com>
Cc: jordan.l.justen@intel.com, Anthony Liguori <aliguori@us.ibm.com>,
patches@linaro.org, qemu-devel@nongnu.org,
Juan Quintela <quintela@redhat.com>
Subject: Re: [Qemu-devel] Use of flash for x86 BIOS (was: [PATCH 0/2] Implement migration support for pflash_cfi01)
Date: Fri, 22 Mar 2013 16:42:03 +0000 [thread overview]
Message-ID: <CAFEAcA-iMobPLuWdPL3XOr-1gcJNGOVTfYD_6gUKdg+aG0bbVg@mail.gmail.com> (raw)
In-Reply-To: <87y5dh2o3j.fsf@blackfin.pond.sub.org>
On 21 March 2013 07:45, Markus Armbruster <armbru@redhat.com> wrote:
> [Note cc: Jordan, who added flash to x86 in commit bd183c79]
>
> Peter Maydell <peter.maydell@linaro.org> writes:
>
>> These patches implement migration support for pflash_cfi01.
>> The first patch just drops some useless state so we don't
>> have to think about it for migration.
>>
>> NB that pflash_cfi01 is used in the x86 pc model. I think this
>> means that migration while the BIOS is accessing the flash
>> wouldn't have worked properly. Since migration from a device
>> with no vmstate to one with vmstate works OK this shouldn't
>> break cross-version migration. However x86 maintainers may
>> wish to review and confirm this for themselves...
>
> x86 maintainers may wish to *switch it off* until it's done fully and
> properly, by setting "pc-sysfw" property "rom_only" to 1.
So does that mean that these patches can't be applied until
the rom_only property is set, or is that a fix that can be made
independently?
thanks
-- PMM
next prev parent reply other threads:[~2013-03-22 16:42 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-19 18:24 [Qemu-devel] [PATCH 0/2] Implement migration support for pflash_cfi01 Peter Maydell
2013-03-19 18:24 ` [Qemu-devel] [PATCH 1/2] pflash_cfi01: Drop unused 'bypass' field Peter Maydell
2013-03-19 18:24 ` [Qemu-devel] [PATCH 2/2] pflash_cfi01: Implement migration support Peter Maydell
2013-03-26 13:32 ` Peter Maydell
2013-03-21 7:45 ` [Qemu-devel] Use of flash for x86 BIOS (was: [PATCH 0/2] Implement migration support for pflash_cfi01) Markus Armbruster
2013-03-22 16:42 ` Peter Maydell [this message]
2013-03-22 18:51 ` [Qemu-devel] Use of flash for x86 BIOS Markus Armbruster
2013-03-22 19:09 ` [Qemu-devel] Use of flash for x86 BIOS (was: [PATCH 0/2] Implement migration support for pflash_cfi01) Jordan Justen
2013-03-22 19:48 ` [Qemu-devel] Use of flash for x86 BIOS Markus Armbruster
2013-04-03 13:48 ` Laszlo Ersek
2013-04-03 18:38 ` Jordan Justen
2013-04-12 15:33 ` Markus Armbruster
2013-04-08 6:06 ` Xiao Guangrong
2013-04-08 8:18 ` Jordan Justen
2013-04-08 8:43 ` Gleb Natapov
2013-04-08 9:19 ` Xiao Guangrong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAFEAcA-iMobPLuWdPL3XOr-1gcJNGOVTfYD_6gUKdg+aG0bbVg@mail.gmail.com \
--to=peter.maydell@linaro.org \
--cc=aliguori@us.ibm.com \
--cc=armbru@redhat.com \
--cc=jordan.l.justen@intel.com \
--cc=patches@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).