From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55610) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UwGL1-0000Fk-CQ for qemu-devel@nongnu.org; Mon, 08 Jul 2013 14:39:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UwGEQ-0000qz-8e for qemu-devel@nongnu.org; Mon, 08 Jul 2013 14:32:46 -0400 Received: from mx1.redhat.com ([209.132.183.28]:2887) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UwGEP-0000qR-Pj for qemu-devel@nongnu.org; Mon, 08 Jul 2013 14:32:13 -0400 Date: Mon, 8 Jul 2013 14:32:05 -0400 From: Luiz Capitulino Message-ID: <20130708143205.23e30dcc@redhat.com> In-Reply-To: <1372931597-28115-10-git-send-email-gaowanlong@cn.fujitsu.com> References: <1372931597-28115-1-git-send-email-gaowanlong@cn.fujitsu.com> <1372931597-28115-10-git-send-email-gaowanlong@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH V4 09/10] NUMA: add hmp command set-mpol List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Wanlong Gao Cc: aliguori@us.ibm.com, ehabkost@redhat.com, qemu-devel@nongnu.org, bsd@redhat.com, y-goto@jp.fujitsu.com, pbonzini@redhat.com, afaerber@suse.de On Thu, 4 Jul 2013 17:53:16 +0800 Wanlong Gao wrote: > Add hmp command set-mpol to set host memory policy for a guest > NUMA node. Then we can also set node's memory policy using > the monitor command like: > (qemu) set-mpol 0 mem-policy=membind,mem-hostnode=0-1 > > Signed-off-by: Wanlong Gao > --- > hmp-commands.hx | 16 ++++++++++++++++ > hmp.c | 35 +++++++++++++++++++++++++++++++++++ > hmp.h | 1 + > 3 files changed, 52 insertions(+) > > diff --git a/hmp-commands.hx b/hmp-commands.hx > index 915b0d1..417b69f 100644 > --- a/hmp-commands.hx > +++ b/hmp-commands.hx > @@ -1567,6 +1567,22 @@ Executes a qemu-io command on the given block device. > ETEXI > > { > + .name = "set-mpol", > + .args_type = "nodeid:i,args:s?", > + .params = "nodeid [args]", > + .help = "set host memory policy for a guest NUMA node", > + .mhandler.cmd = hmp_set_mpol, > + }, > + > +STEXI > +@item set-mpol @var{nodeid} @var{args} > +@findex set-mpol > + > +Set host memory policy for a guest NUMA node > + > +ETEXI > + > + { > .name = "info", > .args_type = "item:s?", > .params = "[subcommand]", > diff --git a/hmp.c b/hmp.c > index 2daed43..57a5730 100644 > --- a/hmp.c > +++ b/hmp.c > @@ -1482,3 +1482,38 @@ void hmp_qemu_io(Monitor *mon, const QDict *qdict) > > hmp_handle_error(mon, &err); > } > + > +void hmp_set_mpol(Monitor *mon, const QDict *qdict) > +{ > + Error *local_err = NULL; > + bool has_mpol = true; > + bool has_hostnode = true; > + const char *mpol = NULL; > + const char *hostnode = NULL; > + QemuOpts *opts; > + > + uint64_t nodeid = qdict_get_int(qdict, "nodeid"); > + const char *args = qdict_get_try_str(qdict, "args"); > + > + if (args == NULL) { > + has_mpol = false; > + has_hostnode = false; > + } else { > + opts = qemu_opts_parse(qemu_find_opts("numa"), args, 1); > + if (opts == NULL) { > + error_setg(&local_err, "Parsing memory policy args failed"); You're still going to call qmp_set_mpol() if this fails. You can replace error_setg() to a monitor_printf() call and return. > + } else { > + mpol = qemu_opt_get(opts, "mem-policy"); > + if (mpol == NULL) { > + has_mpol = false; > + } > + hostnode = qemu_opt_get(opts, "mem-hostnode"); > + if (hostnode == NULL) { > + has_hostnode = false; > + } > + } > + } > + > + qmp_set_mpol(nodeid, has_mpol, mpol, has_hostnode, hostnode, &local_err); > + hmp_handle_error(mon, &local_err); > +} > diff --git a/hmp.h b/hmp.h > index 56d2e92..81f631b 100644 > --- a/hmp.h > +++ b/hmp.h > @@ -86,5 +86,6 @@ void hmp_nbd_server_stop(Monitor *mon, const QDict *qdict); > void hmp_chardev_add(Monitor *mon, const QDict *qdict); > void hmp_chardev_remove(Monitor *mon, const QDict *qdict); > void hmp_qemu_io(Monitor *mon, const QDict *qdict); > +void hmp_set_mpol(Monitor *mon, const QDict *qdict); > > #endif