Re: [libvirt] [PATCH 3 of 3] Add cpu_shares support to virsh schedinfo

2008-10-07 Thread Dan Smith
DB> Not your fault, but this schedinfo command is a really dumb wrt to DB> option handling. Requiring that we add new options for every DB> possible schedular param for every possible hypervisor is just DB> crazy. Agreed. I actually just wrote this patch for my own testing, but decided to send it

Re: [libvirt] [PATCH 3 of 3] Add cpu_shares support to virsh schedinfo

2008-10-07 Thread Daniel P. Berrange
On Tue, Oct 07, 2008 at 08:30:22AM -0700, Dan Smith wrote: > diff -r ebecbe5caa03 -r b07cd92a30e9 src/virsh.c > --- a/src/virsh.c Tue Oct 07 08:21:50 2008 -0700 > +++ b/src/virsh.c Tue Oct 07 08:21:50 2008 -0700 > @@ -1114,6 +1114,7 @@ > {"domain", VSH_OT_DATA, VSH_OFLAG_REQ, gettext_n

[libvirt] [PATCH 3 of 3] Add cpu_shares support to virsh schedinfo

2008-10-07 Thread Dan Smith
diff -r ebecbe5caa03 -r b07cd92a30e9 src/virsh.c --- a/src/virsh.c Tue Oct 07 08:21:50 2008 -0700 +++ b/src/virsh.c Tue Oct 07 08:21:50 2008 -0700 @@ -1114,6 +1114,7 @@ {"domain", VSH_OT_DATA, VSH_OFLAG_REQ, gettext_noop("domain name, id or uuid")}, {"weight", VSH_OT_INT, VSH