[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 08/19] hw/i386: turn off vmport if CONFIG_VMPORT
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PULL 08/19] hw/i386: turn off vmport if CONFIG_VMPORT is disabled |
Date: |
Tue, 16 Jul 2019 20:55:25 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
Hi,
On 7/16/19 10:11 AM, Paolo Bonzini wrote:
> From: Julio Montes <address@hidden>
>
> vmport device is not included when CONFIG_VMPORT is disabled, hence
> QEMU fails with the following error:
>
> `Unknown device 'vmport' for bus 'ISA': unknown.`
>
> v2: imply VMPORT (Paolo Bonzini )
>
> Signed-off-by: Julio Montes <address@hidden>
> Message-Id: <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> hw/i386/Kconfig | 4 ++--
> hw/i386/pc.c | 5 +++++
> 2 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/hw/i386/Kconfig b/hw/i386/Kconfig
> index 4ddf2a9..b9c96ac 100644
> --- a/hw/i386/Kconfig
> +++ b/hw/i386/Kconfig
> @@ -51,6 +51,7 @@ config PC_ACPI
> config I440FX
> bool
> imply E1000_PCI
> + imply VMPORT
> select PC_PCI
> select PC_ACPI
> select ACPI_SMBUS
> @@ -58,7 +59,6 @@ config I440FX
> select IDE_PIIX
> select DIMM
> select SMBIOS
> - select VMPORT
> select VMMOUSE
> select FW_CFG_DMA
>
> @@ -77,6 +77,7 @@ config Q35
> imply VTD
> imply AMD_IOMMU
> imply E1000E_PCI_EXPRESS
> + imply VMPORT
> select PC_PCI
> select PC_ACPI
> select PCI_EXPRESS_Q35
> @@ -84,7 +85,6 @@ config Q35
> select AHCI_ICH9
> select DIMM
> select SMBIOS
> - select VMPORT
> select VMMOUSE
> select FW_CFG_DMA
>
> diff --git a/hw/i386/pc.c b/hw/i386/pc.c
> index c33ce47..549c437 100644
> --- a/hw/i386/pc.c
> +++ b/hw/i386/pc.c
> @@ -83,6 +83,7 @@
> #include "hw/mem/memory-device.h"
> #include "sysemu/replay.h"
> #include "qapi/qmp/qerror.h"
> +#include "config-devices.h"
>
> /* debug PC/ISA interrupts */
> //#define DEBUG_IRQ
> @@ -2793,7 +2794,11 @@ static void pc_machine_initfn(Object *obj)
>
> pcms->max_ram_below_4g = 0; /* use default */
> pcms->smm = ON_OFF_AUTO_AUTO;
> +#ifdef CONFIG_VMPORT
> pcms->vmport = ON_OFF_AUTO_AUTO;
> +#else
> + pcms->vmport = ON_OFF_AUTO_OFF;
> +#endif /* CONFIG_VMPORT */
> /* acpi build is enabled by default if machine supports it */
> pcms->acpi_build_enabled = PC_MACHINE_GET_CLASS(pcms)->has_acpi_build;
> pcms->smbus_enabled = true;
>
Testing v4.1.0-rc1, this commit breaks ./configure
--without-default-devices:
GEN i386-softmmu/config-devices.mak.tmp
select VMMOUSE if I440FX
select VMMOUSE if Q35
config VMMOUSE depends on VMPORT
Traceback (most recent call last):
File "scripts/minikconf.py", line 703, in <module>
config = data.compute_config()
File "scripts/minikconf.py", line 253, in compute_config
clause.process()
File "scripts/minikconf.py", line 188, in process
self.dest.set_value(False, self)
File "scripts/minikconf.py", line 118, in set_value
raise KconfigDataError('contradiction between clauses when setting
%s' % self)
__main__.KconfigDataError: contradiction between clauses when setting
VMMOUSE
97fd1ea8c1065839ae4c19bfdc3590033a5b9788 is the first bad commit
commit 97fd1ea8c1065839ae4c19bfdc3590033a5b9788
Author: Julio Montes <address@hidden>
Date: Fri Jul 12 16:02:57 2019 +0000
hw/i386: turn off vmport if CONFIG_VMPORT is disabled
vmport device is not included when CONFIG_VMPORT is disabled, hence
QEMU fails with the following error:
`Unknown device 'vmport' for bus 'ISA': unknown.`
v2: imply VMPORT (Paolo Bonzini )
Signed-off-by: Julio Montes <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>
- [Qemu-devel] [PULL 00/19] Bugfix/cleanup patches for 2019-07-16, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 01/19] scsi-disk: pass sense correctly for guest-recoverable errors, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 02/19] scsi: explicitly list guest-recoverable sense codes, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 03/19] scsi: add guest-recoverable ZBC errors, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 04/19] iscsi: fix busy/timeout/task set full, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 05/19] iscsi: base all handling of check condition on scsi_sense_to_errno, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 06/19] build-sys: remove slirp cflags from main-loop.o, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 07/19] rdmacm-mux: fix strcpy string warning, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 08/19] hw/i386: turn off vmport if CONFIG_VMPORT is disabled, Paolo Bonzini, 2019/07/16
- Re: [Qemu-devel] [PULL 08/19] hw/i386: turn off vmport if CONFIG_VMPORT is disabled,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PULL 09/19] memory: unref the memory region in simplify flatview, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 10/19] Fix broken build with WHPX enabled, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 11/19] util: merge main-loop.c and iohandler.c, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 12/19] hw/lm32/Kconfig: Milkymist One provides a USB 1.1 Controller, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 14/19] target/i386: sev: Do not unpin ram device memory region, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 15/19] hw/usb/Kconfig: Add CONFIG_USB_EHCI_PCI, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 16/19] hw/usb/Kconfig: USB_XHCI_NEC requires USB_XHCI, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 13/19] checkpatch: detect doubly-encoded UTF-8, Paolo Bonzini, 2019/07/16
- [Qemu-devel] [PULL 17/19] Makefile: do not repeat $(CONFIG_SOFTMMU) in hw/Makefile.objs, Paolo Bonzini, 2019/07/16