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

Re: [Libvir] [PATCH] Add Xen Scheduler APIs to xenDaemonDriver



Hello,

Thanks for applying the first patch (about adding read-only connections).
http://git.et.redhat.com/?p=libvirt.git;a=commit;h=4faeb0d567976377b2661fb5ebd0f1dca7763f80

How about the second one ? We can make some improvements for virsh 
schedinfo command with the patch, so I want the patch to be committed
if it is OK.

Regards,
Saori Fukuta

---
On Thu, 13 Mar 2008 17:30:57 +0900 Saori Fukuta wrote:

> Hi,
> 
> I add the Xen Scheduler APIs to xenDaemonDriver.
>    - xenDaemonGetSchedulerType
>    - xenDaemonGetSchedulerParameters
>    - xenDaemonSetSchedulerParameters
> We will have the following improvements by this patch:
>    1) xenDaemonDriver of Schedulr is usable 
>       when xenHypervisorDriver failed.
>    2) Show the Scheduler information with non-root user.
>    3) Set the Scheduler parameters for inactive domain.
>       (Unfortunately not right now, because we need this fix.
>        http://lists.xensource.com/archives/html/xen-devel/2008-03/msg00301.html)
> 
> And I have a few things to note:
>    - Currently, implemented for the Xen-credit scheduler only.
>    - xenDaemonGetSchedulerType is supported in xendConfigVersion >= 4,
>      because we can get "xen_scheduler" from xend after Cset13394 of
>      Xen (i.e. xendConfigVersion is four or later).
>    - For the above reason, xenDaemonGetSchedulerParameters and
>      xenDaemonSetSchedulerParameters are also supported in 
>      xendConfigVersion >= 4, because they are using xenDaemonGetSchedulerType
>      to get the scheduler type.
>    - At xenDaemonSetSchedulerParameters, there are restrictions 
>      which depends on Xend-version.
>      a) We cannot set the "cap" value by xend before Cset15476 of Xen, 
>         because xend does not support the "cap" value 
>         at xend/server/SrvDomain.py
>      b) We cannot set the scheduler parameters for inactive domain
>         without the following patch.
>         http://lists.xensource.com/archives/html/xen-devel/2008-03/msg00301.html
>         If the above patch is committed, this API will be available 
>         for inactive domain.
> 
> Regards,
> Saori Fukuta



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