Re: [Libvir] [PATCH] Add Xen Scheduler APIs to xenDaemonDriver
On Mon, Mar 24, 2008 at 02:12:25PM +0900, Saori Fukuta wrote:
Hello,
Thanks for applying the first patch (about adding read-only connections).
http://git.et.redhat.com/?p=libvirt.git;a=commit;h=4faeb0d567976377b2661f...
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.
I had forgotten about it, sorry.
It looks okay, though i have to clean a few problems raised by
'make syntax-check' (please try this in future patches). I commited it to CVS.
However, i don't see why it is needed, basically when running with Xen
as root, the hypervisor access should be used, not xend, so this seems to
indicate that you have problem using the hypervisor calls and that's probably
something to worry about. But it's fine having a more complete xend driver
surrport.
thanks,
Daniel
--
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard | virtualization library http://libvirt.org/
veillard(a)redhat.com | libxml GNOME XML XSLT toolkit http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/