[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv3] Support default block interfaces per QEMUMach
From: |
Christian Borntraeger |
Subject: |
Re: [Qemu-devel] [PATCHv3] Support default block interfaces per QEMUMachine |
Date: |
Wed, 21 Nov 2012 12:11:12 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:16.0) Gecko/20121028 Thunderbird/16.0.2 |
On 20/11/12 15:49, Alexander Graf wrote:
> On 11/20/2012 03:30 PM, Christian Borntraeger wrote:
>> There are QEMUMachines that have neither IF_IDE nor IF_SCSI as a
>> default/standard interface to their block devices / drives. Therefore,
>> this patch introduces a new field default_block_type per QEMUMachine
>> struct. The prior use_scsi field becomes thereby obsolete and is
>> replaced through .default_block_type = IF_SCSI.
>>
>> This patch also changes the default for s390x to IF_VIRTIO and
>> removes an early hack that converts IF_IDE drives.
>> Other parties have already claimed interest (e.g. IF_SD for exynos)
>>
>> To create a sane default, for machines that dont specify a
>> default_block_type, this patch makes IF_IDE = 0 and IF_NONE = 1.
>> I checked all users of IF_NONE (blockdev.c and ww/device-hotplug.c)
>> as well as IF_IDE and it seems that it is ok to change the defines -
>> in other words, I found no obvious (to me) assumption in the code
>> regarding IF_NONE==0. IF_NONE is only set if there is an
>> explicit if=none. Without if=* the interface becomes IF_DEFAULT.
>>
>> I would suggest to have some additional care, e.g. by letting
>> this patch sit some days in the block tree.
>>
>> Based on an initial patch from Einar Lueck<address@hidden>
>>
>> Signed-off-by: Christian Borntraeger<address@hidden>
>> CC: Igor Mitsyanko<address@hidden>
>> CC: Markus Armbruster<address@hidden>
>> CC: Kevin Wolf<address@hidden>
>
> Reviewed-by: Alexander Graf <address@hidden>
>
> However, I would consider this 1.4 material :).
Agreed.
Kevin, Stefan,
any chance to queue that up in your tree for 1.4?
Christian
- [Qemu-devel] [PATCHv3] Support default block interfaces per QEMUMachine, Christian Borntraeger, 2012/11/20
- Re: [Qemu-devel] [PATCHv3] Support default block interfaces per QEMUMachine, Alexander Graf, 2012/11/20
- Re: [Qemu-devel] [PATCHv3] Support default block interfaces per QEMUMachine,
Christian Borntraeger <=
- Re: [Qemu-devel] [PATCHv3] Support default block interfaces per QEMUMachine, Igor Mitsyanko, 2012/11/21
- Re: [Qemu-devel] [PATCHv3] Support default block interfaces per QEMUMachine, Markus Armbruster, 2012/11/22
- Re: [Qemu-devel] [PATCHv3] Support default block interfaces per QEMUMachine, Christian Borntraeger, 2012/11/22
- Re: [Qemu-devel] [PATCHv3] Support default block interfaces per QEMUMachine, Markus Armbruster, 2012/11/22
- [Qemu-devel] [PATCH 0/2] Followup patches regarding block default interface, Christian Borntraeger, 2012/11/22
- [Qemu-devel] [PATCH 1/2] block: simply default_drive, Christian Borntraeger, 2012/11/22
- [Qemu-devel] [PATCH 2/2] block: clarify comment about IF_IDE = 0, Christian Borntraeger, 2012/11/22
- Re: [Qemu-devel] [PATCH 0/2] Followup patches regarding block default interface, Stefan Hajnoczi, 2012/11/23
- Re: [Qemu-devel] [PATCH 0/2] Followup patches regarding block default interface, Markus Armbruster, 2012/11/23