On Tue, Jul 17, 2007 at 11:16:24AM +0100, Richard W.M. Jones wrote:
Atsushi SAKAI wrote:
>Hi,
>
>Add Range check in Credit Scheduler.
Hum, it's true that this code is really specific to Xen at this
point in virsh.c
But I think the long term way is to add that check where it can really be
done without adding more hypervisor specific code to virsh.c, could you
rewrite the patch to actually make the check where it belongs in
xenHypervisorSetSchedulerParameters() in src/xen_internal.c, that way
not only virsh is protected but also any other app using the API to
do scheduling tuning,
thanks in advance,
Daniel
--
Red Hat Virtualization group
http://redhat.com/virtualization/
Daniel Veillard | virtualization library
http://libvirt.org/
veillard(a)redhat.com | libxml GNOME XML XSLT toolkit
http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine
http://rpmfind.net/