[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Xen-devel] xen_disk qdevification (was: [PATCH 0/3] Pe
From: |
Olaf Hering |
Subject: |
Re: [Qemu-block] [Xen-devel] xen_disk qdevification (was: [PATCH 0/3] Performance improvements for xen_disk v2) |
Date: |
Wed, 12 Dec 2018 09:59:51 +0100 |
User-agent: |
Mutt/1.11.1 (20181204T200311.382355a5) |
On Fri, Nov 02, Kevin Wolf wrote:
> A while ago, a downstream patch review found out that there are some QMP
> commands that would immediately crash if a xen_disk device were present
> because of the lacking qdevification. This is not the code quality
> standard I envision for QEMU. It's time for non-qdev devices to go.
Do you have that backwards by any chance? IMO the presence of assert()
contributes to bad code quality, not the drivers that trigger those
asserts. It is bad enough that two QEMU releases went out while being in
bad shape.
Anyway, hopefully Paul or whoever will find the time and energy to
convert the code at some point.
Olaf
signature.asc
Description: PGP signature
- Re: [Qemu-block] [Xen-devel] xen_disk qdevification (was: [PATCH 0/3] Performance improvements for xen_disk v2),
Olaf Hering <=