From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44295) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1emfHx-0004Hu-Ez for qemu-devel@nongnu.org; Fri, 16 Feb 2018 07:38:54 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1emfHu-0003WU-6C for qemu-devel@nongnu.org; Fri, 16 Feb 2018 07:38:53 -0500 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:34868 helo=mx1.redhat.com) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1emfHu-0003WG-1d for qemu-devel@nongnu.org; Fri, 16 Feb 2018 07:38:50 -0500 From: Igor Mammedov Date: Fri, 16 Feb 2018 13:37:12 +0100 Message-Id: <1518784641-43151-1-git-send-email-imammedo@redhat.com> Subject: [Qemu-devel] [PATCH v3 0/9] enable numa configuration before machine_init() from QMP List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: eblake@redhat.com, armbru@redhat.com, ehabkost@redhat.com, pkrempa@redhat.com, david@gibson.dropbear.id.au, peter.maydell@linaro.org, pbonzini@redhat.com, cohuck@redhat.com v1->v3: * introduce PRECONFIG runstate with -preconfig option. it's cleaner to manage transitions and do checks than reusing existing PRELAUNCH state. * extend QAPI schema commands with 'runstates' keyword, so that it would be possible to specify in command definition when it is valid to execute. (python changes a bit hackery, since I have little to no idea how it should work) * add preconfig QMP and set-numa-node tests * make mutually exclusive -preconfig and -incoming options, for simplicity sake. Shouldn't be problem as target can be starter with pure CLI, since mapping on source is already known. * Drop HMP part and leave only QMP in preconfig state. Series allows to configure NUMA mapping at runtime using QMP interface. For that to happen it introduces a new '-preconfig' CLI option which allows to pause QEMU before machine_init() is run and adds new set-numa-node QMP command which in conjunction with query-hotpluggable-cpus allows to configure NUMA mapping for cpus. Later we can modify other commands to run early, for example device_add. I recall SPAPR had problem when libvirt started QEMU with -S and, while it's paused, added CPUs with device_add. Intent was to coldplug CPUs (but at that stage it's considered hotplug already), so SPAPR had to work around the issue. Example of configuration session: $QEMU -smp 2 -preconfig ... QMP: # get CPUs layout for current target/machine/CLI -> {'execute': 'query-hotpluggable-cpus' } <- {'return': [ {'props': {'core-id': 0, 'thread-id': 0, 'socket-id': 1}, ... }, {'props': {'core-id': 0, 'thread-id': 0, 'socket-id': 0}, ... } ]} # configure 1st node -> {'execute': 'set-numa-node', 'arguments': { 'type': 'node', 'nodeid': 0 } } <- {'return': {}} -> {'execute': 'set-numa-node', 'arguments': { 'type': 'cpu', 'node-id': 0, 'core-id': 0, 'thread-id': 0, 'socket-id': 1, } } <- {'return': {}} # configure 2nd node -> {'execute': 'set-numa-node', 'arguments': { 'type': 'node', 'nodeid': 1 } } -> {'execute': 'set-numa-node', 'arguments': { 'type': 'cpu', 'node-id': 1, 'core-id': 0, 'thread-id': 0, 'socket-id': 0 } } <- {'return': {}} # [optional] verify configuration -> {'execute': 'query-hotpluggable-cpus' } <- {'return': [ {'props': {'core-id': 0, 'thread-id': 0, 'node-id': 0, 'socket-id': 1}, ... }, {'props': {'core-id': 0, 'thread-id': 0, 'node-id': 1, 'socket-id': 0}, ... } ]} Git tree: https://github.com/imammedo/qemu.git qmp_preconfig_v3 Ref to v1: https://lists.gnu.org/archive/html/qemu-devel/2017-10/msg03583.html Message-Id: <1508170976-96869-1-git-send-email-imammedo@redhat.com> CC: eblake@redhat.com CC: armbru@redhat.com CC: ehabkost@redhat.com CC: pkrempa@redhat.com CC: david@gibson.dropbear.id.au CC: peter.maydell@linaro.org CC: pbonzini@redhat.com CC: cohuck@redhat.com Igor Mammedov (9): numa: postpone options post-processing till machine_run_board_init() numa: split out NumaOptions parsing into parse_NumaOptions() CLI: add -preconfig option HMP: disable monitor in preconfig state QAPI: allow to specify valid runstates per command tests: extend qmp test with pereconfig checks QMP: permit query-hotpluggable-cpus in preconfig state QMP: add set-numa-node command tests: functional tests for QMP command set-numa-node include/qapi/qmp/dispatch.h | 5 ++- include/sysemu/numa.h | 2 + include/sysemu/sysemu.h | 1 + hw/core/machine.c | 5 ++- monitor.c | 32 +++++++++++++-- numa.c | 66 +++++++++++++++++++----------- qapi-schema.json | 32 +++++++++++++-- qapi/qmp-dispatch.c | 39 ++++++++++++++++++ qapi/qmp-registry.c | 4 +- qapi/run-state.json | 9 ++++- qemu-options.hx | 11 +++++ qmp.c | 5 +++ scripts/qapi-commands.py | 46 ++++++++++++++++----- scripts/qapi-introspect.py | 2 +- scripts/qapi.py | 15 ++++--- scripts/qapi2texi.py | 2 +- tests/numa-test.c | 71 +++++++++++++++++++++++++++++++++ tests/qapi-schema/doc-good.out | 4 +- tests/qapi-schema/ident-with-escape.out | 2 +- tests/qapi-schema/indented-expr.out | 4 +- tests/qapi-schema/qapi-schema-test.out | 18 ++++----- tests/qapi-schema/test-qapi.py | 6 +-- tests/qmp-test.c | 56 ++++++++++++++++++++++++++ vl.c | 35 +++++++++++++++- 24 files changed, 401 insertions(+), 71 deletions(-) -- 2.7.4