[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated clean
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups |
Date: |
Tue, 3 Dec 2013 12:49:06 +0100 |
On Tue, 03 Dec 2013 10:44:57 +0100
Markus Armbruster <address@hidden> wrote:
> Peter Crosthwaite <address@hidden> 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 greately 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:
> [...]
> > 32 files changed, 100 insertions(+), 143 deletions(-)
>
> I like it. Nice diffstat, too.
>
> There are some _nofail functions left, but none of them can use
> error_abort.
>
> If anything assigns to error_abort, we're probably screwed. No bright
> idea how to prevent that at compile-time.
Maybe
Error * const error_abort;
- Re: [Qemu-devel] [RFC PATCH v1 2/5] hw: Remove assert_no_error usages, (continued)
- [Qemu-devel] [RFC PATCH v1 5/5] qerror: Remove assert_no_error(), Peter Crosthwaite, 2013/12/03
- Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups, Markus Armbruster, 2013/12/03
- Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups,
Igor Mammedov <=
- Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups, Paolo Bonzini, 2013/12/03
- Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups, Eric Blake, 2013/12/03
- Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups, Markus Armbruster, 2013/12/03
- Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups, Igor Mammedov, 2013/12/03
- Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups, Eric Blake, 2013/12/03
- Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups, Markus Armbruster, 2013/12/04
- Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups, Eric Blake, 2013/12/04
- Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups, Paolo Bonzini, 2013/12/05
- Re: [Qemu-devel] [RFC PATCH v1 0/5] Add error_abort and associated cleanups, Igor Mammedov, 2013/12/05