Re: [libvirt] [PATCH] Add generic parameter=value support for virsh's schedinfo command

2008-10-08 Thread Daniel Veillard
On Wed, Oct 08, 2008 at 08:00:34AM -0700, Dan Smith wrote: > This patch maintains the two Xen-specific --weight and --cap options, > but adds support for setting arbitrary parameters by specifying them in > param=value syntax. > > Changes to the virsh manual are included. > > Changes: > - Replac

[libvirt] [PATCH] Add generic parameter=value support for virsh's schedinfo command

2008-10-08 Thread Dan Smith
This patch maintains the two Xen-specific --weight and --cap options, but adds support for setting arbitrary parameters by specifying them in param=value syntax. Changes to the virsh manual are included. Changes: - Replace use of 'a' conversion modifier with pre-alloc diff -r 51fd150edd42 -r e4

Re: [libvirt] [PATCH] Add generic parameter=value support for virsh's schedinfo command

2008-10-08 Thread Dan Smith
DV> a is not part of C89, that's an extension to the standard if i believe DV> the linux man page. I would rather avoid a portability problem here Oh, right, I always forget about that. I'll send a replacement. Thanks! -- Dan Smith IBM Linux Technology Center Open Hypervisor Team email: [EMA

Re: [libvirt] [PATCH] Add generic parameter=value support for virsh's schedinfo command

2008-10-08 Thread Daniel Veillard
On Tue, Oct 07, 2008 at 01:55:28PM -0700, Dan Smith wrote: > This patch maintains the two Xen-specific --weight and --cap options, > but adds support for setting arbitrary parameters by specifying them in > param=value syntax. > > Changes to the virsh manual are included. Sounds good, but ...

[libvirt] [PATCH] Add generic parameter=value support for virsh's schedinfo command

2008-10-07 Thread Dan Smith
This patch maintains the two Xen-specific --weight and --cap options, but adds support for setting arbitrary parameters by specifying them in param=value syntax. Changes to the virsh manual are included. diff -r 51fd150edd42 -r aae58f42bd4a docs/virsh.pod --- a/docs/virsh.podTue Oct 07 11:45: