[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 0/8] Split ide-drive and scsi-disk qdevs, and mo
From: |
Kevin Wolf |
Subject: |
[Qemu-devel] Re: [PATCH 0/8] Split ide-drive and scsi-disk qdevs, and more |
Date: |
Mon, 12 Jul 2010 11:52:11 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100430 Fedora/3.0.4-2.fc12 Thunderbird/3.0.4 |
Am 06.07.2010 14:37, schrieb Markus Armbruster:
> This patch series is about purging the "type hint" from the block
> layer. My previous series cleaned up improper uses it. Remaining
> uses are info block and qdevs ide-drive, scsidisk.
>
> Remove the type hint from info block. Its value is unreliable anyway.
>
> ide-drive and scsi-disk can either act as disk or as CD drive. They
> use their drive's type hint to decide between disk and CD. This is
> unclean. Disk vs. CD needs to be in qdev, not BlockDriverState,
> because it belongs to the drive's guest part.
>
> Split them into separate devices for disk and CD. Keep the old ones
> for backward compatibility.
>
> Bonus fix: reject empty drives unless media is removable (1-3/8).
>
> This patch series is available at
> git://repo.or.cz/qemu/armbru.git
> tag block-qdev-split: this series, based on
> tag block-fixes-2-v2: my previous series, based on
> tag blockdev-base, which the current kevin/block
>
> Markus Armbruster (8):
> virtio-pci: Check for virtio_blk_init() failure
> virtio-blk: Fix virtio-blk-s390 to require drive
> ide scsi virtio-blk: Reject empty drives unless media is removable
Thanks, applied patches 1-3 to the block branch.
> block QMP: Drop query-block member "type" (type= in info block)
> ide: Split qdev "ide-drive" into "ide-hd" and "ide-cd"
> scsi: Split qdev "scsi-disk" into "scsi-hd" and "scsi-cd"
> blockdev: Store -drive option media in DriveInfo
> block: Remove type hint
As discussed on IRC last week I'll wait for a respin for the remaining ones.
Kevin