From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43147) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vr6z2-00055O-K0 for qemu-devel@nongnu.org; Thu, 12 Dec 2013 09:11:25 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Vr6yx-0002gb-KV for qemu-devel@nongnu.org; Thu, 12 Dec 2013 09:11:20 -0500 Received: from mx1.redhat.com ([209.132.183.28]:63960) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vr6yx-0002g9-C3 for qemu-devel@nongnu.org; Thu, 12 Dec 2013 09:11:15 -0500 Date: Thu, 12 Dec 2013 09:11:11 -0500 From: Luiz Capitulino Message-ID: <20131212091111.4bd0e86c@redhat.com> In-Reply-To: <87lhzqo7mt.fsf@blackfin.pond.sub.org> References: <87ppp3bice.fsf@blackfin.pond.sub.org> <87lhzqo7mt.fsf@blackfin.pond.sub.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3 0/6] Add error_abort and associated cleanups List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster Cc: Igor Mammedov , Peter Crosthwaite , Paolo Bonzini , "qemu-devel@nongnu.org Developers" , Andreas =?UTF-8?B?RsOkcmJlcg==?= On Thu, 12 Dec 2013 08:23:06 +0100 Markus Armbruster wrote: > Peter Crosthwaite writes: > > > On Wed, Dec 11, 2013 at 11:57 PM, Markus Armbruster wrote: > >> Peter Crosthwaite writes: > >> > >>> Following our discussion RE self asserting API calls, here is a spin of > >>> my proposal. This series obsoletes the need for _nofail variants for > >>> Error ** accepting APIs. Is also greatly reduces the verbosity of calls > >>> sites that are currently asserting against errors. > >>> > >>> Patch 1 is the main event - addition of error_abort. The following > >>> patches then cleanup uses of _nofail and assert_no_error(). > >>> > >>> To give it a smoke test, I introduce a (critical) bug into QOM: > >> [...] > >> > >> Reviewed-by: Markus Armbruster > >> > > > > Thanks, > > > > Do you have a suggested patch queue or person this should go via? > > Luiz (cc'ed) has taken error work in the past. Luiz, what about this > series? I can take it (as it has at least one Reviewed-by already).