From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH v2 0/9] qemu-kvm: Clean up and enhance MSI irqchip support Date: Wed, 27 Apr 2011 12:34:13 +0300 Message-ID: <4DB7E315.6030004@redhat.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Marcelo Tosatti , kvm@vger.kernel.org, "Michael S. Tsirkin" To: Jan Kiszka Return-path: Received: from mx1.redhat.com ([209.132.183.28]:25912 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755368Ab1D0JeU (ORCPT ); Wed, 27 Apr 2011 05:34:20 -0400 In-Reply-To: Sender: kvm-owner@vger.kernel.org List-ID: On 04/26/2011 04:19 PM, Jan Kiszka wrote: > I've still plans to consolidate MSI-X mask notifiers and KVM hooks, but > that can wait until we go upstream. > > This version still makes classic MSI usable in irqchip mode, now not > only for PCI devices (AHCI, HDA) but also for the HPET (with msi=on). > Moreover, it contains an additional patch to refresh the MSI IRQ routes > after vmload. > Immediately after migration: Program terminated with signal 11, Segmentation fault. #0 le32_to_cpupu (vector=0, kmm=0x0, dev=) at ../bswap.h:178 178 return p1[0] | (p1[1] << 8) | (p1[2] << 16) | (p1[3] << 24); Missing separate debuginfos, use: debuginfo-install SDL-1.2.14-10.fc14.x86_64 cyrus-sasl-lib-2.1.23-12.fc14.x86_64 cyrus-sasl-plain-2.1.23-12.fc14.x86_64 db4-4.8.30-2.fc14.x86_64 glibc-2.13-1.x86_64 gnutls-2.8.6-2.fc14.x86_64 keyutils-libs-1.2-6.fc12.x86_64 krb5-libs-1.8.2-9.fc14.x86_64 libX11-1.3.4-4.fc14.x86_64 libXau-1.0.6-1.fc14.x86_64 libaio-0.3.109-2.fc13.x86_64 libcom_err-1.41.12-6.fc14.x86_64 libcurl-7.21.0-6.fc14.x86_64 libgcc-4.5.1-4.fc14.x86_64 libgcrypt-1.4.5-4.fc13.x86_64 libgpg-error-1.9-1.fc14.x86_64 libidn-1.18-1.fc14.x86_64 libpng-1.2.44-1.fc14.x86_64 libselinux-2.0.96-6.fc14.1.x86_64 libssh2-1.2.4-1.fc14.x86_64 libtasn1-2.7-1.fc14.x86_64 libxcb-1.7-1.fc14.x86_64 ncurses-libs-5.7-9.20100703.fc14.x86_64 nspr-4.8.7-1.fc14.x86_64 nss-3.12.9-9.fc14.x86_64 nss-softokn-freebl-3.12.9-5.fc14.x86_64 nss-util-3.12.9-1.fc14.x86_64 openldap-2.4.23-4.fc14.x86_64 openssl-1.0.0d-1.fc14.x86_64 zlib-1.2.5-2.fc14.x86_64 (gdb) bt #0 le32_to_cpupu (vector=0, kmm=0x0, dev=) at ../bswap.h:178 #1 pci_get_long (vector=0, kmm=0x0, dev=) at /build/home/tlv/akivity/qemu-kvm/hw/pci.h:326 #2 kvm_msi_message_from_vector (vector=0, kmm=0x0, dev=) at /build/home/tlv/akivity/qemu-kvm/hw/msi.c:120 #3 0x000000000057d59c in kvm_msi_update (dev=0x23eabb0) at /build/home/tlv/akivity/qemu-kvm/hw/msi.c:152 #4 0x000000000041e29b in get_pci_config_device (f=0x2466380, pv=0x23eac28, size=256) at /build/home/tlv/akivity/qemu-kvm/hw/pci.c:346 #5 0x000000000049c36c in vmstate_load_state (f=0x2466380, vmsd=0x5fb880, opaque=0x23eabb0, version_id=2) at savevm.c:1374 #6 0x000000000049c323 in vmstate_load_state (f=0x2466380, vmsd=0x6f07c0, opaque=0x23eabb0, version_id=3) at savevm.c:1372 #7 0x000000000049cf84 in vmstate_load (f=0x2466380) at savevm.c:1450 #8 qemu_loadvm_state (f=0x2466380) at savevm.c:1817 #9 0x0000000000493e69 in process_incoming_migration (f=) at migration.c:66 #10 0x0000000000494b97 in tcp_accept_incoming_migration (opaque=) at migration-tcp.c:163 #11 0x00000000004a3fa7 in qemu_iohandler_poll (readfds=0x7fff56dc0430, writefds=0x7fff56dc03b0, xfds=, ret=) at iohandler.c:120 #12 0x000000000041944a in main_loop_wait (nonblocking=) at /build/home/tlv/akivity/qemu-kvm/vl.c:1336 #13 0x0000000000433a97 in kvm_main_loop () at /build/home/tlv/akivity/qemu-kvm/qemu-kvm.c:1588 #14 0x000000000041a3a6 in main_loop (argc=, argv=, envp=) at /build/home/tlv/akivity/qemu-kvm/vl.c:1369 #15 main (argc=, argv=, envp=) at /build/home/tlv/akivity/qemu-kvm/vl.c:3257 (gdb) fr #3 0x000000000057d59c in kvm_msi_update (dev=0x23eabb0) at /build/home/tlv/akivity/qemu-kvm/hw/msi.c:152 (gdb) p dev.msi_irq_entries $10 = (struct KVMMsiMessage *) 0x0 dev points to the i440fx chipset device. -- error compiling committee.c: too many arguments to function