[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 3/6] hw: Default -drive to if=none instead of id
From: |
Artyom Tarasenko |
Subject: |
Re: [Qemu-block] [PATCH 3/6] hw: Default -drive to if=none instead of ide when ide cannot work |
Date: |
Mon, 23 Jan 2017 13:24:33 +0100 |
On Mon, Jan 23, 2017 at 10:48 AM, Markus Armbruster <address@hidden> wrote:
> Block backends defined with -drive if=ide are meant to be picked up by
> machine initialization code: a suitable frontend gets created and
> wired up automatically.
>
> if=ide drives not picked up that way can still be used with -device as
> if they had if=none, but that's unclean and best avoided. Unused ones
> produce an "Orphaned drive without device" warning.
>
> -drive parameter "if" is optional, and the default depends on the
> machine type. If a machine type doesn't specify a default, the
> default is "ide".
>
> Many machine types implicitly default to if=ide that way, even though
> they don't actually have an IDE controller. This makes no sense.
>
> Change the implicit default to if=none. Affected machines:
>
> * all targets: none
> * aarch64/arm: akita ast2500 canon cheetah collie connex imx25
> integratorcp kzm lm3s6965evb lm3s811evb mainstone musicpal n800 n810
> netduino2 nuri palmetto realview romulus sabrelite smdkc210 sx1 sx1
> verdex z2
> * cris: axis-dev88
> * i386/x86_64: xenpv
> * lm32: lm32-evr lm32-uclinux milkymist
> * m68k: an5206 dummy mcf5208evb
> * microblaze/microblazeel: petalogix-ml605 petalogix-s3adsp1800
> * mips/mips64/mips64el/mipsel: mipssim
> * moxie: moxiesim
> * or32: or32-sim
> * ppc/ppc64/ppcemb: bamboo ref405ep taihu virtex-ml507
> * ppc/ppc64: mpc8544ds ppce500
> * sh4/sh4eb: shix
> * sparc: leon3_generic
> * sparc64: niagara
The sparc64: niagara part,
Acked-By: Artyom Tarasenko <address@hidden>
> * tricore: tricore_testboard
> * unicore32: puv3
> * xtensa/xtensaeb: kc705 lx200 lx60 ml605 sim
>
> None of these machines have an IDE controller, let alone code to
> honor if=ide.
>
> Cc: Peter Maydell <address@hidden>
> Cc: address@hidden
> Cc: Edgar E. Iglesias <address@hidden>
> Cc: Stefano Stabellini <address@hidden>
> Cc: Anthony Perard <address@hidden>
> Cc: address@hidden
> Cc: Michael Walle <address@hidden>
> Cc: Laurent Vivier <address@hidden>
> Cc: Anthony Green <address@hidden>
> Cc: Jia Liu <address@hidden>
> Cc: Alexander Graf <address@hidden>
> Cc: address@hidden
> Cc: Magnus Damm <address@hidden>
> Cc: Fabien Chouteau <address@hidden>
> Cc: Mark Cave-Ayland <address@hidden>
> Cc: Artyom Tarasenko <address@hidden>
> Cc: Bastian Koppelmann <address@hidden>
> Cc: Guan Xuetao <address@hidden>
> Cc: Max Filippov <address@hidden>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
> include/sysemu/blockdev.h | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
> index 16432f3..351a039 100644
> --- a/include/sysemu/blockdev.h
> +++ b/include/sysemu/blockdev.h
> @@ -19,12 +19,11 @@ void blockdev_auto_del(BlockBackend *blk);
> typedef enum {
> IF_DEFAULT = -1, /* for use with drive_add() only */
> /*
> - * IF_IDE must be zero, because we want MachineClass member
> - * block_default_type to default-initialize to IF_IDE
> + * IF_NONE must be zero, because we want MachineClass member
> + * block_default_type to default-initialize to IF_NONE
> */
> - IF_IDE = 0,
> - IF_NONE,
> - IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD, IF_VIRTIO, IF_XEN,
> + IF_NONE = 0,
> + IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD, IF_VIRTIO, IF_XEN,
> IF_COUNT
> } BlockInterfaceType;
>
> --
> 2.7.4
>
--
Regards,
Artyom Tarasenko
SPARC and PPC PReP under qemu blog: http://tyom.blogspot.com/search/label/qemu
- Re: [Qemu-block] [Qemu-devel] [PATCH 6/6] hw/i386/i386: Stop auto-creating lsi53c895a SCSI HBAs, (continued)
- Re: [Qemu-block] [Qemu-devel] [PATCH 6/6] hw/i386/i386: Stop auto-creating lsi53c895a SCSI HBAs, Markus Armbruster, 2017/01/24
- Re: [Qemu-block] [Qemu-devel] [PATCH 6/6] hw/i386/i386: Stop auto-creating lsi53c895a SCSI HBAs, Peter Maydell, 2017/01/24
- Re: [Qemu-block] [Qemu-devel] [PATCH 6/6] hw/i386/i386: Stop auto-creating lsi53c895a SCSI HBAs, Paolo Bonzini, 2017/01/24
- Re: [Qemu-block] [Qemu-devel] [PATCH 6/6] hw/i386/i386: Stop auto-creating lsi53c895a SCSI HBAs, Peter Maydell, 2017/01/24
- Re: [Qemu-block] [Qemu-devel] [PATCH 6/6] hw/i386/i386: Stop auto-creating lsi53c895a SCSI HBAs, Markus Armbruster, 2017/01/24
- Re: [Qemu-block] [Qemu-devel] [PATCH 6/6] hw/i386/i386: Stop auto-creating lsi53c895a SCSI HBAs, Markus Armbruster, 2017/01/25
[Qemu-block] [PATCH 5/6] hw/arm/highbank: Default -drive to if=ide instead of if=scsi, Markus Armbruster, 2017/01/23
[Qemu-block] [PATCH 2/6] hw/arm/cubieboard hw/arm/xlnx-ep108: Fix units_per_default_bus, Markus Armbruster, 2017/01/23
[Qemu-block] [PATCH 4/6] hw: Default -drive to if=none instead of scsi when scsi cannot work, Markus Armbruster, 2017/01/23
[Qemu-block] [PATCH 3/6] hw: Default -drive to if=none instead of ide when ide cannot work, Markus Armbruster, 2017/01/23
- Re: [Qemu-block] [PATCH 3/6] hw: Default -drive to if=none instead of ide when ide cannot work,
Artyom Tarasenko <=