qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [Qemu-devel] [PULL 18/41] blockjobs: add block-job-fina


From: John Snow
Subject: Re: [Qemu-block] [Qemu-devel] [PULL 18/41] blockjobs: add block-job-finalize
Date: Wed, 14 Mar 2018 16:24:34 -0400
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0


On 03/13/2018 02:47 PM, Eric Blake wrote:
> On 03/13/2018 11:17 AM, Kevin Wolf wrote:
>> From: John Snow <address@hidden>
>>
>> Instead of automatically transitioning from PENDING to CONCLUDED, gate
>> the .prepare() and .commit() phases behind an explicit acknowledgement
>> provided by the QMP monitor if auto_finalize = false has been requested.
>>
> 
>>   ##
>> +# @block-job-finalize:
>> +#
>> +# Once a job that has manual=true reaches the pending state, it can be
> 
> Is this wording stale, given that you add two separate auto-* bool flags
> in 19/41?  You may want to prepare a followup patch (doc bug fixes are
> safe during softfreeze, so it need not hold up this pull request) that
> tweaks this and any similar stale wording.
> 

Fixed up in my local branch, will send out once the dust settles on master.

Thanks!

--js



reply via email to

[Prev in Thread] Current Thread [Next in Thread]