Hi Daniel I forgot the spelling check of comment. I contribute the patch that corrects the comment again. Moreever, because some return values were illegal, I corrected it. Signed-off-by: Masayuki Sunou <fj1826dm@xxxxxxxxxxxxxxxxx> Thanks In message <20070307220239.GI1782@xxxxxxxxxx> "Re: [PATCH] check the maximum of virtual CPU" "Daniel Veillard <veillard@xxxxxxxxxx>" wrote: > On Tue, Mar 06, 2007 at 08:08:57PM +0900, Masayuki Sunou wrote: > > Hi, Dan > > > > > > 1. When initially creating a VM > > > > 2. When changing the config of an inactive VM > > > > 3. When changing the config of a running VM > > > > > > Certainly, I had not considered concerning 1 and 2. > > So, I corrected the patch based on your proposal. > > > > ・ virDomainGetMaxVcpus > > Judge state (active/inactive) of the domain, and return information > > corresponding to each state. > > > > ・ virConnectGetMaxVcpus > > Add it as a method that returns the number of maximum CPUs defined by Xen. > > > > > > However, I only added virConnectGetMaxVcpus because I did not understand > > the use image of it. > > > > Is it added as a command of virsh? > > Or, is it used from virsh create and virsh start? > > > > > > Signed-off-by: Masayuki Sunou <fj1826dm@xxxxxxxxxxxxxxxxx> > > Looks good to me, the new function comment should be fixed to avoid the > 'Thje' typo, and 'make rebuild' should be run in the doc subdir to regenerate > documentations, but it's IMHO ready to be applied. > > Thanks a lot ! > > Daniel > > -- > Red Hat Virtualization group http://redhat.com/virtualization/ > Daniel Veillard | virtualization library http://libvirt.org/ > veillard@xxxxxxxxxx | libxml GNOME XML XSLT toolkit http://xmlsoft.org/ > http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/
Attachment:
libvirt-check-max-vcpu.patch
Description: Binary data