[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 09/45] block: rename block_job_complete to bl
From: |
Jeff Cody |
Subject: |
Re: [Qemu-devel] [PATCH v2 09/45] block: rename block_job_complete to block_job_completed |
Date: |
Thu, 27 Sep 2012 16:31:46 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120828 Thunderbird/15.0 |
On 09/27/2012 08:30 AM, Kevin Wolf wrote:
> Am 26.09.2012 17:56, schrieb Paolo Bonzini:
>> The imperative will be used for the QMP command.
>>
>> Signed-off-by: Paolo Bonzini <address@hidden>
>
> I would still be glad if we found a better name. Having two functions
> block_job_complete() and block_job_completed() sounds like a great
> source for confusion.
>
> Kevin
>
If I understand correctly, what we have is:
block_job_completed(): cleans up when a job is done
block_job_complete(): requests that a block job be completed
How about renaming both of them, respectively, to:
block_job_cleanup(): cleans up when a job is done
block_job_request_completion(): requests that a block job be completed
I think that would remove any ambiguity.
- Re: [Qemu-devel] [PATCH v2 06/45] block: add support for job pause/resume, (continued)
[Qemu-devel] [PATCH v2 07/45] qmp: add block-job-pause and block-job-resume, Paolo Bonzini, 2012/09/26
[Qemu-devel] [PATCH v2 08/45] qemu-iotests: add test for pausing a streaming operation, Paolo Bonzini, 2012/09/26
[Qemu-devel] [PATCH v2 09/45] block: rename block_job_complete to block_job_completed, Paolo Bonzini, 2012/09/26
[Qemu-devel] [PATCH v2 11/45] iostatus: move BlockdevOnError declaration to QAPI, Paolo Bonzini, 2012/09/26
[Qemu-devel] [PATCH v2 13/45] iostatus: reorganize io error code, Paolo Bonzini, 2012/09/26
[Qemu-devel] [PATCH v2 10/45] iostatus: rename BlockErrorAction, BlockQMPEventAction, Paolo Bonzini, 2012/09/26
[Qemu-devel] [PATCH v2 12/45] iostatus: change is_read to a bool, Paolo Bonzini, 2012/09/26
[Qemu-devel] [PATCH v2 14/45] block: introduce block job error, Paolo Bonzini, 2012/09/26