[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] Support default interface types per QEMUMac
From: |
Christian Borntraeger |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] Support default interface types per QEMUMachine |
Date: |
Thu, 26 Apr 2012 21:00:01 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:11.0) Gecko/20120412 Thunderbird/11.0.1 |
On 26/04/12 19:11, Peter Maydell wrote:
> On 26 April 2012 18:09, Alexander Graf <address@hidden> wrote:
>> On 26.04.2012, at 18:17, Peter Maydell wrote:
>>> Can you describe what the problem is that you're trying to solve
>>> here? It's not entirely obvious from this commit message.
>>
>> On s390, there is no ide or scsi, so qemu-system-s390x <file> currently
>> uses a dirty hack that loops through IF_IDE and creates virtio devices.
>> I'd assume this patch tried to get rid of that.
>
> What's wrong with just having it be an error if the user
> creates IDE drives for a machine with no IDE interfaces?
For example the fact that drive file=xxx works on x86 but fails on s390 if
I remove the hack in s390_init ;-)
To me that looks more incosistent than having the right default interface even
if "right" changes across hardware.
If you have a second look, we already have different interfaces for different
systems.
e.g. sparc defaults to scsi, pc to ide.
So this patch will allow that s390 chooses IF_VIRTIO and exynos can choose
IF_SD.
Christian
- [Qemu-devel] [PATCH 1/1] Support default interface types per QEMUMachine, (continued)
Re: [Qemu-devel] [PATCH 1/1] Support default interface types per QEMUMachine, Peter Maydell, 2012/04/26
Re: [Qemu-devel] [PATCH 1/1] Support default interface types per QEMUMachine, Igor Mitsyanko, 2012/04/26
Re: [Qemu-devel] [PATCH 1/1] Support default interface types per QEMUMachine, Peter Maydell, 2012/04/26
Re: [Qemu-devel] [PATCH 1/1] Support default interface types per QEMUMachine, Igor Mitsyanko, 2012/04/26
Re: [Qemu-devel] [PATCH 1/1] Support default interface types per QEMUMachine, Peter Maydell, 2012/04/26
Re: [Qemu-devel] [PATCH 1/1] Support default interface types per QEMUMachine, Anthony Liguori, 2012/04/26