On Thu, Apr 12, 2007 at 05:40:23PM +0900, Atsushi SAKAI wrote: > Hi, Rich > > Thank you for your suggestion. > I fixed it. > > Changes > 1)changes static to dynamic. (like sedf, credit, weight, cap) > 2)virsh printf added various print patterns for scheduler parameters. > 3)Tab space changed(from 2 to 4) in xen_unified.c part Hum, I still see +const char * virDomainGetSchedulerType (virDomainPtr domain, + int *nparams); and in virsh.c I still see + schedulertype = virDomainGetSchedulerType(dom, nparams); and schedulertype is not freed So I guess that doesn't address Richard's point, 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/