From patchwork Thu Nov 11 01:37:22 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Michael Roth X-Patchwork-Id: 70741 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [199.232.76.165]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 28BCAB7117 for ; Thu, 11 Nov 2010 12:41:25 +1100 (EST) Received: from localhost ([127.0.0.1]:34163 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PGMA0-0000lS-0Z for incoming@patchwork.ozlabs.org; Wed, 10 Nov 2010 20:41:08 -0500 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 Received: from localhost.localdomain (sig-9-65-242-217.mts.ibm.com [9.65.242.217]) by d01av01.pok.ibm.com (8.14.4/8.13.1/NCO v10.0 AVin) with ESMTP id oAB1bY16009666; Wed, 10 Nov 2010 20:37:46 -0500 From: Michael Roth To: qemu-devel@nongnu.org Date: Wed, 10 Nov 2010 19:37:22 -0600 Message-Id: <1289439450-23556-4-git-send-email-mdroth@linux.vnet.ibm.com> X-Mailer: git-send-email 1.7.0.4 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> X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6, seldom 2.4 (older, 4) 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 Subject: [Qemu-devel] [RFC][PATCH v3 03/11] virtagent: qemu-vp, integrate virtagent server X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org 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()");