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/
[EMAIL PROTECTED]  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/

--
Libvir-list mailing list
[EMAIL PROTECTED]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to