[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Libvir] [PATCH][RFC] shows scheduler information



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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]