[Libvir] Add scheduler patch?

Atsushi SAKAI sakaia at jp.fujitsu.com
Mon Jun 4 09:28:26 UTC 2007

Hi, Daniel

I plan to post Xen/SEDF patch.
Anyway I hope current patch will apply first,
Since coding frame of this command is fixed previously.

Atsushi SAKAI

Daniel Veillard <veillard at redhat.com> wrote:

> On Mon, Jun 04, 2007 at 05:17:56PM +0900, Atsushi SAKAI wrote:
> > Hi, 
> > 
> > Is there any plan to apply scheduler controller patch?
> > 
> > https://www.redhat.com/archives/libvir-list/2007-May/msg00291.html
> > https://www.redhat.com/archives/libvir-list/2007-May/msg00295.html
>   In theory, yes. I'm not enthusiastic about the new APIs but considering
> that there is no unity in the various existing schedulers, we have to make
> an API which stay open like this and will be able to evolve to future needs.
>   I read the patch quickly, and I need to do a more thorough review before
> applying it. I would feel better too if the case XEN_SCHEDULER_SEDF wasn't
> just made of TODOs "/* TODO: Implement for Xen/SEDF */" in most places, 
> I'm afraid that since it's the old scheduler you have little interest in
> implementing those, and it's likely nobody else will so applying the patch
> means adding TODOs to the code nearly forever.
>   So in practice I nedd to make a second pass on it, and if you could
> provide a separate patch cleaning up the Xen/SEDF in the meantime that 
> would be perfect :-)
>    thanks,
> Daniel
> -- 
> Red Hat Virtualization group http://redhat.com/virtualization/
> Daniel Veillard      | virtualization library  http://libvirt.org/
> veillard at redhat.com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
> http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/

More information about the libvir-list mailing list