On 16/03/2019 23.39, Philippe Mathieu-Daudé wrote:
> This fixes when configuring with --without-default-devices:
> 
>   $ qemu-system-mips64 -M malta --nodefaults -bios /dev/null
>   qemu-system-mips64: Unknown device 'piix4-usb-uhci' for bus 'PCI'
>   Aborted (core dumped)
> 
>   (gdb) bt
>   #0  0x00007ffff5a4353f in __GI_raise (sig=sig@entry=6) at 
> ../sysdeps/unix/sysv/linux/raise.c:50
>   #1  0x00007ffff5a2d895 in __GI_abort () at abort.c:79
>   #2  0x00005555558745c3 in qdev_create (bus=bus@entry=0x555556336260, 
> name=name@entry=0x555555b13a4d "piix4-usb-uhci") at hw/core/qdev.c:131
>   #3  0x00005555558cb3e1 in pci_create_multifunction 
> (bus=bus@entry=0x555556336260, devfn=devfn@entry=82, 
> multifunction=multifunction@entry=false, name=name@entry=0x555555b13a4d 
> "piix4-usb-uhci") at hw/pci/pci.c:2104
>   #4  0x00005555558cb88b in pci_create_simple_multifunction 
> (name=name@entry=0x555555b13a4d "piix4-usb-uhci", multifunction=false, 
> devfn=devfn@entry=82, bus=bus@entry=0x555556336260) at hw/pci/pci.c:2126
>   #5  0x00005555558cb88b in pci_create_simple (bus=bus@entry=0x555556336260, 
> devfn=devfn@entry=82, name=name@entry=0x555555b13a4d "piix4-usb-uhci") at 
> hw/pci/pci.c:2126
>   #6  0x000055555575ea38 in mips_malta_init (machine=0x555555ffe430) at 
> hw/mips/mips_malta.c:1392
>   #7  0x000055555587bf2b in machine_run_board_init (machine=0x555555ffe430) 
> at hw/core/machine.c:1030
> 
> Fixes: 7c28b925b7e
> Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com>
> ---
>  hw/isa/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/hw/isa/Kconfig b/hw/isa/Kconfig
> index 57e09a0cb8..c942b47d03 100644
> --- a/hw/isa/Kconfig
> +++ b/hw/isa/Kconfig
> @@ -29,6 +29,7 @@ config PIIX4
>      # For historical reasons, SuperIO devices are created in the board
>      # for PIIX4.
>      select ISA_BUS
> +    select USB_UHCI

The UHCI controller is not instantiated by hw/isa/piix4.c, but by
hw/mips/mips_malta.c as far as I can see. And you can use the "pc"
machine also without uhci. So I think this patch is wrong, the select
should be done by "MALTA" instead.

 Thomas

Reply via email to