[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions |
Date: |
Tue, 06 Oct 2015 10:45:31 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Peter Maydell <address@hidden> writes:
> On 6 October 2015 at 06:46, Markus Armbruster <address@hidden> wrote:
>> Peter Maydell <address@hidden> writes:
>>
>>> On 5 October 2015 at 20:27, Paolo Bonzini <address@hidden> wrote:
>>>>
>>>>
>>>> On 05/10/2015 20:46, Peter Maydell wrote:
>>>>> The easy fix is to have some suitable ifdeffery in virtio-pci.c,
>>>>> similar to how we only register the virtio_9p_pci and virtio_scsi_pci
>>>>
>>>> (vhost_scsi_pci)
>>>>
>>>>> types if they've been configured into this build.
>>>>
>>>> Hmm, actually there's no reason to limit
>>>>
>>>> common-obj-$(CONFIG_VIRTIO) += virtio-input.o
>>>> common-obj-$(CONFIG_VIRTIO) += virtio-input-hid.o
>>>>
>>>> to CONFIG_LINUX. Does it work to extract them out of the if (which is
>>>> only correct for virtio-input-host.o)?
>>>
>>> Yes, though as you predicted we then fall over on virtio-input-host-pci.
>>> If I bodge that one out in virtio-pci.c then the device-introspect-test
>>> passes.
>>
>> Does this do the trick? If yes, I'll fill out the commit message and
>> post it properly.
>
> This is what I tested, yes, but...
>
>> diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c
>> index eda8205..23f131b 100644
>> --- a/hw/virtio/virtio-pci.c
>> +++ b/hw/virtio/virtio-pci.c
>> @@ -2228,7 +2228,9 @@ static void virtio_pci_register_types(void)
>> type_register_static(&virtio_keyboard_pci_info);
>> type_register_static(&virtio_mouse_pci_info);
>> type_register_static(&virtio_tablet_pci_info);
>> +#ifdef CONFIG_LINUX
>> type_register_static(&virtio_host_pci_info);
>> +#endif
>> type_register_static(&virtio_pci_bus_info);
>> type_register_static(&virtio_pci_info);
>> #ifdef CONFIG_VIRTFS
>
> ...this will get you warnings about unused variables/data -- you
> need to #ifdef out the definition of virtio_host_pci_info and
> all the static functions it references too.
I'll finish the job and post for real. Thanks!
- [Qemu-devel] [PULL 03/10] macio: move DBDMA_init from instance_init to realize, (continued)
- [Qemu-devel] [PULL 03/10] macio: move DBDMA_init from instance_init to realize, Markus Armbruster, 2015/10/08
- Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions, Peter Maydell, 2015/10/08
- Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions, Markus Armbruster, 2015/10/08
- Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions, Peter Maydell, 2015/10/08
- Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions, Markus Armbruster, 2015/10/08
- Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions, Peter Maydell, 2015/10/08
- Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions, Paolo Bonzini, 2015/10/08
- Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions, Peter Maydell, 2015/10/08
- Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions, Markus Armbruster, 2015/10/08
- Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions, Peter Maydell, 2015/10/08
- Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions,
Markus Armbruster <=