[libvirt] [PATCH v2 01/12] qemu: qemuDomainJobInfoToParams drop unused code
Nikolay Shirokovskiy
nshirokovskiy at virtuozzo.com
Fri Feb 17 11:29:33 UTC 2017
On 16.02.2017 16:07, Jiri Denemark wrote:
> On Wed, Dec 28, 2016 at 17:39:10 +0300, Nikolay Shirokovskiy wrote:
>> qemu driver does not have VIR_DOMAIN_JOB_BOUNDED jobs.
>> ---
>> src/qemu/qemu_domain.c | 6 ------
>> 1 file changed, 6 deletions(-)
>>
>> diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
>> index acfa695..acc27d0 100644
>> --- a/src/qemu/qemu_domain.c
>> +++ b/src/qemu/qemu_domain.c
>> @@ -432,12 +432,6 @@ qemuDomainJobInfoToParams(qemuDomainJobInfoPtr jobInfo,
>> jobInfo->timeElapsed - jobInfo->timeDelta) < 0)
>> goto error;
>>
>> - if (jobInfo->type == VIR_DOMAIN_JOB_BOUNDED &&
>> - virTypedParamsAddULLong(&par, &npar, &maxpar,
>> - VIR_DOMAIN_JOB_TIME_REMAINING,
>> - jobInfo->timeRemaining) < 0)
>> - goto error;
>> -
>> if (stats->downtime_set &&
>> virTypedParamsAddULLong(&par, &npar, &maxpar,
>> VIR_DOMAIN_JOB_DOWNTIME,
>
> The function is supposed to process all fields in qemuDomainJobInfo and
> we should keep it so. If we want to drop this code, we should just drop
> qemuDomainJobInfo.timeRemaining completely.
>
Ok. Looks like we can drop timeRemaining from cookies as well as its
parsing result never checked.
Nikolay
More information about the libvir-list
mailing list