[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 4/6] block QMP: Deprecate query-block's "type
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v4 4/6] block QMP: Deprecate query-block's "type", drop info block's "type=" |
Date: |
Wed, 18 May 2011 14:44:11 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.15) Gecko/20101027 Fedora/3.0.10-1.fc12 Thunderbird/3.0.10 |
Am 16.05.2011 15:04, schrieb Markus Armbruster:
> query-block's specification documents response member "type" with
> values "hd", "cdrom", "floppy", "unknown".
>
> Its value is unreliable: a block device used as floppy has type
> "floppy" if created with if=floppy, but type "hd" if created with
> if=none.
>
> That's because with if=none, the type is at best a declaration of
> intent: the drive can be connected to any guest device. Its type is
> really the guest device's business. Reporting it here is wrong.
>
> No known user of QMP uses "type". It's unlikely that any unknown
> users exist, because its value is useless unless you know how the
> block device was created. But then you also know the true value.
>
> Fixing the broken value risks breaking (hypothetical!) clients that
> somehow rely on the current behavior. Not fixing the value risks
> breaking (hypothetical!) clients that rely on the value to be
> accurate. Can't entirely avoid hypothetical lossage. Change the
> value to be always "unknown".
>
> This makes "info block" always report "type=unknown". Pointless.
> Change it to not report the type.
>
> Signed-off-by: Markus Armbruster <address@hidden>
Luiz/Anthony, I'm not sure if Markus asked you, but I'm waiting for an
Ack from one of you here before merging this series. Just in case
someone wonders why nothing has happened yet.
Kevin
- [Qemu-devel] [PATCH v4 0/6] Split ide-drive and scsi-disk qdevs, and more, Markus Armbruster, 2011/05/16
- [Qemu-devel] [PATCH v4 6/6] block: Remove type hint, it's guest matter, doesn't belong here, Markus Armbruster, 2011/05/16
- [Qemu-devel] [PATCH v4 3/6] defaults: ide-cd and scsi-cd devices suppress default CD-ROM, Markus Armbruster, 2011/05/16
- [Qemu-devel] [PATCH v4 4/6] block QMP: Deprecate query-block's "type", drop info block's "type=", Markus Armbruster, 2011/05/16
- Re: [Qemu-devel] [PATCH v4 4/6] block QMP: Deprecate query-block's "type", drop info block's "type=",
Kevin Wolf <=
- [Qemu-devel] [PATCH v4 5/6] blockdev: Store -drive option media in DriveInfo, Markus Armbruster, 2011/05/16
- [Qemu-devel] [PATCH v4 1/6] ide: Split qdev "ide-drive" into "ide-hd" and "ide-cd", Markus Armbruster, 2011/05/16
- [Qemu-devel] [PATCH v4 2/6] scsi: Split qdev "scsi-disk" into "scsi-hd" and "scsi-cd", Markus Armbruster, 2011/05/16