On 03/19/2013 01:28 PM, Markus Armbruster wrote: > Please cc: Luiz and me on QMP work in the future. > > Michal Novotny <minov...@redhat.com> writes: > >> This is the patch to introduce the query-cpu-max QMP command to get >> the maximum number of CPUs supported by the currently running emulator >> instance. This may differ machine from machine as defined by -machine >> settings and max_cpus member of QEMUMachine structure. > Humor me: don't start commit message bodies with "This patch" or > variations thereof, and don't repeat the subject. Suggest: > > QMP command query-cpu-max returns the maximum number of CPUs supported > by the currently running emulator instance, as defined in its > QEMUMachine struct. > >> It's been tested both using QMP/qmp utility and telnet session on >> the QEMU session. > What's a QMP/qmp utility? Let's drop this sentence. In the future, > feel free to put testing info below the "---" line.
I meant the utility located in <qemu-dir-clone>/QMP/qmp which is essentially the python script for QMP testing. > >> The HMP counterpart called cpu_max has been introduced by this patch >> too. > Grammar nit: s/has been/is/. Even better, avoid passive voice. > > Hmm, I just rewrote most of your commit message, so why not rewrite all > of it: > > New QMP command query-cpu-max and HMP command cpu_max > > These commands return the maximum number of CPUs supported by the > currently running emulator instance, as defined in its QEMUMachine > struct. > > Perhaps Luiz can fix up the commit message commit, if you don't mind. > > Patch looks good. The commit log change is fine with me. Thanks for your feedback! Michal > Should query commands for machine properties multiply, we should > consider creating a single command returning all of them. I'm not > asking you to do that now. > > Reviewed-by: Markus Armbruster <arm...@redhat.com> -- Michal Novotny <minov...@redhat.com>, RHCE, Red Hat Virtualization | libvirt-php bindings | php-virt-control.org