Re: [libvirt] [PATCH] libvirt: Update docs for hotplug only commands

On Tue, Feb 16, 2010 at 02:35:49PM -0500, Cole Robinson wrote: > The commands updated are SetMem, SetMaxMem, SetVcpus, and PinVcpu. > > Signed-off-by: Cole Robinson <crobinso(a)redhat.com&gt; > --- > src/libvirt.c | 12 ++++++++++++ > 1 files changed, 12 insertions(+), 0 deletions(-) > > diff --git a/src/libvirt.c b/src/libvirt.c > index 0e5f4a2..be17668 100644 > --- a/src/libvirt.c > +++ b/src/libvirt.c > @@ -2758,6 +2758,9 @@ error: > * to Domain0 i.e. the domain where the application runs. > * This function requires privileged access to the hypervisor. > * > + * This command only changes the runtime configuration of the domain, > + * so can only be called on an active domain. > + * > * Returns 0 in case of success and -1 in case of failure. > */ > int > @@ -2808,6 +2811,9 @@ error: > * to Domain0 i.e. the domain where the application runs. > * This function may requires privileged access to the hypervisor. > * > + * This command only changes the runtime configuration of the domain, > + * so can only be called on an active domain. > + * > * Returns 0 in case of success and -1 in case of failure. > */ > int > @@ -4821,6 +4827,9 @@ error: > * does not support it or if growing the number is arbitrary limited. > * This function requires privileged access to the hypervisor. > * > + * This command only changes the runtime configuration of the domain, > + * so can only be called on an active domain. > + * > * Returns 0 in case of success, -1 in case of failure. > */ > > @@ -4879,6 +4888,9 @@ error: > * Dynamically change the real CPUs which can be allocated to a virtual CPU. > * This function requires privileged access to the hypervisor. > * > + * This command only changes the runtime configuration of the domain, > + * so can only be called on an active domain. > + * > * Returns 0 in case of success, -1 in case of failure. > */ > int > -- ACK Daniel