On Fri, Mar 08, 2019 at 02:14:33PM +0100, Markus Armbruster wrote: > Based-on: <20190308094610.21210-1-arm...@redhat.com> > > v3: > * PATCH 05: Commit message tidied up [Philippe, László] > * PATCH 06: Comment tidied up [checkpatch] > * PATCH 12: Comment polish [László] > > v2: > * PATCH 05,06: New [Philippe] > * PATCH 07: Old PATCH 5 rebased onto new patches, with new comment > [Philippe] > * PATCH 08+10: New, factored out of old PATCH 6 [by Philippe] > * PATCH 09: New [by Philippe] > * PATCH 11: Remainder of old PATCH 6, with FIXME resolved, literal > 4096 de-duplicated [László], extraneous error check deleted > [László], and comment typos fixed > * PATCH 12: New > > Markus Armbruster (9): > qdev: Fix latent bug with compat_props and onboard devices > qom: Move compat_props machinery from qdev to QOM > vl: Fix latent bug with -global and onboard devices > sysbus: Fix latent bug with onboard devices > vl: Improve legibility of BlockdevOptions queue > vl: Factor configure_blockdev() out of main() > vl: Create block backends before setting machine properties > pc: Support firmware configuration with -blockdev > docs/interop/firmware.json: Prefer -machine to if=pflash > > Philippe Mathieu-Daudé (3): > pflash_cfi01: Add pflash_cfi01_get_blk() helper > pc_sysfw: Remove unused PcSysFwDevice > pc_sysfw: Pass PCMachineState to pc_system_firmware_init()
OK looks good to me. Reviewed-by: Michael S. Tsirkin <m...@redhat.com> Paolo will you be merging this? > accel/accel.c | 1 + > docs/interop/firmware.json | 20 ++- > hw/block/pflash_cfi01.c | 5 + > hw/core/qdev.c | 21 +--- > hw/core/sysbus.c | 3 - > hw/i386/pc.c | 4 +- > hw/i386/pc_sysfw.c | 249 ++++++++++++++++++++++++------------- > include/hw/block/flash.h | 1 + > include/hw/i386/pc.h | 6 +- > include/hw/qdev-core.h | 2 - > include/qom/object.h | 3 + > qom/object.c | 39 ++++++ > vl.c | 125 ++++++++++--------- > 13 files changed, 303 insertions(+), 176 deletions(-) > > -- > 2.17.2
