[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390 |
Date: |
Thu, 3 May 2012 11:07:16 +0200 |
On 03.05.2012, at 11:05, Paolo Bonzini wrote:
>
>>> The usual old fix was to not even compile them in. Why are they in
>>> the alias list in the s390 build now?
>>
>> Because the alias list is in target-independent code.
>>
>> The old fix was brittle anyway, it dependent on the fact that
>> virtio-blk-pci was not part of libhw. A similar trick broke for
>> cirrus-vga when it became part of libhw.
>>
>> Christian fix is correct.
>
> Uhm, Christian fix would have the same problem actually if
> virtio-*-pci were to be moved in libhw. IIRC I proposed the
> same change on review and Anthony nacked it on these grounds.
> You could move the alias list to target-dependent code, though.
Can't we just make the virtio-*-pci variants fail instantiation and based on
that search the list on?
Alex
- [Qemu-devel] [PATCH] Fix legacy device aliases for s390, Christian Borntraeger, 2012/05/03
- Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390, Alexander Graf, 2012/05/03
- Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390, Christian Borntraeger, 2012/05/03
- Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390, Paolo Bonzini, 2012/05/03
- Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390, Paolo Bonzini, 2012/05/03
- Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390,
Alexander Graf <=
- Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390, Paolo Bonzini, 2012/05/03
- Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390, Anthony Liguori, 2012/05/03
- Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390, Alexander Graf, 2012/05/03
- Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390, Alexander Graf, 2012/05/17
- Re: [Qemu-devel] [PATCH] Fix legacy device aliases for s390, Anthony Liguori, 2012/05/17