From: Anthony Liguori <anthony@codemonkey.ws>
To: Jordan Justen <jordan.l.justen@intel.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v11 0/9] PC system flash support
Date: Wed, 22 Feb 2012 12:54:02 -0600 [thread overview]
Message-ID: <4F4539CA.2070600@codemonkey.ws> (raw)
In-Reply-To: <1329895136-20494-1-git-send-email-jordan.l.justen@intel.com>
On 02/22/2012 01:18 AM, Jordan Justen wrote:
> Enable flash emulation in a PC system using pflash_cfi01.
Applied all.
I know it's taken a while to get these merged, but I'm very pleased with the
final result. Thanks for sticking through it!
Regards,
Anthony Liguori
>
> v11:
> * Convert pc-sysfw to qdev
> - Add rom_only property
> * Remove KVM flash support to remove the need for using
> bdrv_read during machine initialization. KVM should
> now continue to use the same 'rom' initialization
> sequence that it uses today.
>
> v10:
> * Rebase to HEAD
> * "decouple vmstate from memory API" as in c5705a7
> * Break changes into smaller pieces
>
> v9:
> * Add pc-1.1
> * pc-1.0 uses previous rom firmware init code path
>
> v8:
> * Cleanup two chunks of debug code (printf messages)
> * Fix comment in pc.h (pcflash.c => pc_sysfw.c)
>
> v7:
> * Do not add system firmware to qemu roms
> * If kvm is enabled, copy pflash drive contents into a
> read-only ram region, since kvm cannot currently execute
> code from a pflash device.
> * Rename pcflash.c to pc_sysfw.c
>
> v6:
> * Rebase for memory API
> * pflash_cfi01: Set error in status register when a write to
> erase is attempted in read-only mode.
> * Add system firmware to qemu roms
>
> v5:
> * Enable pflash read-only mode
> * Enable -drive with if=pflash to define system firmware image
>
> v4:
> * Rebase
>
> v3:
> * Fix code style issues
> * Add additional comments
>
> v2:
> * Convert debug printf to DPRINTF
>
> Jordan Justen (9):
> blockdev: allow read-only pflash devices
> pflash_cfi01/02: support read-only pflash devices
> vl: make find_default_machine externally visible
> hw/pc: move rom init to pc_sysfw.c
> hw/pc_sysfw: enable pc-sysfw as a qdev
> hw/pc_sysfw: support system flash memory with pflash
> hw/pc_piix: remove is_default for pc-0.15
> hw/pc_piix: add pc-1.1
> pc_piix/pc_sysfw: enable flash by default
>
> Makefile.target | 1 +
> blockdev.c | 3 +-
> default-configs/i386-softmmu.mak | 1 +
> default-configs/x86_64-softmmu.mak | 1 +
> hw/boards.h | 1 +
> hw/pc.c | 56 +-------
> hw/pc.h | 3 +
> hw/pc_piix.c | 62 +++++++++-
> hw/pc_sysfw.c | 254 ++++++++++++++++++++++++++++++++++++
> hw/pflash_cfi01.c | 44 +++++--
> hw/pflash_cfi02.c | 83 +++++++------
> vl.c | 2 +-
> 12 files changed, 403 insertions(+), 108 deletions(-)
> create mode 100644 hw/pc_sysfw.c
>
>
>
next prev parent reply other threads:[~2012-02-22 18:54 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-22 7:18 [Qemu-devel] [PATCH v11 0/9] PC system flash support Jordan Justen
2012-02-22 7:18 ` [Qemu-devel] [PATCH v11 1/9] blockdev: allow read-only pflash devices Jordan Justen
2012-02-22 7:18 ` [Qemu-devel] [PATCH v11 2/9] pflash_cfi01/02: support " Jordan Justen
2012-02-22 7:18 ` [Qemu-devel] [PATCH v11 3/9] vl: make find_default_machine externally visible Jordan Justen
2012-02-22 7:18 ` [Qemu-devel] [PATCH v11 4/9] hw/pc: move rom init to pc_sysfw.c Jordan Justen
2012-02-22 7:18 ` [Qemu-devel] [PATCH v11 5/9] hw/pc_sysfw: enable pc-sysfw as a qdev Jordan Justen
2012-02-22 7:18 ` [Qemu-devel] [PATCH v11 6/9] hw/pc_sysfw: support system flash memory with pflash Jordan Justen
2012-02-22 7:18 ` [Qemu-devel] [PATCH v11 7/9] hw/pc_piix: remove is_default for pc-0.15 Jordan Justen
2012-02-22 7:18 ` [Qemu-devel] [PATCH v11 8/9] hw/pc_piix: add pc-1.1 Jordan Justen
2012-02-22 7:18 ` [Qemu-devel] [PATCH v11 9/9] pc_piix/pc_sysfw: enable flash by default Jordan Justen
2012-02-22 18:54 ` Anthony Liguori [this message]
2012-03-13 12:34 ` [Qemu-devel] [PATCH v11 0/9] PC system flash support Paolo Bonzini
2012-03-14 17:04 ` Jordan Justen
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=4F4539CA.2070600@codemonkey.ws \
--to=anthony@codemonkey.ws \
--cc=jordan.l.justen@intel.com \
--cc=qemu-devel@nongnu.org \
/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).