[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v4 21/21] blockjobs: add manual_mgmt option to tra
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [RFC v4 21/21] blockjobs: add manual_mgmt option to transactions |
Date: |
Wed, 28 Feb 2018 14:24:20 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 02/28/2018 01:29 PM, Kevin Wolf wrote:
> Am 27.02.2018 um 21:24 hat Eric Blake geschrieben:
>> On 02/23/2018 05:51 PM, John Snow wrote:
>>> This allows us to easily force the option for all jobs belonging
>>> to a transaction to ensure consistency with how all those jobs
>>> will be handled.
>>>
>>> This is purely a convenience.
>>>
>>> Signed-off-by: John Snow <address@hidden>
>>> ---
>>
>>> +++ b/qapi/transaction.json
>>> @@ -79,7 +79,8 @@
>>> ##
>>> { 'struct': 'TransactionProperties',
>>> 'data': {
>>> - '*completion-mode': 'ActionCompletionMode'
>>> + '*completion-mode': 'ActionCompletionMode',
>>> + '*manual-mgmt': 'bool'
>>
>> Missing QAPI documentation (what you have elsewhere in the C code can
>> probably be copied here, though).
>>
>> The UI aspect makes sense (I can declare one manual at the transaction level
>> instead of multiple manual declarations per member level within the
>> transaction).
>
> I'm not so sure if I like the interface, it duplicates functionality in
> two places.
>
> At th very least I would make job creation without BLOCK_JOB_MANUAL an
> error if the transaction requires it instead of silently overriding the
> option that was given to the individual job. But honestly, it might be
> better to just leave this one away.
>
> Kevin
>
Sure, I put it in the trailing position here because I see it as
optional. I don't like the idea of having to specify manual for each and
every item in a transaction, but if mixed-mode is possible then this is
less important.
I'll leave it off for now, but I will always fondly remember it, and
then maybe try to sneak it back in for v6.
--js
- Re: [Qemu-devel] [RFC v4 19/21] blockjobs: Expose manual property, (continued)
Re: [Qemu-devel] [RFC v4 19/21] blockjobs: Expose manual property, Kevin Wolf, 2018/02/28
[Qemu-devel] [RFC v4 21/21] blockjobs: add manual_mgmt option to transactions, John Snow, 2018/02/23
[Qemu-devel] [RFC v4 18/21] blockjobs: add block-job-finalize, John Snow, 2018/02/23
[Qemu-devel] [RFC v4 11/21] blockjobs: add block_job_dismiss, John Snow, 2018/02/23
[Qemu-devel] [RFC v4 02/21] blockjobs: model single jobs as transactions, John Snow, 2018/02/23