From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=56358 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PGM75-0007iW-27 for qemu-devel@nongnu.org; Wed, 10 Nov 2010 20:38:42 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PGM6m-0008Mi-MS for qemu-devel@nongnu.org; Wed, 10 Nov 2010 20:38:06 -0500 Received: from e3.ny.us.ibm.com ([32.97.182.143]:41580) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PGM6m-0008MK-JM for qemu-devel@nongnu.org; Wed, 10 Nov 2010 20:37:48 -0500 Received: from d01relay06.pok.ibm.com (d01relay06.pok.ibm.com [9.56.227.116]) by e3.ny.us.ibm.com (8.14.4/8.13.1) with ESMTP id oAB1KTHP001834 for ; Wed, 10 Nov 2010 20:20:29 -0500 Received: from d01av01.pok.ibm.com (d01av01.pok.ibm.com [9.56.224.215]) by d01relay06.pok.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id oAB1bmPl2367726 for ; Wed, 10 Nov 2010 20:37:48 -0500 Received: from d01av01.pok.ibm.com (loopback [127.0.0.1]) by d01av01.pok.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id oAB1blGt009927 for ; Wed, 10 Nov 2010 20:37:47 -0500 From: Michael Roth Date: Wed, 10 Nov 2010 19:37:22 -0600 Message-Id: <1289439450-23556-4-git-send-email-mdroth@linux.vnet.ibm.com> In-Reply-To: <1289439450-23556-1-git-send-email-mdroth@linux.vnet.ibm.com> References: <1289439450-23556-1-git-send-email-mdroth@linux.vnet.ibm.com> Subject: [Qemu-devel] [RFC][PATCH v3 03/11] virtagent: qemu-vp, integrate virtagent server List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: agl@linux.vnet.ibm.com, abeekhof@redhat.com, mdroth@linux.vnet.ibm.com, aliguori@linux.vnet.ibm.com, ryanh@us.ibm.com, amit.shah@redhat.com This allows the guest RPC server to be integrated into the qemu-vp/virtproxy i/o loop Signed-off-by: Michael Roth --- qemu-vp.c | 115 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++----- 1 files changed, 106 insertions(+), 9 deletions(-) diff --git a/qemu-vp.c b/qemu-vp.c index 3a3762a..fe06e07 100644 --- a/qemu-vp.c +++ b/qemu-vp.c @@ -37,6 +37,7 @@ #include "qemu-option.h" #include "qemu_socket.h" #include "virtproxy.h" +#include "virtagent-daemon.h" static bool verbose_enabled = 0; #define DEBUG_ENABLED @@ -252,14 +253,18 @@ static void usage(const char *cmd) "[-o ...]\n" "QEMU virt-proxy communication channel\n" "\n" -" -c, --channel channel options of the form:\n" -" ::[:channel_id]\n" -" -o, --oforward oforward options of the form:\n" -" ::[:channel_id]\n" -" -i, --iforward iforward options of the form:\n" -" ::[:channel_id]\n" -" -v, --verbose display extra debugging information\n" -" -h, --help display this help and exit\n" +" -c, --channel channel options of the form:\n" +" ::[:channel_id]\n" +" -p, --host-agent host rpc server, options of the form:\n" +" [channel_id]\n" +" -g, --guest-agent guest rpc server, options of the form:\n" +" [channel_id]\n" +" -o, --oforward oforward options of the form:\n" +" ::[:channel_id]\n" +" -i, --iforward iforward options of the form:\n" +" ::[:channel_id]\n" +" -v, --verbose display extra debugging information\n" +" -h, --help display this help and exit\n" "\n" " channels are used to establish a data connection between 2 end-points in\n" " the host or the guest (connection method specified by ).\n" @@ -533,13 +538,44 @@ static int init_iforwards(void) { return 0; } +static int init_agent(const VPData *agent_iforward, bool is_host) { + QemuOpts *opts = agent_iforward->opts; + int listen_fd, ret; + + INFO("initializing agent..."); + if (verbose_enabled) { + qemu_opts_print(opts, NULL); + } + + /* create unix socket pair that agent http/rpc daemon will listen on */ + listen_fd = unix_listen_opts(agent_iforward->opts); + if (listen_fd < 0) { + return -1; + } + + /* start RPC server */ + ret = va_server_start(listen_fd, is_host); + if (ret != 0) { + warnx("error starting RPC server"); + goto err; + } + + return 0; + +err: + closesocket(listen_fd); + return -1; +} + int main(int argc, char **argv) { - const char *sopt = "hVvi:o:c:"; + const char *sopt = "hVvi:o:c:g::p::"; struct option lopt[] = { { "help", 0, NULL, 'h' }, { "version", 0, NULL, 'V' }, { "verbose", 0, NULL, 'v' }, + { "host-agent", 0, NULL, 'p' }, + { "guest-agent", 0, NULL, 'g' }, { "iforward", 0, NULL, 'i' }, { "oforward", 0, NULL, 'o' }, { "channel", 0, NULL, 'c' }, @@ -549,10 +585,13 @@ int main(int argc, char **argv) QTAILQ_INIT(&iforwards); QTAILQ_INIT(&oforwards); QTAILQ_INIT(&channels); + VPData *guest_agent_iforward = NULL; + VPData *host_agent_iforward = NULL; while ((ch = getopt_long(argc, argv, sopt, lopt, &opt_ind)) != -1) { QemuOpts *opts; VPData *data; + char optarg_tmp[VP_ARG_LEN]; switch (ch) { case 'i': opts = qemu_opts_create(&vp_opts, NULL, 0); @@ -584,6 +623,50 @@ int main(int argc, char **argv) data->opts = opts; QTAILQ_INSERT_TAIL(&channels, data, next); break; + case 'g': + /* create pre-baked iforward for guest agent */ + if (guest_agent_iforward) { + errx(EXIT_FAILURE, "only one --guest-agent argument allowed"); + } + opts = qemu_opts_create(&vp_opts, NULL, 0); + if (optarg == 0) { + sprintf(optarg_tmp, "%s:%s:-", GUEST_AGENT_SERVICE_ID, + GUEST_AGENT_PATH); + } else { + sprintf(optarg_tmp, "%s:%s:-:%d", GUEST_AGENT_SERVICE_ID, + GUEST_AGENT_PATH, atoi(optarg)); + } + ret = vp_parse(opts, optarg_tmp, 0); + if (ret) { + errx(EXIT_FAILURE, "error parsing arg: %s", optarg); + } + data = qemu_mallocz(sizeof(VPData)); + data->opts = opts; + QTAILQ_INSERT_TAIL(&iforwards, data, next); + guest_agent_iforward = data; + break; + case 'p': + /* create pre-baked iforward for host agent */ + if (host_agent_iforward) { + errx(EXIT_FAILURE, "only one --host-agent argument allowed"); + } + opts = qemu_opts_create(&vp_opts, NULL, 0); + if (optarg == 0) { + sprintf(optarg_tmp, "%s:%s:-", HOST_AGENT_SERVICE_ID, + HOST_AGENT_PATH); + } else { + sprintf(optarg_tmp, "%s:%s:-:%d", HOST_AGENT_SERVICE_ID, + HOST_AGENT_PATH, atoi(optarg)); + } + ret = vp_parse(opts, optarg_tmp, 0); + if (ret) { + errx(EXIT_FAILURE, "error parsing arg: %s", optarg); + } + data = qemu_mallocz(sizeof(VPData)); + data->opts = opts; + QTAILQ_INSERT_TAIL(&iforwards, data, next); + host_agent_iforward = data; + break; case 'v': verbose_enabled = 1; break; @@ -613,6 +696,20 @@ int main(int argc, char **argv) "error initializing service mappings for incoming connections"); } + if (guest_agent_iforward) { + ret = init_agent(guest_agent_iforward, false); + if (ret) { + errx(EXIT_FAILURE, + "error initializing guest agent"); + } + } else if (host_agent_iforward) { + ret = init_agent(host_agent_iforward, true); + if (ret) { + errx(EXIT_FAILURE, + "error initializing host agent"); + } + } + /* main i/o loop */ for (;;) { DEBUG("entering main_loop_wait()"); -- 1.7.0.4