Go to file
Laszlo Ersek f58b39d2d5 virtio-mmio: format transport base address in BusClass.get_dev_path
At the moment the following QEMU command line triggers an assertion
failure (minimal reproducer by Cole):

  qemu-system-aarch64 \
    -machine virt-2.6,accel=tcg \
    -nodefaults \
    -no-user-config \
    -nographic -monitor stdio \
    -device virtio-scsi-device,id=scsi0 \
    -device virtio-scsi-device,id=scsi1 \
    -drive file=foo.img,format=raw,if=none,id=d0 \
    -device scsi-hd,bus=scsi0.0,drive=d0 \
    -drive file=foo.img,format=raw,if=none,id=d1 \
    -device scsi-hd,bus=scsi1.0,drive=d1

  qemu-system-aarch64: migration/savevm.c:615:
  vmstate_register_with_alias_id:
  Assertion `!se->compat || se->instance_id == 0' failed.

The reason is that the vmstate sections for the two scsi-hd devices are
not uniquely identifiable by name.

The direct parent buses of the scsi-hd devices -- scsi0.0 and scsi1.0 --
support the BusClass.get_dev_path member function. scsibus_get_dev_path()
formats a device path prefix with the help of its topologically parent
bus, and then appends the chan🆔lun triplet to it. For both scsi-hd
devices, this triplet is 0:0:0.

(Here we use "device path" in the QEMU migration sense, for vmstate
section identification, not in the OFW or UEFI device path senses.)

The virtio-scsi HBA is plugged into the virtio-mmio bus (implemented by
the internal VirtIOMMIOProxy device). This bus class
(TYPE_VIRTIO_MMIO_BUS) inherits, as its get_dev_path() member function,
the virtio_bus_get_dev_path() method from its parent class
(TYPE_VIRTIO_BUS).

virtio_bus_get_dev_path() does not format any kind of device address on
its own; "virtio addresses" are transport-specific. Therefore
virtio_bus_get_dev_path() asks the topologically parent bus of the proxy
object (implementing the specific virtio transport) to format the address
of the proxy object.

(For virtio-pci devices (where the proxy is an instance of VirtIOPCIProxy,
plugged into a PCI bus), this ends up in pcibus_get_dev_path().)

However, VirtIOMMIOProxy is usually (in practice: always) plugged into
"main-system-bus", the singleton TYPE_SYSTEM_BUS object. This BusClass
does not support formatting QEMU vmstate device paths at all (as
SysBusDevice objects can have zero or more IO ports and zero or more MMIO
regions). Hence the formatting request delegated from
virtio_bus_get_dev_path() gets answered with NULL.

The end result is that the two scsi-hd devices end up with the same device
path "0:0:0", which triggers the assert.

We can solve this by recognizing that virtio-mmio transports are
distinguished from each other by their base addresses in MMIO address
space. Implement virtio_mmio_bus_get_dev_path() as follows:

(1) The virtio device whose devpath is to be formatted resides on a
    virtio-mmio bus that is implemented by a VirtIOMMIOProxy object. Ask
    the parent bus of VirtIOMMIOProxy to format the device path of
    VirtIOMMIOProxy, as a path prefix. (This is identical to what
    virtio_bus_get_dev_path() does.)

(2) Append the base address of VirtIOMMIOProxy to the device path, such
    as:
    - virtio-mmio@000000000a003e00,
    - virtio-mmio@000000000a003c00.

Given that these device paths are placed in the migration stream, step (2)
above, if done unconditionally, would break migration. So make that step
conditional on a new VirtIOMMIOProxy property, which is enabled for 2.7
machine types and later.

Cc: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Cole Robinson <crobinso@redhat.com>
Cc: Dr. David Alan Gilbert <dgilbert@redhat.com>
Cc: Kevin Zhao <kevin.zhao@linaro.org>
Cc: Peter Maydell <peter.maydell@linaro.org>
Cc: Tom Hanson <thomas.hanson@linaro.org>
Reported-by: Kevin Zhao <kevin.zhao@linaro.org>
Reviewed-by: Andrew Jones <drjones@redhat.com>
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Message-id: 1467739394-28357-1-git-send-email-lersek@redhat.com
Fixes: https://bugs.launchpad.net/qemu/+bug/1594239
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
2016-07-14 16:51:36 +01:00
audio Clean up decorations and whitespace around header guards 2016-07-12 16:20:46 +02:00
backends * SCSI scanner support 2016-07-14 13:44:06 +01:00
block vvfat: Fix qcow write target driver specification 2016-07-13 13:41:39 +02:00
bsd-user Clean up decorations and whitespace around header guards 2016-07-12 16:20:46 +02:00
contrib Clean up ill-advised or unusual header guards 2016-07-12 16:20:46 +02:00
crypto Clean up ill-advised or unusual header guards 2016-07-12 16:20:46 +02:00
default-configs dma: Add Xilinx Zynq devcfg device model 2016-07-04 13:15:22 +01:00
disas tci: do not include exec/exec-all.h 2016-05-20 15:07:46 +01:00
docs blockjob: Update description of the 'device' field in the QMP API 2016-07-13 13:26:02 +02:00
dtc@65cc4d2748 dtc: Update dtc / libfdt submodule to version 1.4.0 2015-06-03 23:56:49 +02:00
fpu softfloat: Handle snan_bit_is_one == 0 in MIPS pickNaNMulAdd() 2016-06-24 13:41:32 +01:00
fsdev Clean up #include "..." vs <...> and header guards 2016-07-12 16:04:36 +01:00
gdb-xml target-ppc: gdbstub: Add VSX support 2016-01-30 23:37:38 +11:00
hw virtio-mmio: format transport base address in BusClass.get_dev_path 2016-07-14 16:51:36 +01:00
include virtio-mmio: format transport base address in BusClass.get_dev_path 2016-07-14 16:51:36 +01:00
io coroutine: move entry argument to qemu_coroutine_create 2016-07-13 13:26:02 +02:00
libdecnumber libdecnumber: Clean up includes 2016-02-16 14:29:27 +00:00
linux-headers linux-headers: update 2016-06-14 13:34:50 +02:00
linux-user Clean up decorations and whitespace around header guards 2016-07-12 16:20:46 +02:00
migration hmp: show all of snapshot info on every block dev in output of 'info snapshots' 2016-07-13 13:41:39 +02:00
nbd coroutine: move entry argument to qemu_coroutine_create 2016-07-13 13:26:02 +02:00
net * SCSI scanner support 2016-07-14 13:44:06 +01:00
pc-bios Last round of s390x patches for 2.7: 2016-07-11 18:46:38 +01:00
pixman@87eea99e44 pixman: update internal copy to pixman-0.32.6 2014-09-15 08:14:19 +02:00
po Added Bulgarian translation 2016-07-01 16:06:57 +01:00
qapi block: Remove BB options from blockdev-add 2016-07-13 13:32:28 +02:00
qga Clean up header guards that don't match their file name 2016-07-12 16:19:16 +02:00
qobject json-streamer: fix double-free on exiting during a parse 2016-07-12 18:31:27 +02:00
qom pc: Parse CPU features only once 2016-07-07 15:25:06 -03:00
replay replay: Use new QAPI cloning 2016-07-06 10:52:04 +02:00
roms OpenBIOS: switch over to official OpenBIOS git repo 2016-07-12 19:07:56 +01:00
scripts scripts: New clean-header-guards.pl 2016-07-12 16:19:16 +02:00
slirp Clean up decorations and whitespace around header guards 2016-07-12 16:20:46 +02:00
stubs pc: acpi: introduce AcpiDeviceIfClass.madt_cpu hook 2016-06-24 05:21:16 +03:00
target-alpha target-*: Clean up cpu.h header guards 2016-07-12 16:19:16 +02:00
target-arm target-*: Clean up cpu.h header guards 2016-07-12 16:19:16 +02:00
target-cris Clean up decorations and whitespace around header guards 2016-07-12 16:20:46 +02:00
target-i386 Clean up ill-advised or unusual header guards 2016-07-12 16:20:46 +02:00
target-lm32 target-*: Clean up cpu.h header guards 2016-07-12 16:19:16 +02:00
target-m68k target-*: Clean up cpu.h header guards 2016-07-12 16:19:16 +02:00
target-microblaze target-*: Clean up cpu.h header guards 2016-07-12 16:19:16 +02:00
target-mips Clean up ill-advised or unusual header guards 2016-07-12 16:20:46 +02:00
target-moxie target-*: Clean up cpu.h header guards 2016-07-12 16:19:16 +02:00
target-openrisc Clean up header guards that don't match their file name 2016-07-12 16:19:16 +02:00
target-ppc Clean up ill-advised or unusual header guards 2016-07-12 16:20:46 +02:00
target-s390x target-*: Clean up cpu.h header guards 2016-07-12 16:19:16 +02:00
target-sh4 target-*: Clean up cpu.h header guards 2016-07-12 16:19:16 +02:00
target-sparc target-sparc: Elide duplicate updates to fprs 2016-07-12 11:03:01 -07:00
target-tilegx Clean up header guards that don't match their file name 2016-07-12 16:19:16 +02:00
target-tricore Clean up decorations and whitespace around header guards 2016-07-12 16:20:46 +02:00
target-unicore32 target-*: Clean up cpu.h header guards 2016-07-12 16:19:16 +02:00
target-xtensa Clean up header guards that don't match their file name 2016-07-12 16:19:16 +02:00
tcg Clean up decorations and whitespace around header guards 2016-07-12 16:20:46 +02:00
tests iotests: Make 157 actually format-agnostic 2016-07-13 13:41:39 +02:00
trace Clean up decorations and whitespace around header guards 2016-07-12 16:20:46 +02:00
ui Clean up decorations and whitespace around header guards 2016-07-12 16:20:46 +02:00
util * SCSI scanner support 2016-07-14 13:44:06 +01:00
.dir-locals.el Add .dir-locals.el file to configure emacs coding style 2015-10-08 19:46:01 +03:00
.exrc
.gitignore trace: add build framework for merging trace-events files 2016-06-20 17:22:14 +01:00
.gitmodules PPC: Add u-boot firmware for e500 2014-06-16 13:24:35 +02:00
.mailmap
.travis.yml .travis.yml: disable Sparse testing 2016-06-08 17:47:22 +01:00
accel.c accel: make configure_accelerator return void 2016-05-18 15:04:27 +03:00
aio-posix.c aio-posix: Skip external nodes in aio_dispatch 2016-04-22 16:43:59 +02:00
aio-win32.c all: Clean up includes 2016-02-04 17:41:30 +00:00
arch_init.c cpu: Eliminate cpudef_init(), cpudef_setup() 2016-05-23 19:47:37 -03:00
async.c include/qemu/osdep.h: Don't include qapi/error.h 2016-03-22 22:20:15 +01:00
balloon.c all: Clean up includes 2016-02-04 17:41:30 +00:00
block.c coroutine: move entry argument to qemu_coroutine_create 2016-07-13 13:26:02 +02:00
blockdev-nbd.c nbd: enable use of TLS with nbd-server-start command 2016-02-16 17:17:49 +01:00
blockdev.c blockdev: Fix regression with the default naming of throttling groups 2016-07-13 13:41:39 +02:00
blockjob.c block/qdev: Allow configuring rerror/werror with qdev properties 2016-07-13 13:32:27 +02:00
bootdevice.c error: Remove NULL checks on error_propagate() calls 2016-06-20 16:38:13 +02:00
bt-host.c all: Clean up includes 2016-02-04 17:41:30 +00:00
bt-vhci.c all: Clean up includes 2016-02-04 17:41:30 +00:00
Changelog
CODING_STYLE CODING_STYLE: update mixed declaration rules 2015-09-09 15:34:54 +02:00
configure build: Use $(AS) for optionrom explicitly 2016-07-10 10:05:46 -07:00
COPYING
COPYING.LIB
cpu-exec-common.c cpu-exec: Rename cpu_resume_from_signal() to cpu_loop_exit_noexc() 2016-06-09 15:55:02 +01:00
cpu-exec.c tb hash: track translated blocks with qht 2016-06-11 17:11:16 -07:00
cpus.c seqlock: rename write_lock/unlock to write_begin/end 2016-06-11 22:59:34 +00:00
cputlb.c cputlb: Add address parameter to VICTIM_TLB_HIT 2016-07-08 13:04:41 -07:00
device_tree.c qemu-common: stop including qemu/bswap.h from qemu-common.h 2016-05-19 16:42:28 +02:00
device-hotplug.c blockdev: Split monitor reference from BB creation 2016-03-17 15:47:56 +01:00
disas.c all: Clean up includes 2016-02-04 17:41:30 +00:00
dma-helpers.c dma-helpers: dma_blk_io() cancel support 2016-06-28 13:08:31 +01:00
dump.c error: Remove NULL checks on error_propagate() calls 2016-06-20 16:38:13 +02:00
exec.c Use #include "..." for our own headers, <...> for others 2016-07-12 16:19:16 +02:00
gdbstub.c linux-user pull request for June 2016 2016-06-08 18:34:32 +01:00
HACKING HACKING: Add a section on error handling and reporting 2016-02-09 13:19:49 +01:00
hmp-commands-info.hx hmp: Add 'info hotpluggable-cpus' HMP command 2016-06-17 16:33:49 +10:00
hmp-commands.hx migration: define 'tls-creds' and 'tls-hostname' migration parameters 2016-05-26 11:32:10 +05:30
hmp.c stream: Add 'job-id' parameter to 'block-stream' 2016-07-13 13:26:02 +02:00
hmp.h hmp: Add 'info hotpluggable-cpus' HMP command 2016-06-17 16:33:49 +10:00
iohandler.c iohandler: Introduce iohandler_get_aio_context 2016-04-22 16:43:42 +02:00
ioport.c hw: remove pio_addr_t 2016-05-19 16:42:30 +02:00
iothread.c all: Clean up includes 2016-02-04 17:41:30 +00:00
kvm-all.c KVM: use KVM_CAP_MAX_VCPU_ID 2016-06-16 18:39:04 +02:00
kvm-stub.c cpu: Reclaim vCPU objects 2016-05-30 14:03:59 +10:00
LICENSE vfio: move hw/misc/vfio.c to hw/vfio/pci.c Move vfio.h into include/hw/vfio 2014-12-19 15:24:06 -07:00
main-loop.c main-loop: check return value before using pointer 2016-07-12 18:31:27 +02:00
MAINTAINERS MAINTAINERS: Remove Blue Swirl leftovers 2016-06-30 13:34:49 +01:00
Makefile Use "-s" instead of "--quiet" to resolve non-fatal build error on FreeBSD. 2016-07-12 18:31:26 +02:00
Makefile.objs trace: split out trace events for linux-user/ directory 2016-06-20 17:22:17 +01:00
Makefile.target linux-user: Create a hostdep.h for each host architecture 2016-06-26 13:17:21 +03:00
memory_mapping.c all: Remove unnecessary glib.h includes 2016-06-07 18:19:24 +03:00
memory.c memory: Assert that memory_region_init_rom_device() ops aren't NULL 2016-07-04 13:06:35 +01:00
module-common.c all: Clean up includes 2016-02-04 17:41:30 +00:00
monitor.c Drop Emacs local variables lists redundant with .dir-locals.el 2016-07-12 16:19:16 +02:00
numa.c opts-visitor: Favor new visit_free() function 2016-07-06 10:52:04 +02:00
os-posix.c os-posix: include sys/mman.h 2016-06-16 18:39:03 +02:00
os-win32.c all: Clean up includes 2016-02-04 17:41:30 +00:00
page_cache.c coccinelle: Remove unnecessary variables for function return value 2016-06-20 16:38:13 +02:00
qapi-schema.json tap: vhost busy polling support 2016-07-07 14:29:04 +08:00
qdev-monitor.c util: move declarations out of qemu-common.h 2016-03-22 22:20:17 +01:00
qdict-test-data.txt
qemu-bridge-helper.c all: Remove unnecessary glib.h includes 2016-06-07 18:19:24 +03:00
qemu-char.c char: do not use atexit cleanup handler 2016-07-13 13:30:00 +02:00
qemu-doc.texi Allow users to specify the vmdk virtual hardware version. 2016-05-12 15:22:08 +02:00
qemu-ga.texi docs: Style the command and its options in the synopsis 2016-01-26 15:58:11 +01:00
qemu-img-cmds.hx qemu-img bench: Add --flush-interval 2016-06-08 10:21:09 +02:00
qemu-img.c qemu-img: Use strerror() for generic resize error 2016-07-13 13:41:38 +02:00
qemu-img.texi trace: enable tracing in qemu-img 2016-06-28 21:14:12 +01:00
qemu-io-cmds.c qemu-io: Use correct range limitations 2016-07-13 13:41:38 +02:00
qemu-io.c trace: enable tracing in qemu-io 2016-06-28 21:14:12 +01:00
qemu-nbd.c trace: enable tracing in qemu-nbd 2016-06-28 21:14:12 +01:00
qemu-nbd.texi trace: enable tracing in qemu-nbd 2016-06-28 21:14:12 +01:00
qemu-option-trace.texi doc: move text describing --trace to specific .texi file 2016-06-28 21:14:12 +01:00
qemu-options-wrapper.h
qemu-options.h Clean up ill-advised or unusual header guards 2016-07-12 16:20:46 +02:00
qemu-options.hx tap: vhost busy polling support 2016-07-07 14:29:04 +08:00
qemu-seccomp.c seccomp: adding sysinfo system call to whitelist 2016-04-16 20:27:44 +02:00
qemu-tech.texi tcg: Rename tcg-target.c to tcg-target.inc.c 2016-02-23 08:30:38 -08:00
qemu-timer.c qemu-timer: Use DIV_ROUND_UP 2016-06-07 18:19:25 +03:00
qemu.nsi nsis: Add QEMU version information to Windows registry 2015-09-24 20:52:28 +02:00
qemu.sasl
qmp-commands.hx commit: Add 'job-id' parameter to 'block-commit' 2016-07-13 13:26:02 +02:00
qmp.c qmp-input-visitor: Favor new visit_free() function 2016-07-06 10:52:04 +02:00
qtest.c qemu-common: push cpu.h inclusion out of qemu-common.h 2016-05-19 16:42:29 +02:00
README README: fill out some useful quickstart information 2015-10-13 18:48:46 +02:00
rules.mak build: Use $(AS) for optionrom explicitly 2016-07-10 10:05:46 -07:00
softmmu_template.h cputlb: Fix for self-modifying writes across page boundaries 2016-07-08 13:05:07 -07:00
spice-qemu-char.c qapi: Don't special-case simple union wrappers 2016-03-18 10:29:26 +01:00
tcg-runtime.c all: Clean up includes 2016-02-04 17:41:30 +00:00
tci.c tci: do not include exec/exec-all.h 2016-05-20 15:07:46 +01:00
thread-pool.c coroutine: move entry argument to qemu_coroutine_create 2016-07-13 13:26:02 +02:00
thunk.c thunk: Rename args and fields in host-target bitmask conversion code 2016-06-07 18:19:24 +03:00
tpm.c qapi: Don't special-case simple union wrappers 2016-03-18 10:29:26 +01:00
trace-events trace: [*-user] Add events to trace guest syscalls in syscall emulation mode 2016-06-28 21:14:12 +01:00
translate-all.c translate-all: Fix user-mode self-modifying code in 2 page long TB 2016-07-08 13:17:38 -07:00
translate-all.h user-exec: Push resume-from-signal code out to handle_cpu_signal() 2016-06-09 15:55:02 +01:00
translate-common.c include: move CPU-related definitions out of qemu-common.h 2016-05-19 13:08:04 +02:00
user-exec.c user-exec: Remove unused code for OSX hosts 2016-06-26 13:17:20 +03:00
VERSION Open 2.7 development tree 2016-05-12 12:35:25 +01:00
version.rc
vl.c char: do not use atexit cleanup handler 2016-07-13 13:30:00 +02:00
xen-common-stub.c xen: Clean up includes 2016-01-29 15:07:23 +00:00
xen-common.c xen: drop XenXC and associated interface wrappers 2016-02-10 12:01:24 +00:00
xen-hvm-stub.c fix MSI injection on Xen 2016-02-06 20:44:10 +02:00
xen-hvm.c xen: fix ram init regression 2016-07-04 14:50:00 +03:00
xen-mapcache.c os-posix: include sys/mman.h 2016-06-16 18:39:03 +02:00

         QEMU README
         ===========

QEMU is a generic and open source machine & userspace emulator and
virtualizer.

QEMU is capable of emulating a complete machine in software without any
need for hardware virtualization support. By using dynamic translation,
it achieves very good performance. QEMU can also integrate with the Xen
and KVM hypervisors to provide emulated hardware while allowing the
hypervisor to manage the CPU. With hypervisor support, QEMU can achieve
near native performance for CPUs. When QEMU emulates CPUs directly it is
capable of running operating systems made for one machine (e.g. an ARMv7
board) on a different machine (e.g. an x86_64 PC board).

QEMU is also capable of providing userspace API virtualization for Linux
and BSD kernel interfaces. This allows binaries compiled against one
architecture ABI (e.g. the Linux PPC64 ABI) to be run on a host using a
different architecture ABI (e.g. the Linux x86_64 ABI). This does not
involve any hardware emulation, simply CPU and syscall emulation.

QEMU aims to fit into a variety of use cases. It can be invoked directly
by users wishing to have full control over its behaviour and settings.
It also aims to facilitate integration into higher level management
layers, by providing a stable command line interface and monitor API.
It is commonly invoked indirectly via the libvirt library when using
open source applications such as oVirt, OpenStack and virt-manager.

QEMU as a whole is released under the GNU General Public License,
version 2. For full licensing details, consult the LICENSE file.


Building
========

QEMU is multi-platform software intended to be buildable on all modern
Linux platforms, OS-X, Win32 (via the Mingw64 toolchain) and a variety
of other UNIX targets. The simple steps to build QEMU are:

  mkdir build
  cd build
  ../configure
  make

Complete details of the process for building and configuring QEMU for
all supported host platforms can be found in the qemu-tech.html file.
Additional information can also be found online via the QEMU website:

  http://qemu-project.org/Hosts/Linux
  http://qemu-project.org/Hosts/W32


Submitting patches
==================

The QEMU source code is maintained under the GIT version control system.

   git clone git://git.qemu-project.org/qemu.git

When submitting patches, the preferred approach is to use 'git
format-patch' and/or 'git send-email' to format & send the mail to the
qemu-devel@nongnu.org mailing list. All patches submitted must contain
a 'Signed-off-by' line from the author. Patches should follow the
guidelines set out in the HACKING and CODING_STYLE files.

Additional information on submitting patches can be found online via
the QEMU website

  http://qemu-project.org/Contribute/SubmitAPatch
  http://qemu-project.org/Contribute/TrivialPatches


Bug reporting
=============

The QEMU project uses Launchpad as its primary upstream bug tracker. Bugs
found when running code built from QEMU git or upstream released sources
should be reported via:

  https://bugs.launchpad.net/qemu/

If using QEMU via an operating system vendor pre-built binary package, it
is preferable to report bugs to the vendor's own bug tracker first. If
the bug is also known to affect latest upstream code, it can also be
reported via launchpad.

For additional information on bug reporting consult:

  http://qemu-project.org/Contribute/ReportABug


Contact
=======

The QEMU community can be contacted in a number of ways, with the two
main methods being email and IRC

 - qemu-devel@nongnu.org
   http://lists.nongnu.org/mailman/listinfo/qemu-devel
 - #qemu on irc.oftc.net

Information on additional methods of contacting the community can be
found online via the QEMU website:

  http://qemu-project.org/Contribute/StartHere

-- End