From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56670) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XRGm6-0007jp-Sn for qemu-devel@nongnu.org; Tue, 09 Sep 2014 04:27:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XRGm0-00007D-Nl for qemu-devel@nongnu.org; Tue, 09 Sep 2014 04:27:42 -0400 Received: from mx1.redhat.com ([209.132.183.28]:28196) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XRGm0-000077-G5 for qemu-devel@nongnu.org; Tue, 09 Sep 2014 04:27:36 -0400 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s898RZwT004542 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Tue, 9 Sep 2014 04:27:35 -0400 Date: Tue, 9 Sep 2014 10:27:33 +0200 From: Kevin Wolf Message-ID: <20140909082733.GD4847@noname.str.redhat.com> References: <20140829160727.69f66ecd@redhat.com> <20140908104217.48f2354a@redhat.com> <20140908153318.GH4297@noname.redhat.com> <20140908125701.3be785e9@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140908125701.3be785e9@redhat.com> Subject: Re: [Qemu-devel] [PATCH] block: extend BLOCK_IO_ERROR event with nospace indicator List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Luiz Capitulino Cc: fromani@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com Am 08.09.2014 um 18:57 hat Luiz Capitulino geschrieben: > On Mon, 8 Sep 2014 17:33:18 +0200 > Kevin Wolf wrote: > > > Am 08.09.2014 um 16:42 hat Luiz Capitulino geschrieben: > > > On Fri, 29 Aug 2014 16:07:27 -0400 > > > Luiz Capitulino wrote: > > > > > > > Management software, such as RHEV's vdsm, want to be able to allocate > > > > disk space on demand. The basic use case is to start a VM with a small > > > > disk and then the disk is enlarged when QEMU hits a ENOSPC condition. > > > > > > > > To this end, the management software has to be notified when QEMU > > > > encounters ENOSPC. The solution implemented by this commit is simple: > > > > it extends the BLOCK_IO_ERROR with a 'nospace' key, which is true > > > > when QEMU is stopped due to ENOSPC. > > > > > > > > Note that support for querying this event is already present in > > > > query-block by means of the 'io-status' key. Also, the new 'nospace' > > > > BLOCK_IO_ERROR field shares the same semantics with 'io-status', > > > > which basically means that werror= has to be set to either > > > > 'stop' or 'enospc' to enable 'nospace'. > > > > > > > > Finally, this commit also updates the 'io-status' key doc in the > > > > schema with a list of supported device models. > > > > > > > > Signed-off-by: Luiz Capitulino > > > > > > Kevin, are you going to take this via block layer tree? > > > > Yes, thanks, I've applied it now. > > > > What was our conclusion wrt the human-readable strerror() string for > > debugging? Didn't we want to add that as well? > > I can do it on top of this patch. So, just adding a new field for this > is fine? I think so. Perhaps we should give it an 'x-' name to make clear that it's a debugging help and not supposed to be parsed by management tools. Or would that be abuse of that namespace? The alternative solution (or actually we could do both) would be to store it somewhere in bs and put it into query-block. Kevin