Re: [libvirt] [PATCH 1/4] qemu: Make Set*Mem commands hotplug only

2010/2/12 Cole Robinson <crobinso(a)redhat.com&gt;: > SetMem and SetMaxMem are hotplug only APIs, any persistent config > changes are supposed to go via XML definition. The original implementation > of these calls were incorrect and had the nasty side effect of making > a psuedo persistent change that would be lost after libvirtd restart > (I didn't know any better). > > Fix these APIs to rightly reject non running domains. > --- Could you document this somewhere that virDomainSetMaxMemory may be called for active domains only, like in the documentation of virDomainSetMaxMemory in libvirt.c? Several details of the libvirt API are only documentation deep down in the mailing list, or the developers heads. Now you've fixed the code to do what is supposed to do, but it's still not documented well. The same goes for the other commit where you made it callable for an active domain only.