Re: [PATCH v2 0/3] hyperv: vmbus: ACPI various corrections
On Tue, 23 Jun 2020 17:44:57 +0200 Paolo Bonzini wrote: > On 23/06/20 16:57, Igor Mammedov wrote: > >>> > >>> dtc | 2 +- > >>> hw/hyperv/vmbus.c| 3 +-- > >>> hw/i386/acpi-build.c | 17 + > >>> include/hw/hyperv/vmbus-bridge.h | 3 +-- > >>> 4 files changed, 12 insertions(+), 13 deletions(-) > >>> > >> Queued, thanks. > >> > >> Paolo > >> > >> > > Paolo, there is a newer vesion on list > > and I don't think _ADR patch should be applied at all > > > > Can you explain? The only difference I saw was the bogus dtc update. for the record, why is explained in https://www.mail-archive.com/qemu-devel@nongnu.org/msg715460.html "Re: [PATCH v3 2/3] i386: acpi: vmbus: Add _ADR definition" change is not visible because vmbus is not enabled by default and doesn't have bios_tables test-case that would turn oi on and make it visible. > > Paolo >
Re: [PATCH v2 0/3] hyperv: vmbus: ACPI various corrections
On 23/06/20 16:57, Igor Mammedov wrote: >>> >>> dtc | 2 +- >>> hw/hyperv/vmbus.c| 3 +-- >>> hw/i386/acpi-build.c | 17 + >>> include/hw/hyperv/vmbus-bridge.h | 3 +-- >>> 4 files changed, 12 insertions(+), 13 deletions(-) >>> >> Queued, thanks. >> >> Paolo >> >> > Paolo, there is a newer vesion on list > and I don't think _ADR patch should be applied at all > Can you explain? The only difference I saw was the bogus dtc update. Paolo
Re: [PATCH v2 0/3] hyperv: vmbus: ACPI various corrections
On Tue, 23 Jun 2020 12:02:40 +0200 Paolo Bonzini wrote: > On 17/06/20 18:09, Jon Doron wrote: > > After doing further tests and looking at the latest HyperV ACPI DSDT. > > Do minor fix to our VMBus ACPI entry. > > > > v2: > > Renamed irq0 to irq now that there is a single IRQ required > > > > Jon Doron (3): > > hyperv: vmbus: Remove the 2nd IRQ > > i386: acpi: vmbus: Add _ADR definition > > acpi: i386: Move VMBus DSDT entry to SB > > > > dtc | 2 +- > > hw/hyperv/vmbus.c| 3 +-- > > hw/i386/acpi-build.c | 17 + > > include/hw/hyperv/vmbus-bridge.h | 3 +-- > > 4 files changed, 12 insertions(+), 13 deletions(-) > > > > Queued, thanks. > > Paolo > > Paolo, there is a newer vesion on list and I don't think _ADR patch should be applied at all
Re: [PATCH v2 0/3] hyperv: vmbus: ACPI various corrections
On 17/06/20 18:09, Jon Doron wrote: > After doing further tests and looking at the latest HyperV ACPI DSDT. > Do minor fix to our VMBus ACPI entry. > > v2: > Renamed irq0 to irq now that there is a single IRQ required > > Jon Doron (3): > hyperv: vmbus: Remove the 2nd IRQ > i386: acpi: vmbus: Add _ADR definition > acpi: i386: Move VMBus DSDT entry to SB > > dtc | 2 +- > hw/hyperv/vmbus.c| 3 +-- > hw/i386/acpi-build.c | 17 + > include/hw/hyperv/vmbus-bridge.h | 3 +-- > 4 files changed, 12 insertions(+), 13 deletions(-) > Queued, thanks. Paolo
Re: [PATCH v2 0/3] hyperv: vmbus: ACPI various corrections
Patchew URL: https://patchew.org/QEMU/20200617160904.681845-1-ari...@gmail.com/ Hi, This series failed the asan build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN === #!/bin/bash export ARCH=x86_64 make docker-image-fedora V=1 NETWORK=1 time make docker-test-debug@fedora TARGET_LIST=x86_64-softmmu J=14 NETWORK=1 === TEST SCRIPT END === CC qga/main.o CC qga/commands-posix.o CC qga/channel-posix.o /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) CC qga/qapi-generated/qga-qapi-types.o CC qga/qapi-generated/qga-qapi-visit.o CC qga/qapi-generated/qga-qapi-commands.o CC qga/qapi-generated/qga-qapi-init-commands.o AR libqemuutil.a LINKelf2dmp /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) CC qemu-img.o AR libvhost-user.a GEN docs/interop/qemu-ga-ref.html --- LINKqemu-ga LINKqemu-keymap LINKivshmem-client /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) LINKivshmem-server /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) LINKqemu-nbd /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) AS pc-bios/optionrom/linuxboot.o AS pc-bios/optionrom/multiboot.o CC pc-bios/optionrom/linuxboot_dma.o AS pc-bios/optionrom/kvmvapic.o /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) AS pc-bios/optionrom/pvh.o CC pc-bios/optionrom/pvh_main.o BUILD pc-bios/optionrom/multiboot.img --- LINKqemu-edid LINKfsdev/virtfs-proxy-helper LINKscsi/qemu-pr-helper /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) LINKqemu-bridge-helper LINKvirtiofsd /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o) BUILD pc-bios/optionrom/pvh.img BUILD pc-bios/optionrom/pvh.raw SIGNpc-bios/optionrom/pvh.bin /usr/bin/ld: /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors_vfork.S.o): warning: common of `__interception::real_vfork' overridden by definition from /usr/lib64/clang/10.0.0/lib/linux/libclang_rt.asan-x86_64.a(asan_interceptors.cpp.o)
[PATCH v2 0/3] hyperv: vmbus: ACPI various corrections
After doing further tests and looking at the latest HyperV ACPI DSDT. Do minor fix to our VMBus ACPI entry. v2: Renamed irq0 to irq now that there is a single IRQ required Jon Doron (3): hyperv: vmbus: Remove the 2nd IRQ i386: acpi: vmbus: Add _ADR definition acpi: i386: Move VMBus DSDT entry to SB dtc | 2 +- hw/hyperv/vmbus.c| 3 +-- hw/i386/acpi-build.c | 17 + include/hw/hyperv/vmbus-bridge.h | 3 +-- 4 files changed, 12 insertions(+), 13 deletions(-) -- 2.24.1