[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 10:55:57 +0200 |
On 03.05.2012, at 10:53, Christian Borntraeger wrote:
> On 03/05/12 10:49, Alexander Graf wrote:
>>
>> On 03.05.2012, at 10:47, Christian Borntraeger wrote:
>>
>>> When qemu is called with -device virtio-serial/blk/net on s390, this alias
>>> is translated to virtio-serial/blk/net-pci instead of s390, since these
>>> drivers are first in the alias table.
>>> Let the core code check if the driver exist, if not lets search further.
>>> This fixes errors like:
>>>
>>> qemu-kvm: -device virtio-serial,id=virtio-serial0: Parameter 'driver'
>>> expects device type
>>>
>>> Signed-off-by: Christian Borntraeger <address@hidden>
>>
>> The usual old fix was to not even compile them in. Why are they in the alias
>> list in the s390 build now?
>
> Huh? The aliases always worked for me when I compiled qemu myself for s390.
> This changed with
>
> commit 6acbe4c6f18e7de00481ff30574262b58526de45
> Author: Anthony Liguori <address@hidden>
> Date: Thu Dec 22 11:05:00 2011 -0600
>
> qdev: remove baked in notion of aliases (v2)
>
> In other word v1.0 is fine, master is not. This looks like a regression, no?
Yeah. According to your code however, we have the "virtio-blk-pci" and
"virtio-blk-s390" modules both in the alias list, regardless of the
architecture. The reason it worked before was that the alias list was
specifically built based on qdev devices for each target. Apparently that's
changed now.
Anthony, was this change on purpose? If so, Christian's patch is correct and
should be applied.
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,
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, Paolo Bonzini, 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, 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