[libvirt] [libvirt-perl PATCH v2] Virt.xs: fix flag issue on set_scheduler_parameters

[Eric Blake]

On 08/29/2012 03:03 AM, Alex Jia wrote:
> From: Alex Jia <Alex Jia ajia at redhat.com>
> 
> The default flags are inconsistent on both qemuSetSchedulerParameters()
> and qemuGetSchedulerParameters() in libvirt, the qemuGetSchedulerParameters()
> always passes 'VIR_DOMAIN_AFFECT_CURRENT' flag to the qemuGetSchedulerParametersFlags(),
> it should be a expected behavior, but the qemuSetSchedulerParameters() always passes
> 'VIR_DOMAIN_AFFECT_LIVE' flag to the qemuSetSchedulerParametersFlags(), if users use
> default flag=0 or explicitly give a 'VIR_DOMAIN_AFFECT_CURRENT' flag to the
> set_scheduler_parameters() in perl-Sys-Virt, because the flag value is 0, the result is
> the virDomainSetSchedulerParameters() is called incorrectly.
> 
> In fact, it's enough to call the qemuSetSchedulerParametersFlags() regardless of what flag is
> like 'virsh schedinfo' command.

But doing this makes it harder to talk to older libvirtd that lacked the
new function.  I'm not sure whether we want to blindly force the need
for the newer function to exist, or whether we should be reproducing the
same logic for deciding when to try the older API based on the flags
presented by the user.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 617 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120831/9394e1a0/attachment-0001.sig>