[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 00/15] block: A bunch of fixes for Sheepdog and

From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH 00/15] block: A bunch of fixes for Sheepdog and Gluster
Date: Fri, 03 Mar 2017 06:39:05 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

Eric Blake <address@hidden> writes:

> On 03/02/2017 03:43 PM, Markus Armbruster wrote:
>> Bad error handling, memory leaks, and lack of blockdev-add support.
> How hard are we trying to get blockdev-add working in 2.9?  Or is this
> series 2.10 material now?

Definitely not 2.10: seven patches fix or document bugs, one improves
error messages, five are straightforward cleanups.  The series touches
only these two block drivers, including QAPI schema parts not used
anywhere else:

>>  block/gluster.c      | 127 +++++++--------
>>  block/sheepdog.c     | 436 
>> +++++++++++++++++++++++++++++++++++++--------------
>>  qapi-schema.json     |  38 +++++
>>  qapi/block-core.json |  73 +++------
>>  4 files changed, 443 insertions(+), 231 deletions(-)

With the pending pull requests merged, blockdev-add *is* working, except
for sheepdog.  I'm considering that a bug, and I need *two* patches to
fix it.  One touches only sheepdog.c, and the other only adds to the
QAPI schema.

If we decide not to fix the bug, I'd recommend to declare blockdev-add
supported in 2.9 anyway, with a release note that sheepdog support is

reply via email to

[Prev in Thread] Current Thread [Next in Thread]