Re: [PATCH v2 01/12] qemu: qemuDomainJobInfoToParams drop unused code

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

 




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

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux