qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: John Snow <jsnow@redhat.com>
To: Kevin Wolf <kwolf@redhat.com>
Cc: pkrempa@redhat.com, jtc@redhat.com, qemu-devel@nongnu.org,
	qemu-block@nongnu.org
Subject: Re: [Qemu-devel] [RFC v4 18/21] blockjobs: add block-job-finalize
Date: Wed, 28 Feb 2018 14:14:54 -0500	[thread overview]
Message-ID: <24618453-fd43-de12-1e14-2fc9ed2c4b8b@redhat.com> (raw)
In-Reply-To: <20180228181553.GL4855@localhost.localdomain>



On 02/28/2018 01:15 PM, Kevin Wolf wrote:
> Is it because you want to avoid that the user picks an automatic job for
> completing the mixed transaction?

I wanted to avoid the case that a job without the manual property would
be stuck "pending" -- which I have defined to mean that it is waiting on
an authorization from the user -- which isn't really true: it's waiting
on its peers in the transaction to receive that authorization.

It would indeed be simpler to just let them stick around in PENDING like
their peers, but it felt like a hacky way to allow mixed-mode
transactions -- like they had been promoted for just a subset of their
lifetime.

So, mostly it was a semantic decision and not a functional one based on
what I considered "WAITING" to mean vs "PENDING".

If the definitions (and documentation) are adjusted it can be changed
for the simpler layout if it seems just as good from the API
perspective. (It's certainly better implementationally, as you say.)

--js

  reply	other threads:[~2018-02-28 19:15 UTC|newest]

Thread overview: 98+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-23 23:51 [Qemu-devel] [RFC v4 00/21] blockjobs: add explicit job management John Snow
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 01/21] blockjobs: fix set-speed kick John Snow
2018-02-27 16:32   ` Eric Blake
2018-02-27 17:18   ` Kevin Wolf
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 02/21] blockjobs: model single jobs as transactions John Snow
2018-02-27 16:36   ` Eric Blake
2018-02-27 17:18   ` Kevin Wolf
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 03/21] blockjobs: add manual property John Snow
2018-02-27 16:39   ` Eric Blake
2018-02-27 17:18   ` Kevin Wolf
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 04/21] blockjobs: add status enum John Snow
2018-02-27 16:44   ` Eric Blake
2018-02-27 17:19   ` Kevin Wolf
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 05/21] blockjobs: add state transition table John Snow
2018-02-27 16:27   ` Kevin Wolf
2018-02-27 16:45     ` John Snow
2018-02-27 17:08       ` Kevin Wolf
2018-02-27 18:58         ` John Snow
2018-02-27 18:58   ` Eric Blake
2018-02-27 19:06     ` John Snow
2018-02-27 19:22     ` John Snow
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 06/21] iotests: add pause_wait John Snow
2018-02-27 17:19   ` Kevin Wolf
2018-02-27 19:01   ` Eric Blake
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 07/21] blockjobs: add block_job_verb permission table John Snow
2018-02-27 17:19   ` Kevin Wolf
2018-02-27 19:25   ` Eric Blake
2018-02-27 19:38     ` John Snow
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 08/21] blockjobs: add ABORTING state John Snow
2018-02-27 19:34   ` Eric Blake
2018-02-28 14:54   ` Kevin Wolf
2018-02-28 19:26     ` John Snow
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 09/21] blockjobs: add CONCLUDED state John Snow
2018-02-27 19:38   ` Eric Blake
2018-02-27 19:44     ` John Snow
2018-02-28 15:37   ` Kevin Wolf
2018-02-28 19:29     ` John Snow
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 10/21] blockjobs: add NULL state John Snow
2018-02-27 19:41   ` Eric Blake
2018-02-28 15:42   ` Kevin Wolf
2018-02-28 20:04     ` John Snow
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 11/21] blockjobs: add block_job_dismiss John Snow
2018-02-27 19:44   ` Eric Blake
2018-02-28 15:53   ` Kevin Wolf
2018-02-28 20:35     ` John Snow
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 12/21] blockjobs: ensure abort is called for cancelled jobs John Snow
2018-02-27 19:49   ` Eric Blake
2018-02-27 20:43     ` John Snow
2018-02-28 16:05   ` Kevin Wolf
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 13/21] blockjobs: add commit, abort, clean helpers John Snow
2018-02-27 19:50   ` Eric Blake
2018-02-28 16:07   ` Kevin Wolf
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 14/21] blockjobs: add block_job_txn_apply function John Snow
2018-02-27 19:52   ` Eric Blake
2018-02-28 16:32   ` Kevin Wolf
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 15/21] blockjobs: add prepare callback John Snow
2018-02-27 19:56   ` Eric Blake
2018-02-27 20:45     ` John Snow
2018-02-28 17:04   ` Kevin Wolf
2018-03-07  3:19     ` John Snow
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 16/21] blockjobs: add waiting status John Snow
2018-02-27 20:00   ` Eric Blake
2018-02-27 20:50     ` John Snow
2018-02-28 17:46       ` Kevin Wolf
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 17/21] blockjobs: add PENDING status and event John Snow
2018-02-27 20:05   ` Eric Blake
2018-02-27 20:54     ` John Snow
2018-02-28 17:55   ` Kevin Wolf
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 18/21] blockjobs: add block-job-finalize John Snow
2018-02-27 20:13   ` Eric Blake
2018-02-28 18:15   ` Kevin Wolf
2018-02-28 19:14     ` John Snow [this message]
2018-03-01 10:01       ` Kevin Wolf
2018-03-01 19:24         ` John Snow
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 19/21] blockjobs: Expose manual property John Snow
2018-02-27 20:16   ` Eric Blake
2018-02-27 20:42     ` John Snow
2018-02-27 21:57     ` John Snow
2018-02-27 22:27       ` Eric Blake
2018-02-28 18:23       ` Kevin Wolf
2018-02-28 19:19         ` John Snow
2018-02-28 18:25   ` Kevin Wolf
2018-02-28 19:20     ` John Snow
2018-02-28 19:27       ` [Qemu-devel] [Qemu-block] " Eric Blake
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 20/21] iotests: test manual job dismissal John Snow
2018-02-27 20:21   ` Eric Blake
2018-02-27 20:41     ` John Snow
2018-02-23 23:51 ` [Qemu-devel] [RFC v4 21/21] blockjobs: add manual_mgmt option to transactions John Snow
2018-02-27 20:24   ` Eric Blake
2018-02-28 18:29     ` Kevin Wolf
2018-02-28 19:24       ` John Snow
2018-03-01 10:10         ` Kevin Wolf
2018-02-24  0:30 ` [Qemu-devel] [RFC v4 00/21] blockjobs: add explicit job management no-reply
2018-02-24 14:31 ` no-reply
2018-02-27 21:01   ` John Snow
2018-02-28 18:32     ` Kevin Wolf
2018-02-25 23:25 ` no-reply
2018-02-27 20:58   ` John Snow

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=24618453-fd43-de12-1e14-2fc9ed2c4b8b@redhat.com \
    --to=jsnow@redhat.com \
    --cc=jtc@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=pkrempa@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).