[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/5] Split ide-drive and scsi-disk qdevs, and
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/5] Split ide-drive and scsi-disk qdevs, and more |
Date: |
Thu, 12 May 2011 17:05:23 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Kevin Wolf <address@hidden> writes:
> Am 09.05.2011 11:51, 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, scsi-disk.
>>
>> 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.
>>
>> Remove the type hint from info block. Its value is unreliable anyway.
>> libvirt doesn't use it.
>>
>> I posted v1 quite some time ago. Since we were working towards a
>> release then, we decided to take only the bonus bug fixes (PATCH 1-3),
>> and revisit the rest later. Which has turned out to be "somewhat"
>> later than anticpiated. Sorry about that.
>
> Thanks, applied to the block branch.
I found a bug, v3 sent.
- [Qemu-devel] [PATCH v2 0/5] Split ide-drive and scsi-disk qdevs, and more, Markus Armbruster, 2011/05/09
- [Qemu-devel] [PATCH v2 3/5] block QMP: Drop query-block member "type" (type= in info block), Markus Armbruster, 2011/05/09
- [Qemu-devel] [PATCH v2 5/5] block: Remove type hint, it's guest matter, doesn't belong here, Markus Armbruster, 2011/05/09
- [Qemu-devel] [PATCH v2 4/5] blockdev: Store -drive option media in DriveInfo, Markus Armbruster, 2011/05/09
- [Qemu-devel] [PATCH v2 2/5] scsi: Split qdev "scsi-disk" into "scsi-hd" and "scsi-cd", Markus Armbruster, 2011/05/09
- [Qemu-devel] [PATCH v2 1/5] ide: Split qdev "ide-drive" into "ide-hd" and "ide-cd", Markus Armbruster, 2011/05/09
- Re: [Qemu-devel] [PATCH v2 0/5] Split ide-drive and scsi-disk qdevs, and more, Gerd Hoffmann, 2011/05/09
- Re: [Qemu-devel] [PATCH v2 0/5] Split ide-drive and scsi-disk qdevs, and more, Kevin Wolf, 2011/05/10
- Re: [Qemu-devel] [PATCH v2 0/5] Split ide-drive and scsi-disk qdevs, and more,
Markus Armbruster <=
- [Qemu-devel] [PATCH v2 0/5] Split ide-drive and scsi-disk qdevs, and more, Markus Armbruster, 2011/05/12
- [Qemu-devel] [PATCH v3 4/6] blockdev: Store -drive option media in DriveInfo, Markus Armbruster, 2011/05/12
- [Qemu-devel] [PATCH v3 5/6] block: Remove type hint, it's guest matter, doesn't belong here, Markus Armbruster, 2011/05/12
- [Qemu-devel] [PATCH v3 6/6] defaults: ide-cd and scsi-cd devices suppress default CD-ROM, Markus Armbruster, 2011/05/12
- [Qemu-devel] [PATCH v3 3/6] block QMP: Drop query-block member "type" (type= in info block), Markus Armbruster, 2011/05/12
- Re: [Qemu-devel] [PATCH v3 3/6] block QMP: Drop query-block member "type" (type= in info block), Luiz Capitulino, 2011/05/12
- Re: [Qemu-devel] [PATCH v3 3/6] block QMP: Drop query-block member "type" (type= in info block), Markus Armbruster, 2011/05/12
- Re: [Qemu-devel] [PATCH v3 3/6] block QMP: Drop query-block member "type" (type= in info block), Luiz Capitulino, 2011/05/12
- Re: [Qemu-devel] [PATCH v3 3/6] block QMP: Drop query-block member "type" (type= in info block), Markus Armbruster, 2011/05/12
- Re: [Qemu-devel] [PATCH v3 3/6] block QMP: Drop query-block member "type" (type= in info block), Luiz Capitulino, 2011/05/12