From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58891) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WGyjc-0000kP-KS for qemu-devel@nongnu.org; Fri, 21 Feb 2014 17:38:27 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WGyjX-0000wQ-JD for qemu-devel@nongnu.org; Fri, 21 Feb 2014 17:38:20 -0500 Received: from mx1.redhat.com ([209.132.183.28]:17997) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WGyjX-0000wG-Aa for qemu-devel@nongnu.org; Fri, 21 Feb 2014 17:38:15 -0500 Date: Fri, 21 Feb 2014 23:38:09 +0100 From: Kevin Wolf Message-ID: <20140221223809.GI3346@dhcp-200-207.str.redhat.com> References: <1393017681-12794-1-git-send-email-benoit.canet@irqsave.net> <1393017681-12794-7-git-send-email-benoit.canet@irqsave.net> <5307CEA6.5050003@redhat.com> <20140221223045.GB13076@irqsave.net> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <20140221223045.GB13076@irqsave.net> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH V19 06/12] quorum: Add quorum mechanism. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?iso-8859-1?Q?Beno=EEt?= Canet Cc: qemu-devel@nongnu.org, stefanha@redhat.com, mreitz@redhat.com Am 21.02.2014 um 23:30 hat Beno=EEt Canet geschrieben: > The Friday 21 Feb 2014 =E0 15:09:42 (-0700), Eric Blake wrote : > > On 02/21/2014 02:21 PM, Beno=EEt Canet wrote: > > > From: Beno=EEt Canet > > >=20 > > > This patchset enables the core of the quorum mechanism. > > > The num_children reads are compared to get the majority version and= if this > > > version exists more than threshold times the guest won't see the er= ror at all. > >=20 > > > +++ b/docs/qmp/qmp-events.txt > > > @@ -500,3 +500,39 @@ Example: > > > =20 > > > Note: If action is "reset", "shutdown", or "pause" the WATCHDOG ev= ent is > > > followed respectively by the RESET, SHUTDOWN, or STOP events. > > > + > > > +QUORUM_FAILURE > > > +-------------- > >=20 > > This should have been inserted in sorted order to match the rest of t= he > > file, rather than at the end. Can be rearranged in a followup patch, > > now that this series is queued. > >=20 > > > + > > > +Emitted by the Quorum block driver if it fails to establish a quor= um. > > > + > > > +Data: > > > + > > > +- "reference": device name if defined else node name. > > > +- "sector-num": Number of the first sector of the failed read op= eration. > >=20 > > I'm assuming sector-num is relative to the guest's view of the data? > Yes, To be more precise: It's the sector number specified by the layer above quorum. This can be the guest, but in a different configuration it could be another block driver. > > > + > > > +QUORUM_REPORT_BAD > > > +----------------- > > > + > > > +Emitted to report a corruption of a Quorum file. > > > + > > > +Data: > > > + > > > +- "ret": The IO return code. > >=20 > > What values is this likely to contain? Is it a finite set, in which > > case it would be nice to have a QAPI enum that describes the set of > > return codes, rather than a raw number? >=20 > It's anything that the block stack could return as an error. In other words, it's meaning depends on the host and the value is only suitable for human readers. Perhaps we could change this to strerror(), which is, I believe, the same as error_setg_errno() does. Kevin