All of lore.kernel.org
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: Greg Kurz <groug@kaod.org>
Cc: "Kevin Wolf" <kwolf@redhat.com>,
	"Vladimir Sementsov-Ogievskiy" <vsementsov@virtuozzo.com>,
	qemu-block@nongnu.org, "Paul Durrant" <paul@xen.org>,
	"Laszlo Ersek" <lersek@redhat.com>,
	"Christian Schoenebeck" <qemu_oss@crudebyte.com>,
	"Michael Roth" <mdroth@linux.vnet.ibm.com>,
	qemu-devel@nongnu.org,
	"Stefano Stabellini" <sstabellini@kernel.org>,
	"Gerd Hoffmann" <kraxel@redhat.com>,
	"Stefan Hajnoczi" <stefanha@redhat.com>,
	"Anthony Perard" <anthony.perard@citrix.com>,
	xen-devel@lists.xenproject.org, "Max Reitz" <mreitz@redhat.com>,
	"Philippe Mathieu-Daudé" <philmd@redhat.com>,
	"Stefan Berger" <stefanb@linux.ibm.com>
Subject: Re: [PATCH v10 1/9] error: auto propagated local_err
Date: Wed, 24 Jun 2020 18:53:05 +0200	[thread overview]
Message-ID: <87k0zw8ky6.fsf@dusky.pond.sub.org> (raw)
In-Reply-To: <20200615083835.54e3fcb1@bahia.lan> (Greg Kurz's message of "Mon,  15 Jun 2020 08:38:35 +0200")

Greg Kurz <groug@kaod.org> writes:

> On Mon, 15 Jun 2020 07:21:03 +0200
> Markus Armbruster <armbru@redhat.com> wrote:
>
>> Greg Kurz <groug@kaod.org> writes:
>> 
>> > On Tue, 17 Mar 2020 18:16:17 +0300
>> > Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> wrote:
>> >
>> >> Introduce a new ERRP_AUTO_PROPAGATE macro, to be used at start of
>> >> functions with an errp OUT parameter.
>> >> 
>> >> It has three goals:
>> >> 
>> >> 1. Fix issue with error_fatal and error_prepend/error_append_hint: user
>> >> can't see this additional information, because exit() happens in
>> >> error_setg earlier than information is added. [Reported by Greg Kurz]
>> >> 
>> >
>> > I have more of these coming and I'd really like to use ERRP_AUTO_PROPAGATE.
>> >
>> > It seems we have a consensus on the macro itself but this series is gated
>> > by the conversion of the existing code base.
>> >
>> > What about merging this patch separately so that people can start using
>> > it at least ?
>> 
>> Please give me a few more days to finish the work I feel should go in
>> before the conversion.  With any luck, Vladimir can then rebase /
>> recreate the conversion easily, and you can finally use the macro for
>> your own work.
>> 
>
> Sure. Thanks.

Just posted "[PATCH 00/46] Less clumsy error checking".  The sheer size
of the thing and the length of its dependency chain explains why it took
me so long.  I feel bad about delaying you all the same.  Apologies!

I hope we can converge quickly enough to get Vladimir's work on top
ready in time for the soft freeze.



  reply	other threads:[~2020-06-24 16:53 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-17 15:16 [Xen-devel] [PATCH v10 0/9] error: auto propagated local_err part I Vladimir Sementsov-Ogievskiy
2020-03-17 15:16 ` Vladimir Sementsov-Ogievskiy
2020-03-17 15:16 ` [Xen-devel] [PATCH v10 1/9] error: auto propagated local_err Vladimir Sementsov-Ogievskiy
2020-03-17 15:16   ` Vladimir Sementsov-Ogievskiy
2020-06-10 14:39   ` Greg Kurz
2020-06-10 14:39     ` Greg Kurz
2020-06-15  5:21     ` Markus Armbruster
2020-06-15  6:38       ` Greg Kurz
2020-06-24 16:53         ` Markus Armbruster [this message]
2020-06-24 19:48           ` Greg Kurz
2020-03-17 15:16 ` [Xen-devel] [PATCH v10 2/9] scripts: Coccinelle script to use ERRP_AUTO_PROPAGATE() Vladimir Sementsov-Ogievskiy
2020-03-17 15:16   ` Vladimir Sementsov-Ogievskiy
2020-03-20 16:18   ` [Xen-devel] " Markus Armbruster
2020-03-20 16:18     ` Markus Armbruster
2020-03-17 15:16 ` [PATCH v10 3/9] SD (Secure Card): introduce ERRP_AUTO_PROPAGATE Vladimir Sementsov-Ogievskiy
2020-03-17 15:16 ` [PATCH v10 4/9] pflash: " Vladimir Sementsov-Ogievskiy
2020-03-17 15:16 ` [PATCH v10 5/9] fw_cfg: " Vladimir Sementsov-Ogievskiy
2020-03-17 15:16 ` [PATCH v10 6/9] virtio-9p: " Vladimir Sementsov-Ogievskiy
2020-03-17 15:16 ` [PATCH v10 7/9] TPM: " Vladimir Sementsov-Ogievskiy
2020-03-17 15:16 ` [PATCH v10 8/9] nbd: " Vladimir Sementsov-Ogievskiy
2020-03-17 15:16 ` [Xen-devel] [PATCH v10 9/9] xen: " Vladimir Sementsov-Ogievskiy
2020-03-17 15:16   ` Vladimir Sementsov-Ogievskiy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87k0zw8ky6.fsf@dusky.pond.sub.org \
    --to=armbru@redhat.com \
    --cc=anthony.perard@citrix.com \
    --cc=groug@kaod.org \
    --cc=kraxel@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=lersek@redhat.com \
    --cc=mdroth@linux.vnet.ibm.com \
    --cc=mreitz@redhat.com \
    --cc=paul@xen.org \
    --cc=philmd@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu_oss@crudebyte.com \
    --cc=sstabellini@kernel.org \
    --cc=stefanb@linux.ibm.com \
    --cc=stefanha@redhat.com \
    --cc=vsementsov@virtuozzo.com \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.