[Libvir] [PATCH] add scheduler API(take 3?)
sakaia at jp.fujitsu.com
Thu May 31 04:29:03 UTC 2007
Thank you very much for your patch.
It works fine.
And Your patch is very helpful to understand the Error handling policy on
It is improves my knowledge.
"Richard W.M. Jones" <rjones at redhat.com> wrote:
> The attached patch fixes some things in your add_scheduler10.patch (you
> need to apply this patch on top of your patch):
> * Lots of error checking added. If the lowest level function detects an
> error, it *must* report a useful message.
> * Make libvirt.h match libvirt.h.in. If you edit just libvirt.h then
> any changes you make will be lost next time someone runs ./configure.
> * Removed VIR_DOMAIN_SCHED_FIELD_INIT. This field is now initialised
> with zero.
> * Changed the type of that field to the more logical
> virSchedParameterType type.
> * virDomainSetSchedulerParameters, nparams does not really need to be a
> pointer (can it ever be changed?)
> * In virsh, only call virDomainSetSchedulerParameters if the user has
> requested a parameter to change.
> * In virDomainGetSchedulerType, allow nparams (pointer) to be NULL - eg.
> in case the caller doesn't care about the number of parameters.
> Note: My Xen machine is completely broken at the moment so I haven't
> been able to test this, just compile it.
> Emerging Technologies, Red Hat - http://et.redhat.com/~rjones/
> Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod
> Street, Windsor, Berkshire, SL4 1TE, United Kingdom. Registered in
> England and Wales under Company Registration No. 03798903
More information about the libvir-list