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@xxxxxxxxxx | libxml GNOME XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/ -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list