* [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19
@ 2024-12-19 13:48 Paolo Bonzini
2024-12-19 13:48 ` [PULL 29/42] bql: add a "mock" BQL for Rust unit tests Paolo Bonzini
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Paolo Bonzini @ 2024-12-19 13:48 UTC (permalink / raw)
To: qemu-devel
The following changes since commit 8032c78e556cd0baec111740a6c636863f9bd7c8:
Merge tag 'firmware-20241216-pull-request' of https://gitlab.com/kraxel/qemu into staging (2024-12-16 14:20:33 -0500)
are available in the Git repository at:
https://gitlab.com/bonzini/qemu.git tags/for-upstream
for you to fetch changes up to 9c70b74bd8c7a3753b80a1aab22a0dc58da1ed68:
rust: pl011: simplify handling of the FIFO enabled bit in LCR (2024-12-19 14:37:58 +0100)
----------------------------------------------------------------
* qdev: second part of Property cleanups
* rust: second part of QOM rework
* rust: callbacks wrapper
* rust: pl011 bugfixes
* kvm: cleanup errors in kvm_convert_memory()
----------------------------------------------------------------
v1->v2: I noticed that "meson test" returns success even if the tests actually failed
with SIGSEGV! Sorry for not noticing that before. For now I added a bandaid
in the form of "--test-threads 1" and replaced the Mutex<Cell<bool>> with a
BqlCell<bool> for documentation purposes.
Maciej S. Szmigiero (1):
target/i386: Reset TSCs of parked vCPUs too on VM reset
Paolo Bonzini (17):
rust: qom: put class_init together from multiple ClassInitImpl<>
rust: qom: add possibility of overriding unparent
rust: rename qemu-api modules to follow C code a bit more
rust: re-export C types from qemu-api submodules
bql: add a "mock" BQL for Rust unit tests
rust: tests: allow writing more than one test
rust: qom: add casting functionality
rust: qom: add initial subset of methods on Object
rust: qemu-api: add a module to wrap functions and zero-sized closures
kvm: consistently return 0/-errno from kvm_convert_memory
rust: pl011: fix declaration of LineControl bits
rust: pl011: match break logic of C version
rust: pl011: always use reset() method on registers
rust: pl011: fix break errors and definition of Data struct
rust: pl011: extend registers to 32 bits
rust: pl011: fix migration stream
rust: pl011: simplify handling of the FIFO enabled bit in LCR
Richard Henderson (24):
migration: Constify migration_properties
hw/ide: Constify sysbus_ahci_properties
target/ppc: Remove empty property list
target/s390x: Use s390x_cpu_properties for system mode only
hw/pci-host/astro: Remove empty Property list
hw/ppc: Only register spapr_nvdimm_properties if CONFIG_LIBPMEM
hw/tricore: Remove empty Property lists
hw/s390x: Remove empty Property lists
hw/xen: Remove empty Property lists
hw/sparc: Remove empty Property lists
hw/virtio: Remove empty Property lists
include/hw/qdev-core: Detect most empty Property lists at compile time
hw/core: Introduce device_class_set_props_n
migration: Use device_class_set_props_n
hw/scsi/megasas: Use device_class_set_props_n
hw/arm/armsse: Use device_class_set_props_n
rust/qemu-api: Use device_class_set_props_n
hw/core: Replace device_class_set_props with a macro
target/riscv: Do not abuse DEFINE_PROP_END_OF_LIST
include/hw/qdev-properties: Remove DEFINE_PROP_END_OF_LIST
include/hw/qdev-properties: Shrink struct Property
hw/core/qdev-properties: Constify Property argument to object_field_prop_ptr
hw/core/qdev-properties: Constify Property argument to PropertyInfo.print
Constify all opaque Property pointers
docs/devel/migration/compatibility.rst | 1 -
docs/devel/virtio-backends.rst | 1 -
configs/targets/i386-softmmu.mak | 1 +
configs/targets/x86_64-softmmu.mak | 1 +
include/hw/qdev-core.h | 35 +-
include/hw/qdev-properties.h | 15 +-
include/qemu/main-loop.h | 8 +
include/sysemu/kvm.h | 8 +
migration/options.h | 3 +-
accel/kvm/kvm-all.c | 19 +-
backends/tpm/tpm_util.c | 4 +-
cpu-target.c | 1 -
hw/9pfs/virtio-9p-device.c | 1 -
hw/acpi/erst.c | 1 -
hw/acpi/generic_event_device.c | 1 -
hw/acpi/piix4.c | 1 -
hw/acpi/vmgenid.c | 1 -
hw/adc/aspeed_adc.c | 1 -
hw/adc/npcm7xx_adc.c | 1 -
hw/arm/armsse.c | 9 +-
hw/arm/armv7m.c | 2 -
hw/arm/aspeed_soc_common.c | 1 -
hw/arm/fsl-imx25.c | 1 -
hw/arm/fsl-imx6.c | 1 -
hw/arm/fsl-imx6ul.c | 1 -
hw/arm/fsl-imx7.c | 1 -
hw/arm/integratorcp.c | 1 -
hw/arm/msf2-soc.c | 1 -
hw/arm/npcm7xx.c | 1 -
hw/arm/nrf51_soc.c | 1 -
hw/arm/smmu-common.c | 1 -
hw/arm/smmuv3.c | 1 -
hw/arm/stellaris.c | 1 -
hw/arm/strongarm.c | 1 -
hw/arm/xlnx-versal.c | 1 -
hw/arm/xlnx-zynqmp.c | 1 -
hw/audio/ac97.c | 1 -
hw/audio/adlib.c | 1 -
hw/audio/asc.c | 1 -
hw/audio/cs4231a.c | 1 -
hw/audio/es1370.c | 1 -
hw/audio/gus.c | 1 -
hw/audio/hda-codec.c | 1 -
hw/audio/intel-hda.c | 2 -
hw/audio/pcspk.c | 1 -
hw/audio/pl041.c | 1 -
hw/audio/sb16.c | 1 -
hw/audio/via-ac97.c | 1 -
hw/audio/virtio-snd-pci.c | 1 -
hw/audio/virtio-snd.c | 1 -
hw/audio/wm8750.c | 1 -
hw/avr/atmega.c | 1 -
hw/block/fdc-isa.c | 1 -
hw/block/fdc-sysbus.c | 2 -
hw/block/fdc.c | 1 -
hw/block/m25p80.c | 1 -
hw/block/nand.c | 1 -
hw/block/pflash_cfi01.c | 1 -
hw/block/pflash_cfi02.c | 1 -
hw/block/swim.c | 1 -
hw/block/vhost-user-blk.c | 1 -
hw/block/virtio-blk.c | 1 -
hw/block/xen-block.c | 5 +-
hw/char/avr_usart.c | 1 -
hw/char/bcm2835_aux.c | 1 -
hw/char/cadence_uart.c | 1 -
hw/char/cmsdk-apb-uart.c | 1 -
hw/char/debugcon.c | 1 -
hw/char/digic-uart.c | 1 -
hw/char/escc.c | 1 -
hw/char/exynos4210_uart.c | 1 -
hw/char/goldfish_tty.c | 1 -
hw/char/grlib_apbuart.c | 1 -
hw/char/ibex_uart.c | 1 -
hw/char/imx_serial.c | 1 -
hw/char/ipoctal232.c | 1 -
hw/char/mcf_uart.c | 1 -
hw/char/nrf51_uart.c | 1 -
hw/char/parallel.c | 1 -
hw/char/pl011.c | 1 -
hw/char/renesas_sci.c | 1 -
hw/char/sclpconsole-lm.c | 1 -
hw/char/sclpconsole.c | 1 -
hw/char/serial-isa.c | 1 -
hw/char/serial-mm.c | 1 -
hw/char/serial-pci-multi.c | 2 -
hw/char/serial-pci.c | 1 -
hw/char/serial.c | 1 -
hw/char/sh_serial.c | 1 -
hw/char/shakti_uart.c | 1 -
hw/char/sifive_uart.c | 1 -
hw/char/spapr_vty.c | 1 -
hw/char/stm32f2xx_usart.c | 1 -
hw/char/stm32l4x5_usart.c | 1 -
hw/char/terminal3270.c | 1 -
hw/char/virtio-console.c | 1 -
hw/char/virtio-serial-bus.c | 2 -
hw/char/xen_console.c | 1 -
hw/char/xilinx_uartlite.c | 1 -
hw/core/generic-loader.c | 1 -
hw/core/guest-loader.c | 1 -
hw/core/or-irq.c | 1 -
hw/core/platform-bus.c | 1 -
hw/core/qdev-properties-system.c | 50 +--
hw/core/qdev-properties.c | 107 ++---
hw/core/qdev.c | 1 +
hw/core/split-irq.c | 1 -
hw/cpu/a15mpcore.c | 1 -
hw/cpu/a9mpcore.c | 1 -
hw/cpu/arm11mpcore.c | 1 -
hw/cpu/cluster.c | 1 -
hw/cpu/realview_mpcore.c | 1 -
hw/cxl/switch-mailbox-cci.c | 1 -
hw/display/artist.c | 1 -
hw/display/ati.c | 1 -
hw/display/bcm2835_fb.c | 1 -
hw/display/bochs-display.c | 1 -
hw/display/cg3.c | 1 -
hw/display/cirrus_vga.c | 1 -
hw/display/cirrus_vga_isa.c | 1 -
hw/display/exynos4210_fimd.c | 1 -
hw/display/g364fb.c | 1 -
hw/display/i2c-ddc.c | 1 -
hw/display/macfb.c | 2 -
hw/display/pl110.c | 1 -
hw/display/qxl.c | 1 -
hw/display/ramfb-standalone.c | 1 -
hw/display/sm501.c | 2 -
hw/display/tcx.c | 1 -
hw/display/vga-isa.c | 1 -
hw/display/vga-mmio.c | 1 -
hw/display/vga-pci.c | 2 -
hw/display/vhost-user-gpu.c | 1 -
hw/display/virtio-gpu-gl.c | 1 -
hw/display/virtio-gpu-pci.c | 1 -
hw/display/virtio-gpu-rutabaga.c | 1 -
hw/display/virtio-gpu.c | 1 -
hw/display/virtio-vga.c | 1 -
hw/display/vmware_vga.c | 1 -
hw/display/xlnx_dp.c | 1 -
hw/dma/i82374.c | 1 -
hw/dma/i8257.c | 1 -
hw/dma/pl080.c | 1 -
hw/dma/pl330.c | 2 -
hw/dma/xilinx_axidma.c | 1 -
hw/dma/xlnx-zdma.c | 1 -
hw/dma/xlnx_csu_dma.c | 1 -
hw/gpio/imx_gpio.c | 1 -
hw/gpio/npcm7xx_gpio.c | 1 -
hw/gpio/omap_gpio.c | 1 -
hw/gpio/pca9552.c | 1 -
hw/gpio/pca9554.c | 1 -
hw/gpio/pl061.c | 1 -
hw/gpio/sifive_gpio.c | 1 -
hw/gpio/stm32l4x5_gpio.c | 1 -
hw/hyperv/hv-balloon.c | 2 -
hw/hyperv/syndbg.c | 1 -
hw/hyperv/vmbus.c | 2 -
hw/i2c/aspeed_i2c.c | 2 -
hw/i2c/core.c | 1 -
hw/i2c/i2c_mux_pca954x.c | 1 -
hw/i2c/omap_i2c.c | 1 -
hw/i386/amd_iommu.c | 1 -
hw/i386/intel_iommu.c | 1 -
hw/i386/kvm/clock.c | 1 -
hw/i386/kvm/i8254.c | 1 -
hw/i386/kvm/ioapic.c | 1 -
hw/i386/sgx-epc.c | 1 -
hw/i386/vmmouse.c | 1 -
hw/i386/vmport.c | 2 -
hw/i386/x86-iommu.c | 1 -
hw/i386/xen/xen_pvdevice.c | 1 -
hw/ide/ahci-sysbus.c | 3 +-
hw/ide/cf.c | 1 -
hw/ide/cmd646.c | 1 -
hw/ide/ide-dev.c | 3 -
hw/ide/isa.c | 1 -
hw/ide/macio.c | 1 -
hw/ide/mmio.c | 1 -
hw/input/pckbd.c | 2 -
hw/input/stellaris_gamepad.c | 1 -
hw/input/virtio-input-hid.c | 3 -
hw/input/virtio-input-host.c | 1 -
hw/input/virtio-input.c | 1 -
hw/intc/apic_common.c | 1 -
hw/intc/arm_gic_common.c | 1 -
hw/intc/arm_gicv2m.c | 1 -
hw/intc/arm_gicv3_common.c | 1 -
hw/intc/arm_gicv3_its.c | 1 -
hw/intc/arm_gicv3_its_kvm.c | 1 -
hw/intc/armv7m_nvic.c | 1 -
hw/intc/exynos4210_combiner.c | 1 -
hw/intc/exynos4210_gic.c | 1 -
hw/intc/goldfish_pic.c | 1 -
hw/intc/grlib_irqmp.c | 1 -
hw/intc/i8259_common.c | 1 -
hw/intc/ioapic.c | 1 -
hw/intc/loongarch_extioi.c | 1 -
hw/intc/loongarch_pch_msi.c | 1 -
hw/intc/loongarch_pch_pic.c | 1 -
hw/intc/loongson_ipi_common.c | 1 -
hw/intc/m68k_irqc.c | 1 -
hw/intc/mips_gic.c | 1 -
hw/intc/omap_intc.c | 1 -
hw/intc/ompic.c | 1 -
hw/intc/openpic.c | 1 -
hw/intc/openpic_kvm.c | 1 -
hw/intc/pnv_xive.c | 1 -
hw/intc/pnv_xive2.c | 1 -
hw/intc/ppc-uic.c | 1 -
hw/intc/riscv_aclint.c | 2 -
hw/intc/riscv_aplic.c | 1 -
hw/intc/riscv_imsic.c | 1 -
hw/intc/rx_icu.c | 1 -
hw/intc/s390_flic.c | 2 -
hw/intc/sifive_plic.c | 1 -
hw/intc/spapr_xive.c | 1 -
hw/intc/xics.c | 2 -
hw/intc/xilinx_intc.c | 1 -
hw/intc/xive.c | 4 -
hw/intc/xive2.c | 2 -
hw/intc/xlnx-pmu-iomod-intc.c | 1 -
hw/ipack/ipack.c | 1 -
hw/ipmi/ipmi.c | 1 -
hw/ipmi/ipmi_bmc_extern.c | 1 -
hw/ipmi/ipmi_bmc_sim.c | 1 -
hw/ipmi/isa_ipmi_bt.c | 1 -
hw/ipmi/isa_ipmi_kcs.c | 1 -
hw/isa/lpc_ich9.c | 1 -
hw/isa/pc87312.c | 1 -
hw/isa/piix.c | 1 -
hw/m68k/mcf5206.c | 1 -
hw/m68k/mcf_intc.c | 1 -
hw/m68k/next-cube.c | 1 -
hw/m68k/q800-glue.c | 1 -
hw/mem/cxl_type3.c | 1 -
hw/mem/nvdimm.c | 1 -
hw/mem/pc-dimm.c | 1 -
hw/mem/sparse-mem.c | 1 -
hw/mips/cps.c | 1 -
hw/misc/a9scu.c | 1 -
hw/misc/allwinner-h3-dramc.c | 1 -
hw/misc/allwinner-r40-dramc.c | 1 -
hw/misc/allwinner-sid.c | 1 -
hw/misc/applesmc.c | 1 -
hw/misc/arm11scu.c | 1 -
hw/misc/arm_l2x0.c | 1 -
hw/misc/arm_sysctl.c | 1 -
hw/misc/armsse-cpuid.c | 1 -
hw/misc/aspeed_hace.c | 1 -
hw/misc/aspeed_i3c.c | 1 -
hw/misc/aspeed_lpc.c | 1 -
hw/misc/aspeed_sbc.c | 1 -
hw/misc/aspeed_scu.c | 1 -
hw/misc/aspeed_sdmc.c | 1 -
hw/misc/bcm2835_cprman.c | 1 -
hw/misc/bcm2835_property.c | 1 -
hw/misc/debugexit.c | 1 -
hw/misc/eccmemctl.c | 1 -
hw/misc/empty_slot.c | 1 -
hw/misc/iotkit-secctl.c | 1 -
hw/misc/iotkit-sysctl.c | 1 -
hw/misc/iotkit-sysinfo.c | 1 -
hw/misc/ivshmem.c | 2 -
hw/misc/led.c | 1 -
hw/misc/mac_via.c | 1 -
hw/misc/macio/cuda.c | 1 -
hw/misc/macio/macio.c | 2 -
hw/misc/macio/pmu.c | 1 -
hw/misc/mips_cmgcr.c | 1 -
hw/misc/mips_cpc.c | 1 -
hw/misc/mips_itu.c | 1 -
hw/misc/mos6522.c | 1 -
hw/misc/mps2-fpgaio.c | 1 -
hw/misc/mps2-scc.c | 1 -
hw/misc/msf2-sysreg.c | 1 -
hw/misc/npcm7xx_gcr.c | 1 -
hw/misc/nrf51_rng.c | 1 -
hw/misc/pci-testdev.c | 1 -
hw/misc/pvpanic-isa.c | 1 -
hw/misc/pvpanic-pci.c | 1 -
hw/misc/sifive_e_aon.c | 1 -
hw/misc/sifive_u_otp.c | 1 -
hw/misc/stm32l4x5_rcc.c | 1 -
hw/misc/tz-mpc.c | 1 -
hw/misc/tz-msc.c | 1 -
hw/misc/tz-ppc.c | 1 -
hw/misc/unimp.c | 1 -
hw/misc/xlnx-versal-cframe-reg.c | 2 -
hw/misc/xlnx-versal-cfu.c | 2 -
hw/misc/xlnx-versal-trng.c | 4 +-
hw/misc/xlnx-versal-xramc.c | 1 -
hw/misc/zynq_slcr.c | 1 -
hw/net/allwinner-sun8i-emac.c | 1 -
hw/net/allwinner_emac.c | 1 -
hw/net/cadence_gem.c | 1 -
hw/net/can/xlnx-versal-canfd.c | 1 -
hw/net/can/xlnx-zynqmp-can.c | 1 -
hw/net/dp8393x.c | 1 -
hw/net/e1000.c | 1 -
hw/net/e1000e.c | 1 -
hw/net/eepro100.c | 1 -
hw/net/fsl_etsec/etsec.c | 1 -
hw/net/ftgmac100.c | 2 -
hw/net/igb.c | 1 -
hw/net/imx_fec.c | 1 -
hw/net/lan9118.c | 1 -
hw/net/lance.c | 1 -
hw/net/lasi_i82596.c | 1 -
hw/net/mcf_fec.c | 1 -
hw/net/mipsnet.c | 1 -
hw/net/msf2-emac.c | 1 -
hw/net/mv88w8618_eth.c | 1 -
hw/net/ne2000-isa.c | 1 -
hw/net/ne2000-pci.c | 1 -
hw/net/npcm7xx_emc.c | 1 -
hw/net/npcm_gmac.c | 1 -
hw/net/opencores_eth.c | 1 -
hw/net/pcnet-pci.c | 1 -
hw/net/rocker/rocker.c | 1 -
hw/net/rtl8139.c | 1 -
hw/net/smc91c111.c | 1 -
hw/net/spapr_llan.c | 1 -
hw/net/stellaris_enet.c | 1 -
hw/net/sungem.c | 1 -
hw/net/sunhme.c | 1 -
hw/net/tulip.c | 1 -
hw/net/virtio-net.c | 1 -
hw/net/vmxnet3.c | 1 -
hw/net/xen_nic.c | 1 -
hw/net/xgmac.c | 1 -
hw/net/xilinx_axienet.c | 1 -
hw/net/xilinx_ethlite.c | 1 -
hw/nubus/nubus-bridge.c | 1 -
hw/nubus/nubus-device.c | 1 -
hw/nvme/ctrl.c | 1 -
hw/nvme/nguid.c | 4 +-
hw/nvme/ns.c | 1 -
hw/nvme/subsys.c | 1 -
hw/nvram/ds1225y.c | 1 -
hw/nvram/eeprom_at24c.c | 1 -
hw/nvram/fw_cfg.c | 3 -
hw/nvram/mac_nvram.c | 1 -
hw/nvram/nrf51_nvm.c | 1 -
hw/nvram/spapr_nvram.c | 1 -
hw/nvram/xlnx-bbram.c | 1 -
hw/nvram/xlnx-efuse.c | 1 -
hw/nvram/xlnx-versal-efuse-cache.c | 2 -
hw/nvram/xlnx-versal-efuse-ctrl.c | 2 -
hw/nvram/xlnx-zynqmp-efuse.c | 2 -
hw/pci-bridge/cxl_downstream.c | 1 -
hw/pci-bridge/cxl_root_port.c | 1 -
hw/pci-bridge/cxl_upstream.c | 1 -
hw/pci-bridge/gen_pcie_root_port.c | 1 -
hw/pci-bridge/pci_bridge_dev.c | 1 -
hw/pci-bridge/pci_expander_bridge.c | 2 -
hw/pci-bridge/pcie_pci_bridge.c | 1 -
hw/pci-bridge/pcie_root_port.c | 1 -
hw/pci-bridge/xio3130_downstream.c | 1 -
hw/pci-host/astro.c | 5 -
hw/pci-host/dino.c | 1 -
hw/pci-host/gpex.c | 1 -
hw/pci-host/grackle.c | 1 -
hw/pci-host/gt64120.c | 1 -
hw/pci-host/i440fx.c | 1 -
hw/pci-host/mv64361.c | 1 -
hw/pci-host/pnv_phb.c | 4 -
hw/pci-host/pnv_phb3.c | 1 -
hw/pci-host/pnv_phb4.c | 1 -
hw/pci-host/pnv_phb4_pec.c | 1 -
hw/pci-host/ppce500.c | 1 -
hw/pci-host/q35.c | 2 -
hw/pci-host/raven.c | 1 -
hw/pci-host/sabre.c | 1 -
hw/pci-host/uninorth.c | 1 -
hw/pci-host/versatile.c | 1 -
hw/pci-host/xilinx-pcie.c | 1 -
hw/pci/pci.c | 1 -
hw/pci/pci_bridge.c | 1 -
hw/pci/pci_host.c | 1 -
hw/pci/pcie_port.c | 2 -
hw/ppc/pnv.c | 1 -
hw/ppc/pnv_adu.c | 1 -
hw/ppc/pnv_chiptod.c | 1 -
hw/ppc/pnv_core.c | 2 -
hw/ppc/pnv_homer.c | 1 -
hw/ppc/pnv_i2c.c | 1 -
hw/ppc/pnv_lpc.c | 1 -
hw/ppc/pnv_pnor.c | 1 -
hw/ppc/pnv_psi.c | 1 -
hw/ppc/ppc405_uc.c | 1 -
hw/ppc/ppc440_uc.c | 1 -
hw/ppc/ppc4xx_devs.c | 2 -
hw/ppc/ppc4xx_sdram.c | 2 -
hw/ppc/prep_systemio.c | 1 -
hw/ppc/rs6000_mc.c | 1 -
hw/ppc/spapr_cpu_core.c | 1 -
hw/ppc/spapr_nvdimm.c | 10 +-
hw/ppc/spapr_pci.c | 1 -
hw/ppc/spapr_rng.c | 1 -
hw/ppc/spapr_tpm_proxy.c | 1 -
hw/remote/proxy.c | 1 -
hw/riscv/opentitan.c | 1 -
hw/riscv/riscv-iommu-pci.c | 1 -
hw/riscv/riscv-iommu.c | 1 -
hw/riscv/riscv_hart.c | 1 -
hw/riscv/sifive_u.c | 1 -
hw/rtc/allwinner-rtc.c | 1 -
hw/rtc/goldfish_rtc.c | 1 -
hw/rtc/m48t59-isa.c | 1 -
hw/rtc/m48t59.c | 1 -
hw/rtc/mc146818rtc.c | 1 -
hw/rtc/pl031.c | 1 -
hw/rx/rx62n.c | 1 -
hw/s390x/3270-ccw.c | 5 -
hw/s390x/ccw-device.c | 1 -
hw/s390x/css-bridge.c | 1 -
hw/s390x/css.c | 4 +-
hw/s390x/ipl.c | 1 -
hw/s390x/s390-pci-bus.c | 5 +-
hw/s390x/s390-skeys.c | 1 -
hw/s390x/s390-stattrib.c | 1 -
hw/s390x/vhost-scsi-ccw.c | 1 -
hw/s390x/vhost-user-fs-ccw.c | 1 -
hw/s390x/vhost-vsock-ccw.c | 1 -
hw/s390x/virtio-ccw-9p.c | 1 -
hw/s390x/virtio-ccw-balloon.c | 1 -
hw/s390x/virtio-ccw-blk.c | 1 -
hw/s390x/virtio-ccw-crypto.c | 1 -
hw/s390x/virtio-ccw-gpu.c | 1 -
hw/s390x/virtio-ccw-input.c | 1 -
hw/s390x/virtio-ccw-net.c | 1 -
hw/s390x/virtio-ccw-rng.c | 1 -
hw/s390x/virtio-ccw-scsi.c | 1 -
hw/s390x/virtio-ccw-serial.c | 1 -
hw/scsi/megasas.c | 7 +-
hw/scsi/mptsas.c | 1 -
hw/scsi/scsi-bus.c | 1 -
hw/scsi/scsi-disk.c | 3 -
hw/scsi/scsi-generic.c | 1 -
hw/scsi/spapr_vscsi.c | 1 -
hw/scsi/vhost-scsi.c | 1 -
hw/scsi/vhost-user-scsi.c | 1 -
hw/scsi/virtio-scsi.c | 1 -
hw/scsi/vmw_pvscsi.c | 1 -
hw/sd/allwinner-sdhost.c | 1 -
hw/sd/aspeed_sdhci.c | 1 -
hw/sd/sd.c | 3 -
hw/sd/sdhci-pci.c | 1 -
hw/sd/sdhci.c | 1 -
hw/sparc/sun4m.c | 5 -
hw/sparc/sun4m_iommu.c | 1 -
hw/sparc64/sun4u.c | 7 -
hw/ssi/aspeed_smc.c | 2 -
hw/ssi/ibex_spi_host.c | 1 -
hw/ssi/npcm7xx_fiu.c | 1 -
hw/ssi/pnv_spi.c | 1 -
hw/ssi/sifive_spi.c | 1 -
hw/ssi/ssi.c | 1 -
hw/ssi/xilinx_spi.c | 1 -
hw/ssi/xilinx_spips.c | 2 -
hw/ssi/xlnx-versal-ospi.c | 1 -
hw/timer/a9gtimer.c | 1 -
hw/timer/allwinner-a10-pit.c | 1 -
hw/timer/arm_mptimer.c | 1 -
hw/timer/arm_timer.c | 1 -
hw/timer/aspeed_timer.c | 1 -
hw/timer/avr_timer16.c | 1 -
hw/timer/grlib_gptimer.c | 1 -
hw/timer/hpet.c | 1 -
hw/timer/i8254_common.c | 1 -
hw/timer/ibex_timer.c | 1 -
hw/timer/mss-timer.c | 1 -
hw/timer/nrf51_timer.c | 1 -
hw/timer/pxa2xx_timer.c | 1 -
hw/timer/renesas_cmt.c | 1 -
hw/timer/renesas_tmr.c | 1 -
hw/timer/sifive_pwm.c | 1 -
hw/timer/slavio_timer.c | 1 -
hw/timer/sse-timer.c | 1 -
hw/timer/stm32f2xx_timer.c | 1 -
hw/timer/xilinx_timer.c | 1 -
hw/tpm/tpm_crb.c | 1 -
hw/tpm/tpm_spapr.c | 1 -
hw/tpm/tpm_tis_i2c.c | 1 -
hw/tpm/tpm_tis_isa.c | 1 -
hw/tpm/tpm_tis_sysbus.c | 1 -
hw/tricore/tc27x_soc.c | 5 -
hw/tricore/tricore_testdevice.c | 5 -
hw/ufs/lu.c | 1 -
hw/ufs/ufs.c | 1 -
hw/usb/bus.c | 1 -
hw/usb/canokey.c | 1 -
hw/usb/ccid-card-emulated.c | 1 -
hw/usb/ccid-card-passthru.c | 1 -
hw/usb/dev-audio.c | 1 -
hw/usb/dev-hid.c | 3 -
hw/usb/dev-hub.c | 1 -
hw/usb/dev-mtp.c | 1 -
hw/usb/dev-network.c | 1 -
hw/usb/dev-serial.c | 2 -
hw/usb/dev-smartcard-reader.c | 2 -
hw/usb/dev-storage-classic.c | 1 -
hw/usb/dev-uas.c | 1 -
hw/usb/hcd-dwc2.c | 1 -
hw/usb/hcd-dwc3.c | 1 -
hw/usb/hcd-ehci-pci.c | 1 -
hw/usb/hcd-ehci-sysbus.c | 1 -
hw/usb/hcd-ohci-pci.c | 1 -
hw/usb/hcd-ohci-sysbus.c | 1 -
hw/usb/hcd-uhci.c | 2 -
hw/usb/hcd-xhci-nec.c | 1 -
hw/usb/hcd-xhci-sysbus.c | 1 -
hw/usb/hcd-xhci.c | 1 -
hw/usb/host-libusb.c | 1 -
hw/usb/redirect.c | 1 -
hw/usb/u2f-emulated.c | 1 -
hw/usb/u2f-passthru.c | 1 -
hw/vfio/ap.c | 1 -
hw/vfio/ccw.c | 1 -
hw/vfio/pci-quirks.c | 4 +-
hw/vfio/pci.c | 2 -
hw/vfio/platform.c | 1 -
hw/virtio/vdpa-dev-pci.c | 5 -
hw/virtio/vdpa-dev.c | 1 -
hw/virtio/vhost-scsi-pci.c | 1 -
hw/virtio/vhost-user-blk-pci.c | 1 -
hw/virtio/vhost-user-device.c | 1 -
hw/virtio/vhost-user-fs-pci.c | 1 -
hw/virtio/vhost-user-fs.c | 1 -
hw/virtio/vhost-user-gpio.c | 1 -
hw/virtio/vhost-user-i2c.c | 1 -
hw/virtio/vhost-user-input.c | 1 -
hw/virtio/vhost-user-rng-pci.c | 1 -
hw/virtio/vhost-user-rng.c | 1 -
hw/virtio/vhost-user-scmi.c | 1 -
hw/virtio/vhost-user-scsi-pci.c | 1 -
hw/virtio/vhost-user-snd-pci.c | 5 -
hw/virtio/vhost-user-snd.c | 1 -
hw/virtio/vhost-user-vsock-pci.c | 1 -
hw/virtio/vhost-user-vsock.c | 1 -
hw/virtio/vhost-vsock-common.c | 1 -
hw/virtio/vhost-vsock-pci.c | 1 -
hw/virtio/vhost-vsock.c | 1 -
hw/virtio/virtio-9p-pci.c | 1 -
hw/virtio/virtio-balloon.c | 1 -
hw/virtio/virtio-blk-pci.c | 1 -
hw/virtio/virtio-crypto-pci.c | 1 -
hw/virtio/virtio-crypto.c | 1 -
hw/virtio/virtio-input-pci.c | 1 -
hw/virtio/virtio-iommu-pci.c | 1 -
hw/virtio/virtio-iommu.c | 1 -
hw/virtio/virtio-mem.c | 1 -
hw/virtio/virtio-mmio.c | 1 -
hw/virtio/virtio-net-pci.c | 1 -
hw/virtio/virtio-nsm.c | 1 -
hw/virtio/virtio-pci.c | 2 -
hw/virtio/virtio-pmem.c | 1 -
hw/virtio/virtio-rng-pci.c | 1 -
hw/virtio/virtio-rng.c | 1 -
hw/virtio/virtio-scsi-pci.c | 1 -
hw/virtio/virtio-serial-pci.c | 1 -
hw/virtio/virtio.c | 1 -
hw/watchdog/sbsa_gwdt.c | 1 -
hw/watchdog/wdt_aspeed.c | 1 -
hw/watchdog/wdt_imx2.c | 1 -
hw/xen/xen-bus.c | 1 -
hw/xen/xen-legacy-backend.c | 17 -
hw/xen/xen_pt.c | 1 -
migration/migration.c | 3 +-
migration/options.c | 4 +-
stubs/iothread-lock.c | 8 +-
system/qdev-monitor.c | 15 +-
target/arm/cpu.c | 1 -
target/avr/cpu.c | 1 -
target/hexagon/cpu.c | 1 -
target/i386/cpu.c | 2 -
target/i386/kvm/kvm.c | 15 +
target/microblaze/cpu.c | 1 -
target/mips/cpu.c | 1 -
target/ppc/cpu_init.c | 6 -
target/riscv/cpu.c | 13 +-
target/s390x/cpu.c | 7 +-
target/sparc/cpu.c | 1 -
tests/unit/test-qdev-global-props.c | 1 -
rust/Cargo.toml | 1 +
rust/hw/char/pl011/src/device.rs | 154 ++++---
rust/hw/char/pl011/src/device_class.rs | 10 +-
rust/hw/char/pl011/src/lib.rs | 161 +++----
rust/qemu-api-macros/src/lib.rs | 2 +-
rust/qemu-api/meson.build | 8 +-
rust/qemu-api/src/callbacks.rs | 144 ++++++
rust/qemu-api/src/cell.rs | 26 +-
rust/qemu-api/src/definitions.rs | 168 -------
rust/qemu-api/src/lib.rs | 6 +-
rust/qemu-api/src/module.rs | 43 ++
rust/qemu-api/src/prelude.rs | 10 +-
rust/qemu-api/src/{device_class.rs => qdev.rs} | 67 ++-
rust/qemu-api/src/qom.rs | 584 +++++++++++++++++++++++++
rust/qemu-api/src/sysbus.rs | 20 +-
rust/qemu-api/src/vmstate.rs | 31 +-
rust/qemu-api/tests/tests.rs | 209 ++++++---
602 files changed, 1416 insertions(+), 1269 deletions(-)
create mode 100644 rust/qemu-api/src/callbacks.rs
delete mode 100644 rust/qemu-api/src/definitions.rs
create mode 100644 rust/qemu-api/src/module.rs
rename rust/qemu-api/src/{device_class.rs => qdev.rs} (72%)
create mode 100644 rust/qemu-api/src/qom.rs
--
2.47.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PULL 29/42] bql: add a "mock" BQL for Rust unit tests
2024-12-19 13:48 [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19 Paolo Bonzini
@ 2024-12-19 13:48 ` Paolo Bonzini
2024-12-19 13:48 ` [PULL 30/42] rust: tests: allow writing more than one test Paolo Bonzini
2024-12-19 18:27 ` [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19 Richard Henderson
2 siblings, 0 replies; 7+ messages in thread
From: Paolo Bonzini @ 2024-12-19 13:48 UTC (permalink / raw)
To: qemu-devel
Right now, the stub BQL in stubs/iothread-lock.c always reports itself as
unlocked. However, Rust would like to run its tests in an environment where
the BQL *is* locked. Provide an extremely dirty function that flips the
return value of bql_is_locked() to true.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
include/qemu/main-loop.h | 8 ++++++++
stubs/iothread-lock.c | 8 +++++++-
rust/qemu-api/meson.build | 2 +-
rust/qemu-api/src/cell.rs | 26 +++++++++++++++++++++++---
4 files changed, 39 insertions(+), 5 deletions(-)
diff --git a/include/qemu/main-loop.h b/include/qemu/main-loop.h
index 646306c272f..3935a573391 100644
--- a/include/qemu/main-loop.h
+++ b/include/qemu/main-loop.h
@@ -247,6 +247,14 @@ void event_notifier_set_handler(EventNotifier *e,
GSource *iohandler_get_g_source(void);
AioContext *iohandler_get_aio_context(void);
+/**
+ * rust_bql_mock_lock:
+ *
+ * Called from Rust doctests to make bql_lock() return true.
+ * Do not touch.
+ */
+void rust_bql_mock_lock(void);
+
/**
* bql_locked: Return lock status of the Big QEMU Lock (BQL)
*
diff --git a/stubs/iothread-lock.c b/stubs/iothread-lock.c
index 54676598950..6050c081f53 100644
--- a/stubs/iothread-lock.c
+++ b/stubs/iothread-lock.c
@@ -1,11 +1,17 @@
#include "qemu/osdep.h"
#include "qemu/main-loop.h"
+static bool bql_is_locked = false;
static uint32_t bql_unlock_blocked;
bool bql_locked(void)
{
- return false;
+ return bql_is_locked;
+}
+
+void rust_bql_mock_lock(void)
+{
+ bql_is_locked = true;
}
void bql_lock_impl(const char *file, int line)
diff --git a/rust/qemu-api/meson.build b/rust/qemu-api/meson.build
index 7ff408ad68e..50ec00e128d 100644
--- a/rust/qemu-api/meson.build
+++ b/rust/qemu-api/meson.build
@@ -60,7 +60,7 @@ test('rust-qemu-api-integration',
dependencies: [qemu_api, qemu_api_macros],
link_whole: [rust_qemu_api_objs, libqemuutil]),
args: [
- '--test',
+ '--test', '--test-threads', '1',
'--format', 'pretty',
],
protocol: 'rust',
diff --git a/rust/qemu-api/src/cell.rs b/rust/qemu-api/src/cell.rs
index 28349de291a..eae4e2ce786 100644
--- a/rust/qemu-api/src/cell.rs
+++ b/rust/qemu-api/src/cell.rs
@@ -124,9 +124,18 @@
use crate::bindings;
-// TODO: When building doctests do not include the actual BQL, because cargo
-// does not know how to link them to libqemuutil. This can be fixed by
-// running rustdoc from "meson test" instead of relying on cargo.
+/// An internal function that is used by doctests.
+pub fn bql_start_test() {
+ if cfg!(MESON) {
+ // SAFETY: integration tests are run with --test-threads=1, while
+ // unit tests and doctests are not multithreaded and do not have
+ // any BQL-protected data. Just set bql_locked to true.
+ unsafe {
+ bindings::rust_bql_mock_lock();
+ }
+ }
+}
+
pub fn bql_locked() -> bool {
// SAFETY: the function does nothing but return a thread-local bool
!cfg!(MESON) || unsafe { bindings::bql_locked() }
@@ -220,6 +229,7 @@ impl<T> BqlCell<T> {
///
/// ```
/// use qemu_api::cell::BqlCell;
+ /// # qemu_api::cell::bql_start_test();
///
/// let c = BqlCell::new(5);
/// ```
@@ -236,6 +246,7 @@ pub const fn new(value: T) -> BqlCell<T> {
///
/// ```
/// use qemu_api::cell::BqlCell;
+ /// # qemu_api::cell::bql_start_test();
///
/// let c = BqlCell::new(5);
///
@@ -253,6 +264,7 @@ pub fn set(&self, val: T) {
///
/// ```
/// use qemu_api::cell::BqlCell;
+ /// # qemu_api::cell::bql_start_test();
///
/// let cell = BqlCell::new(5);
/// assert_eq!(cell.get(), 5);
@@ -274,6 +286,7 @@ pub fn replace(&self, val: T) -> T {
///
/// ```
/// use qemu_api::cell::BqlCell;
+ /// # qemu_api::cell::bql_start_test();
///
/// let c = BqlCell::new(5);
/// let five = c.into_inner();
@@ -293,6 +306,7 @@ impl<T: Copy> BqlCell<T> {
///
/// ```
/// use qemu_api::cell::BqlCell;
+ /// # qemu_api::cell::bql_start_test();
///
/// let c = BqlCell::new(5);
///
@@ -315,6 +329,7 @@ impl<T> BqlCell<T> {
///
/// ```
/// use qemu_api::cell::BqlCell;
+ /// # qemu_api::cell::bql_start_test();
///
/// let c = BqlCell::new(5);
///
@@ -333,6 +348,7 @@ impl<T: Default> BqlCell<T> {
///
/// ```
/// use qemu_api::cell::BqlCell;
+ /// # qemu_api::cell::bql_start_test();
///
/// let c = BqlCell::new(5);
/// let five = c.take();
@@ -461,6 +477,7 @@ fn panic_already_borrowed(&self) -> ! {
///
/// ```
/// use qemu_api::cell::BqlRefCell;
+ /// # qemu_api::cell::bql_start_test();
///
/// let c = BqlRefCell::new(5);
///
@@ -472,6 +489,7 @@ fn panic_already_borrowed(&self) -> ! {
///
/// ```should_panic
/// use qemu_api::cell::BqlRefCell;
+ /// # qemu_api::cell::bql_start_test();
///
/// let c = BqlRefCell::new(5);
///
@@ -513,6 +531,7 @@ pub fn borrow(&self) -> BqlRef<'_, T> {
///
/// ```
/// use qemu_api::cell::BqlRefCell;
+ /// # qemu_api::cell::bql_start_test();
///
/// let c = BqlRefCell::new("hello".to_owned());
///
@@ -525,6 +544,7 @@ pub fn borrow(&self) -> BqlRef<'_, T> {
///
/// ```should_panic
/// use qemu_api::cell::BqlRefCell;
+ /// # qemu_api::cell::bql_start_test();
///
/// let c = BqlRefCell::new(5);
/// let m = c.borrow();
--
2.47.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PULL 30/42] rust: tests: allow writing more than one test
2024-12-19 13:48 [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19 Paolo Bonzini
2024-12-19 13:48 ` [PULL 29/42] bql: add a "mock" BQL for Rust unit tests Paolo Bonzini
@ 2024-12-19 13:48 ` Paolo Bonzini
2024-12-19 18:27 ` [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19 Richard Henderson
2 siblings, 0 replies; 7+ messages in thread
From: Paolo Bonzini @ 2024-12-19 13:48 UTC (permalink / raw)
To: qemu-devel
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
rust/qemu-api/tests/tests.rs | 111 ++++++++++++++++++++---------------
1 file changed, 63 insertions(+), 48 deletions(-)
diff --git a/rust/qemu-api/tests/tests.rs b/rust/qemu-api/tests/tests.rs
index 68557fb85c7..dc0ed190192 100644
--- a/rust/qemu-api/tests/tests.rs
+++ b/rust/qemu-api/tests/tests.rs
@@ -6,7 +6,9 @@
use qemu_api::{
bindings::*,
- c_str, declare_properties, define_property,
+ c_str,
+ cell::{self, BqlCell},
+ declare_properties, define_property,
prelude::*,
qdev::{DeviceImpl, DeviceState, Property},
qom::ObjectImpl,
@@ -14,55 +16,68 @@
zeroable::Zeroable,
};
+// Test that macros can compile.
+pub static VMSTATE: VMStateDescription = VMStateDescription {
+ name: c_str!("name").as_ptr(),
+ unmigratable: true,
+ ..Zeroable::ZERO
+};
+
+#[derive(qemu_api_macros::offsets)]
+#[repr(C)]
+#[derive(qemu_api_macros::Object)]
+pub struct DummyState {
+ parent: DeviceState,
+ migrate_clock: bool,
+}
+
+declare_properties! {
+ DUMMY_PROPERTIES,
+ define_property!(
+ c_str!("migrate-clk"),
+ DummyState,
+ migrate_clock,
+ unsafe { &qdev_prop_bool },
+ bool
+ ),
+}
+
+unsafe impl ObjectType for DummyState {
+ type Class = <DeviceState as ObjectType>::Class;
+ const TYPE_NAME: &'static CStr = c_str!("dummy");
+}
+
+impl ObjectImpl for DummyState {
+ type ParentType = DeviceState;
+ const ABSTRACT: bool = false;
+}
+
+impl DeviceImpl for DummyState {
+ fn properties() -> &'static [Property] {
+ &DUMMY_PROPERTIES
+ }
+ fn vmsd() -> Option<&'static VMStateDescription> {
+ Some(&VMSTATE)
+ }
+}
+
+fn init_qom() {
+ static ONCE: BqlCell<bool> = BqlCell::new(false);
+
+ cell::bql_start_test();
+ if !ONCE.get() {
+ unsafe {
+ module_call_init(module_init_type::MODULE_INIT_QOM);
+ }
+ ONCE.set(true);
+ }
+}
+
#[test]
-fn test_device_decl_macros() {
- // Test that macros can compile.
- pub static VMSTATE: VMStateDescription = VMStateDescription {
- name: c_str!("name").as_ptr(),
- unmigratable: true,
- ..Zeroable::ZERO
- };
-
- #[derive(qemu_api_macros::offsets)]
- #[repr(C)]
- #[derive(qemu_api_macros::Object)]
- pub struct DummyState {
- pub _parent: DeviceState,
- pub migrate_clock: bool,
- }
-
- declare_properties! {
- DUMMY_PROPERTIES,
- define_property!(
- c_str!("migrate-clk"),
- DummyState,
- migrate_clock,
- unsafe { &qdev_prop_bool },
- bool
- ),
- }
-
- unsafe impl ObjectType for DummyState {
- type Class = <DeviceState as ObjectType>::Class;
- const TYPE_NAME: &'static CStr = c_str!("dummy");
- }
-
- impl ObjectImpl for DummyState {
- type ParentType = DeviceState;
- const ABSTRACT: bool = false;
- }
-
- impl DeviceImpl for DummyState {
- fn properties() -> &'static [Property] {
- &DUMMY_PROPERTIES
- }
- fn vmsd() -> Option<&'static VMStateDescription> {
- Some(&VMSTATE)
- }
- }
-
+/// Create and immediately drop an instance.
+fn test_object_new() {
+ init_qom();
unsafe {
- module_call_init(module_init_type::MODULE_INIT_QOM);
object_unref(object_new(DummyState::TYPE_NAME.as_ptr()).cast());
}
}
--
2.47.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19
2024-12-19 13:48 [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19 Paolo Bonzini
2024-12-19 13:48 ` [PULL 29/42] bql: add a "mock" BQL for Rust unit tests Paolo Bonzini
2024-12-19 13:48 ` [PULL 30/42] rust: tests: allow writing more than one test Paolo Bonzini
@ 2024-12-19 18:27 ` Richard Henderson
2024-12-19 18:38 ` Paolo Bonzini
2 siblings, 1 reply; 7+ messages in thread
From: Richard Henderson @ 2024-12-19 18:27 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel
On 12/19/24 05:48, Paolo Bonzini wrote:
> The following changes since commit 8032c78e556cd0baec111740a6c636863f9bd7c8:
>
> Merge tag 'firmware-20241216-pull-request' of https://gitlab.com/kraxel/qemu into staging (2024-12-16 14:20:33 -0500)
>
> are available in the Git repository at:
>
> https://gitlab.com/bonzini/qemu.git tags/for-upstream
>
> for you to fetch changes up to 9c70b74bd8c7a3753b80a1aab22a0dc58da1ed68:
>
> rust: pl011: simplify handling of the FIFO enabled bit in LCR (2024-12-19 14:37:58 +0100)
>
> ----------------------------------------------------------------
> * qdev: second part of Property cleanups
> * rust: second part of QOM rework
> * rust: callbacks wrapper
> * rust: pl011 bugfixes
> * kvm: cleanup errors in kvm_convert_memory()
>
> ----------------------------------------------------------------
> v1->v2: I noticed that "meson test" returns success even if the tests actually failed
> with SIGSEGV! Sorry for not noticing that before. For now I added a bandaid
> in the form of "--test-threads 1" and replaced the Mutex<Cell<bool>> with a
> BqlCell<bool> for documentation purposes.
>
> Maciej S. Szmigiero (1):
> target/i386: Reset TSCs of parked vCPUs too on VM reset
>
> Paolo Bonzini (17):
> rust: qom: put class_init together from multiple ClassInitImpl<>
> rust: qom: add possibility of overriding unparent
> rust: rename qemu-api modules to follow C code a bit more
> rust: re-export C types from qemu-api submodules
> bql: add a "mock" BQL for Rust unit tests
> rust: tests: allow writing more than one test
> rust: qom: add casting functionality
> rust: qom: add initial subset of methods on Object
> rust: qemu-api: add a module to wrap functions and zero-sized closures
> kvm: consistently return 0/-errno from kvm_convert_memory
> rust: pl011: fix declaration of LineControl bits
> rust: pl011: match break logic of C version
> rust: pl011: always use reset() method on registers
> rust: pl011: fix break errors and definition of Data struct
> rust: pl011: extend registers to 32 bits
> rust: pl011: fix migration stream
> rust: pl011: simplify handling of the FIFO enabled bit in LCR
>
> Richard Henderson (24):
> migration: Constify migration_properties
> hw/ide: Constify sysbus_ahci_properties
> target/ppc: Remove empty property list
> target/s390x: Use s390x_cpu_properties for system mode only
> hw/pci-host/astro: Remove empty Property list
> hw/ppc: Only register spapr_nvdimm_properties if CONFIG_LIBPMEM
> hw/tricore: Remove empty Property lists
> hw/s390x: Remove empty Property lists
> hw/xen: Remove empty Property lists
> hw/sparc: Remove empty Property lists
> hw/virtio: Remove empty Property lists
> include/hw/qdev-core: Detect most empty Property lists at compile time
> hw/core: Introduce device_class_set_props_n
> migration: Use device_class_set_props_n
> hw/scsi/megasas: Use device_class_set_props_n
> hw/arm/armsse: Use device_class_set_props_n
> rust/qemu-api: Use device_class_set_props_n
> hw/core: Replace device_class_set_props with a macro
> target/riscv: Do not abuse DEFINE_PROP_END_OF_LIST
> include/hw/qdev-properties: Remove DEFINE_PROP_END_OF_LIST
> include/hw/qdev-properties: Shrink struct Property
> hw/core/qdev-properties: Constify Property argument to object_field_prop_ptr
> hw/core/qdev-properties: Constify Property argument to PropertyInfo.print
> Constify all opaque Property pointers
Please, really, replace my constify series with v2.
r~
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19
2024-12-19 18:27 ` [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19 Richard Henderson
@ 2024-12-19 18:38 ` Paolo Bonzini
2024-12-19 20:06 ` Stefan Hajnoczi
0 siblings, 1 reply; 7+ messages in thread
From: Paolo Bonzini @ 2024-12-19 18:38 UTC (permalink / raw)
To: Richard Henderson, qemu-devel
On 12/19/24 19:27, Richard Henderson wrote:
> Please, really, replace my constify series with v2.
Sure, sorry about that. I squashed this difference in from v1's final result:
diff --git b/include/hw/qdev-core.h a/include/hw/qdev-core.h
index c404bf55550..e6ef80b7fd0 100644
--- b/include/hw/qdev-core.h
+++ a/include/hw/qdev-core.h
@@ -947,8 +947,8 @@ char *qdev_get_own_fw_dev_path_from_handler(BusState *bus, DeviceState *dev);
* you attempt to add an existing property defined by a parent class.
* To modify an inherited property you need to use????
*
- * Validate that @props has at least one Property plus the terminator.
- * Validate that @props is an array via ARRAY_SIZE.
+ * Validate that @props has at least one Property.
+ * Validate that @props is an array, not a pointer, via ARRAY_SIZE.
* Validate that the array does not have a legacy terminator at compile-time;
* requires -O2 and the array to be const.
*/
and this one because otherwise clippy complains:
diff --git b/rust/qemu-api/src/device_class.rs a/rust/qemu-api/src/device_class.rs
index cca51fe1a98..c98f0b2c7da 100644
--- b/rust/qemu-api/src/device_class.rs
+++ a/rust/qemu-api/src/device_class.rs
@@ -87,7 +87,7 @@ fn vmsd() -> Option<&'static VMStateDescription> {
dc.vmsd = vmsd;
}
let prop = <T as DeviceImpl>::properties();
- if prop.len() != 0 {
+ if !prop.is_empty() {
bindings::device_class_set_props_n(dc, prop.as_ptr(), prop.len());
}
}
Paolo
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19
2024-12-19 18:38 ` Paolo Bonzini
@ 2024-12-19 20:06 ` Stefan Hajnoczi
2024-12-20 7:34 ` Paolo Bonzini
0 siblings, 1 reply; 7+ messages in thread
From: Stefan Hajnoczi @ 2024-12-19 20:06 UTC (permalink / raw)
To: Paolo Bonzini; +Cc: qemu-devel
[-- Attachment #1: Type: text/plain, Size: 140 bytes --]
Hi Paolo,
Will there be a v3? It wasn't clear to me from Richard and your replies
to v2 whether another revision is needed.
Thanks,
Stefan
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19
2024-12-19 20:06 ` Stefan Hajnoczi
@ 2024-12-20 7:34 ` Paolo Bonzini
0 siblings, 0 replies; 7+ messages in thread
From: Paolo Bonzini @ 2024-12-20 7:34 UTC (permalink / raw)
To: Stefan Hajnoczi; +Cc: qemu-devel
[-- Attachment #1: Type: text/plain, Size: 265 bytes --]
Il gio 19 dic 2024, 21:06 Stefan Hajnoczi <stefanha@redhat.com> ha scritto:
> Hi Paolo,
> Will there be a v3? It wasn't clear to me from Richard and your replies
> to v2 whether another revision is needed.
>
Yes, I will send it now.
Paolo
> Thanks,
> Stefan
>
[-- Attachment #2: Type: text/html, Size: 930 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2024-12-20 7:35 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-12-19 13:48 [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19 Paolo Bonzini
2024-12-19 13:48 ` [PULL 29/42] bql: add a "mock" BQL for Rust unit tests Paolo Bonzini
2024-12-19 13:48 ` [PULL 30/42] rust: tests: allow writing more than one test Paolo Bonzini
2024-12-19 18:27 ` [PULL v2 00/42] Rust, qdev, target/i386 changes for 2024-12-19 Richard Henderson
2024-12-19 18:38 ` Paolo Bonzini
2024-12-19 20:06 ` Stefan Hajnoczi
2024-12-20 7:34 ` Paolo Bonzini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.