From patchwork Mon Sep 11 17:14:54 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Oleksandr Grytsov X-Patchwork-Id: 9947771 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id DAC20603F4 for ; Mon, 11 Sep 2017 17:18:08 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id CA80628CD0 for ; Mon, 11 Sep 2017 17:18:08 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id BF3FE28CD4; Mon, 11 Sep 2017 17:18:08 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.6 required=2.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, RCVD_IN_DNSWL_MED, RCVD_IN_SORBS_SPAM, T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id E497E28CD0 for ; Mon, 11 Sep 2017 17:18:07 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1drSJ1-0001y1-Cf; Mon, 11 Sep 2017 17:15:31 +0000 Received: from mail6.bemta6.messagelabs.com ([193.109.254.103]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1drSIz-0001vj-Ca for xen-devel@lists.xenproject.org; Mon, 11 Sep 2017 17:15:29 +0000 Received: from [193.109.254.147] by server-11.bemta-6.messagelabs.com id DA/84-03616-0B4C6B95; Mon, 11 Sep 2017 17:15:28 +0000 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrFIsWRWlGSWpSXmKPExsVyMfS6o+7qI9s iDQ4dULH4vmUykwOjx+EPV1gCGKNYM/OS8isSWDOmHVvBXrAxruLk0u+sDYwvXboYuTiEBGYw StxuO8MC4rAIvGSRaPw4nx3EkRDoZ5U4uuo/WxcjJ5ATJ7H/8zl2CDtN4sLufUwQdoXEvVuLG UFsIQF5iYMtl9khxk5jkvg8s40ZJMEmoCVx//YKsEEiAkoS91ZNBmtmFkiR2LuuDaxZWMBGYu asHUDNHEBnqEr0ftUACfMKOEjM+DOTGWKXnMTNc51gNqeAo8SSKV2sEHsdJI4872SfwCi4gJF hFaNGcWpRWWqRrqGJXlJRZnpGSW5iZo6uoYGZXm5qcXFiempOYlKxXnJ+7iZGYMgxAMEOxusb Aw4xSnIwKYnyvju+JVKILyk/pTIjsTgjvqg0J7X4EKMGB4fAz6d/WxilWPLy81KVJHgND2+LF BIsSk1PrUjLzAFGBUypBAePkghvDkiat7ggMbc4Mx0idYrRnuPCnUt/mDiObboMJA/suQUkO2 7e/cMkBDZVSpyXG6RNAKQtozQPbigsii8xykoJ8zICnSzEU5BalJtZgir/ilGcg1FJGOI2nsy 8Erjdr4DOYgI6i+fSFpCzShIRUlINjA2L3MSObtb4t+/LNlfj8KMXv8Q+k909O32xRlHxhf1v r653e3BgudGk6AVbPC9udVqnsD+5e2nlsalvts+PKZTi/8i1bmvsjQXSP2SZRdjMDq9/1cG// vWOjpVlmS+XPbxdkjTvwMxcI6U9nTzH1ZoOppps9hDRe/Z7ufIHk/7I2/m3Jx+875OkxFKckW ioxVxUnAgAnZuuw90CAAA= X-Env-Sender: al1img@gmail.com X-Msg-Ref: server-7.tower-27.messagelabs.com!1505150123!111613856!1 X-Originating-IP: [209.85.215.65] X-SpamReason: No, hits=0.0 required=7.0 tests= X-StarScan-Received: X-StarScan-Version: 9.4.45; banners=-,-,- X-VirusChecked: Checked Received: (qmail 4357 invoked from network); 11 Sep 2017 17:15:23 -0000 Received: from mail-lf0-f65.google.com (HELO mail-lf0-f65.google.com) (209.85.215.65) by server-7.tower-27.messagelabs.com with AES128-GCM-SHA256 encrypted SMTP; 11 Sep 2017 17:15:23 -0000 Received: by mail-lf0-f65.google.com with SMTP id q132so4222820lfe.4 for ; Mon, 11 Sep 2017 10:15:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=OvOXUNKfWpE2aT9ka7y7TBJ6UOhLJSGFh1jyaiU24kE=; b=j1BKI2uhX8LbuZH5N/kidADB2FEymPpG4v+7oj4BcOjuwM/rQ+WJ1DyWoqPJcP4GaM vHEiXJOCfrRJnucLCp3mCOe0yQWHFm6elYZHTkfQlErYPgvHEyjl5X4+xW3nmHNiANbo xV9Y2LEpa8JsZjiXZX8Wwr89MUwJjeSwoUQuvhTM3NrNkLTl02NxHkvm0p3gWC4BsYyd NDodvtF6IMGyrhv9co/Ycuma2xaZxnl08XhexpC3QQgZLpk0HfGLH/VLS2Dm+S4s9puo 7iasIweowXKxknulDhkMynkavpEYgfzeUhGUB0lRLXQ+BckwzoLneS+aD/5RzVyRmdd+ LBNw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=OvOXUNKfWpE2aT9ka7y7TBJ6UOhLJSGFh1jyaiU24kE=; b=nNWQgq2CVVifqa7M03RH9ROB0q9Q544kd3M5O9r/ZZFFf+TrJKXgKjBgk95IT557Iq PgwplOwuJ2zo3pfy+eKwJc/F6/bElH/BXrhgtssfJ4OYxxoDSzF3s0RA3fnmd0CXz3LH J6H6+1zWBQNaYadUAkwi/1/LsxoA3m1laXVPbSlt23jLyaRycLIssQU3u7G0o418cHP5 fI83tE9xXfAdZ/YfwTkduuChTxOQvq+d24e0mLonJiJ8xGi9gZ3zN4H8XsnBs56SV9vv GduiG5bVyZo8XyCSyr0cJiHYgimE6wNFioKEGySi8oG9aR1pe2sxZfj8Gx7YuvYeLmaY judg== X-Gm-Message-State: AHPjjUiC/x3gZYrF8tQtBSUCVaOH22Y6tCwZ4aMgiVaIuNNUzXCARNin NNIzPYkMoEOT6K2o X-Google-Smtp-Source: AOwi7QD2mTaY2EISNARen8AYMNLx1ksz53NqVsIVe9/5rjaUCUbKO6zQl3XAEitu7X7g182NoibMlg== X-Received: by 10.46.23.25 with SMTP id l25mr1140539lje.178.1505150122399; Mon, 11 Sep 2017 10:15:22 -0700 (PDT) Received: from al1-pc.kyiv.epam.com (ll-57.209.223.85.sovam.net.ua. [85.223.209.57]) by smtp.gmail.com with ESMTPSA id g21sm1893044lje.52.2017.09.11.10.15.21 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 11 Sep 2017 10:15:21 -0700 (PDT) From: Oleksandr Grytsov To: xen-devel@lists.xenproject.org Date: Mon, 11 Sep 2017 20:14:54 +0300 Message-Id: <1505150102-30806-5-git-send-email-al1img@gmail.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1505150102-30806-1-git-send-email-al1img@gmail.com> References: <1505150102-30806-1-git-send-email-al1img@gmail.com> Cc: ian.jackson@eu.citrix.com, wei.liu2@citrix.com, Oleksandr Grytsov Subject: [Xen-devel] [PATCH v5 04/12] xl: add PV display device commands X-BeenThere: xen-devel@lists.xen.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" X-Virus-Scanned: ClamAV using ClamSMTP From: Oleksandr Grytsov Add commands: vdispl-attach, vdispl-list, vdispl-detach and domain config vdispl parser Signed-off-by: Oleksandr Grytsov Acked-by: Wei Liu --- tools/xl/Makefile | 1 + tools/xl/xl.h | 3 + tools/xl/xl_cmdtable.c | 19 ++++++ tools/xl/xl_parse.c | 75 ++++++++++++++++++++++- tools/xl/xl_parse.h | 2 +- tools/xl/xl_vdispl.c | 163 +++++++++++++++++++++++++++++++++++++++++++++++++ 6 files changed, 261 insertions(+), 2 deletions(-) create mode 100644 tools/xl/xl_vdispl.c diff --git a/tools/xl/Makefile b/tools/xl/Makefile index c868899..41a09ed 100644 --- a/tools/xl/Makefile +++ b/tools/xl/Makefile @@ -21,6 +21,7 @@ XL_OBJS += xl_vtpm.o xl_block.o xl_nic.o xl_usb.o XL_OBJS += xl_sched.o xl_pci.o xl_vcpu.o xl_cdrom.o xl_mem.o XL_OBJS += xl_psr.o xl_info.o xl_console.o xl_misc.o XL_OBJS += xl_vmcontrol.o xl_saverestore.o xl_migrate.o +XL_OBJS += xl_vdispl.o $(XL_OBJS): CFLAGS += $(CFLAGS_libxentoollog) $(XL_OBJS): CFLAGS += $(CFLAGS_XL) diff --git a/tools/xl/xl.h b/tools/xl/xl.h index 5d3d2a4..98d62e9 100644 --- a/tools/xl/xl.h +++ b/tools/xl/xl.h @@ -167,6 +167,9 @@ int main_blockdetach(int argc, char **argv); int main_vtpmattach(int argc, char **argv); int main_vtpmlist(int argc, char **argv); int main_vtpmdetach(int argc, char **argv); +int main_vdisplattach(int argc, char **argv); +int main_vdispllist(int argc, char **argv); +int main_vdispldetach(int argc, char **argv); int main_usbctrl_attach(int argc, char **argv); int main_usbctrl_detach(int argc, char **argv); int main_usbdev_attach(int argc, char **argv); diff --git a/tools/xl/xl_cmdtable.c b/tools/xl/xl_cmdtable.c index ba0159d..d0330cd 100644 --- a/tools/xl/xl_cmdtable.c +++ b/tools/xl/xl_cmdtable.c @@ -377,6 +377,25 @@ struct cmd_spec cmd_table[] = { "Destroy a domain's virtual TPM device", " ", }, + { "vdispl-attach", + &main_vdisplattach, 1, 1, + "Create a new virtual display device", + " [backend=] [be-alloc=] [connectors='']", + " BackAlloc - set to 1 to if backend allocates display buffers\n" + " Connectors - list of connector's description in ID:WxH format,\n" + " where: ID - unique connector ID, W - connector width,\n" + " H - connector height: connectors='id0:800x600;id1:1024x768'\n" + }, + { "vdispl-list", + &main_vdispllist, 0, 0, + "List virtual display devices for a domain", + "", + }, + { "vdispl-detach", + &main_vdispldetach, 0, 1, + "Destroy a domain's virtual display device", + " ", + }, { "uptime", &main_uptime, 0, 0, "Print uptime for all/some domains", diff --git a/tools/xl/xl_parse.c b/tools/xl/xl_parse.c index 02ddd2e..9965b83 100644 --- a/tools/xl/xl_parse.c +++ b/tools/xl/xl_parse.c @@ -804,6 +804,51 @@ int parse_usbdev_config(libxl_device_usbdev *usbdev, char *token) return 0; } +int parse_vdispl_config(libxl_device_vdispl *vdispl, char *token) +{ + char *oparg; + libxl_string_list connectors = NULL; + int i; + int rc; + + if (MATCH_OPTION("backend", token, oparg)) { + vdispl->backend_domname = strdup(oparg); + } else if (MATCH_OPTION("be-alloc", token, oparg)) { + vdispl->be_alloc = strtoul(oparg, NULL, 0); + } else if (MATCH_OPTION("connectors", token, oparg)) { + split_string_into_string_list(oparg, ";", &connectors); + + vdispl->num_connectors = libxl_string_list_length(&connectors); + vdispl->connectors = calloc(vdispl->num_connectors, + sizeof(*vdispl->connectors)); + + for(i = 0; i < vdispl->num_connectors; i++) + { + char *resolution; + + rc = split_string_into_pair(connectors[i], ":", + &vdispl->connectors[i].id, + &resolution); + + rc= sscanf(resolution, "%ux%u", &vdispl->connectors[i].width, + &vdispl->connectors[i].height); + if (rc != 2) { + fprintf(stderr, "Can't parse connector resolution\n"); + goto out; + } + } + } else { + fprintf(stderr, "Unknown string \"%s\" in vdispl spec\n", token); + rc = 1; goto out; + } + + rc = 0; + +out: + libxl_string_list_dispose(&connectors); + return rc; +} + void parse_config_data(const char *config_source, const char *config_data, int config_len, @@ -813,7 +858,7 @@ void parse_config_data(const char *config_source, long l, vcpus = 0; XLU_Config *config; XLU_ConfigList *cpus, *vbds, *nics, *pcis, *cvfbs, *cpuids, *vtpms, - *usbctrls, *usbdevs, *p9devs; + *usbctrls, *usbdevs, *p9devs, *vdispls; XLU_ConfigList *channels, *ioports, *irqs, *iomem, *viridian, *dtdevs, *mca_caps; int num_ioports, num_irqs, num_iomem, num_cpus, num_viridian, num_mca_caps; @@ -1489,6 +1534,34 @@ void parse_config_data(const char *config_source, } } + if (!xlu_cfg_get_list(config, "vdispl", &vdispls, 0, 0)) { + d_config->num_vdispls = 0; + d_config->vdispls = NULL; + while ((buf = xlu_cfg_get_listitem(vdispls, d_config->num_vdispls)) != NULL) { + libxl_device_vdispl *vdispl; + char * buf2 = strdup(buf); + char *p; + vdispl = ARRAY_EXTEND_INIT(d_config->vdispls, + d_config->num_vdispls, + libxl_device_vdispl_init); + p = strtok (buf2, ","); + while (p != NULL) + { + while (*p == ' ') p++; + if (parse_vdispl_config(vdispl, p)) { + free(buf2); + exit(1); + } + p = strtok (NULL, ","); + } + free(buf2); + if (vdispl->num_connectors == 0) { + fprintf(stderr, "At least one connector should be specified.\n"); + exit(1); + } + } + } + if (!xlu_cfg_get_list (config, "channel", &channels, 0, 0)) { d_config->num_channels = 0; d_config->channels = NULL; diff --git a/tools/xl/xl_parse.h b/tools/xl/xl_parse.h index db8bc3f..cc459fb 100644 --- a/tools/xl/xl_parse.h +++ b/tools/xl/xl_parse.h @@ -33,7 +33,7 @@ int parse_usbctrl_config(libxl_device_usbctrl *usbctrl, char *token); int parse_usbdev_config(libxl_device_usbdev *usbdev, char *token); int parse_cpurange(const char *cpu, libxl_bitmap *cpumap); int parse_nic_config(libxl_device_nic *nic, XLU_Config **config, char *token); - +int parse_vdispl_config(libxl_device_vdispl *vdispl, char *token); int match_option_size(const char *prefix, size_t len, char *arg, char **argopt); diff --git a/tools/xl/xl_vdispl.c b/tools/xl/xl_vdispl.c new file mode 100644 index 0000000..3cc99b6 --- /dev/null +++ b/tools/xl/xl_vdispl.c @@ -0,0 +1,163 @@ +/* + * Copyright (C) 2016 EPAM Systems Inc. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as published + * by the Free Software Foundation; version 2.1 only. with the special + * exception on linking described in file LICENSE. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Lesser General Public License for more details. + */ + +#include + +#include +#include +#include + +#include "xl.h" +#include "xl_utils.h" +#include "xl_parse.h" + +int main_vdisplattach(int argc, char **argv) +{ + int opt; + int rc; + uint32_t domid; + libxl_device_vdispl vdispl; + + SWITCH_FOREACH_OPT(opt, "", NULL, "vdispl-attach", 1) { + /* No options */ + } + + libxl_device_vdispl_init(&vdispl); + domid = find_domain(argv[optind++]); + + for (argv += optind, argc -= optind; argc > 0; ++argv, --argc) { + rc = parse_vdispl_config(&vdispl, *argv); + if (rc) goto out; + } + + if (vdispl.num_connectors == 0) { + fprintf(stderr, "At least one connector should be specified.\n"); + rc = ERROR_FAIL; goto out; + } + + if (dryrun_only) { + char *json = libxl_device_vdispl_to_json(ctx, &vdispl); + printf("vdispl: %s\n", json); + free(json); + goto out; + } + + if (libxl_device_vdispl_add(ctx, domid, &vdispl, 0)) { + fprintf(stderr, "libxl_device_vdispl_add failed.\n"); + rc = ERROR_FAIL; goto out; + } + + rc = 0; + +out: + libxl_device_vdispl_dispose(&vdispl); + return rc; +} + +int main_vdispllist(int argc, char **argv) +{ + int opt; + int i, j, n; + libxl_device_vdispl *vdispls; + libxl_vdisplinfo vdisplinfo; + + SWITCH_FOREACH_OPT(opt, "", NULL, "vdispl-list", 1) { + /* No options */ + } + + for (argv += optind, argc -= optind; argc > 0; --argc, ++argv) { + uint32_t domid; + + if (libxl_domain_qualifier_to_domid(ctx, *argv, &domid) < 0) { + fprintf(stderr, "%s is an invalid domain identifier\n", *argv); + continue; + } + + vdispls = libxl_device_vdispl_list(ctx, domid, &n); + + if (!vdispls) continue; + + for (i = 0; i < n; i++) { + libxl_vdisplinfo_init(&vdisplinfo); + if (libxl_device_vdispl_getinfo(ctx, domid, &vdispls[i], + &vdisplinfo) == 0) { + printf("DevId: %d, BE: %d, handle: %d, state: %d, " + "be-alloc: %d, BE-path: %s, FE-path: %s\n", + vdisplinfo.devid, vdisplinfo.backend_id, + vdisplinfo.frontend_id, + vdisplinfo.state, vdisplinfo.be_alloc, + vdisplinfo.backend, vdisplinfo.frontend); + + for (j = 0; j < vdisplinfo.num_connectors; j++) { + printf("\tConnector: %d, id: %s, width: %d, height: %d, " + "req-rref: %d, req-evtch: %d, " + "evt-rref: %d, evt-evtch: %d\n", + j, vdisplinfo.connectors[j].id, + vdisplinfo.connectors[j].width, + vdisplinfo.connectors[j].height, + vdisplinfo.connectors[j].req_rref, + vdisplinfo.connectors[j].req_evtch, + vdisplinfo.connectors[j].evt_rref, + vdisplinfo.connectors[j].evt_evtch); + } + } + libxl_vdisplinfo_dispose(&vdisplinfo); + } + libxl_device_vdispl_list_free(vdispls, n); + } + return 0; +} + +int main_vdispldetach(int argc, char **argv) +{ + uint32_t domid, devid; + int opt, rc; + libxl_device_vdispl vdispl; + + SWITCH_FOREACH_OPT(opt, "", NULL, "vdispl-detach", 2) { + /* No options */ + } + + domid = find_domain(argv[optind++]); + devid = atoi(argv[optind++]); + + libxl_device_vdispl_init(&vdispl); + + if (libxl_devid_to_device_vdispl(ctx, domid, devid, &vdispl)) { + fprintf(stderr, "Error: Device %d not connected.\n", devid); + rc = ERROR_FAIL; + goto out; + } + + rc = libxl_device_vdispl_remove(ctx, domid, &vdispl, 0); + if (rc) { + fprintf(stderr, "libxl_device_vdispl_remove failed.\n"); + rc = ERROR_FAIL; + goto out; + } + + rc = 0; + +out: + libxl_device_vdispl_dispose(&vdispl); + return rc; +} + +/* + * Local variables: + * mode: C + * c-basic-offset: 4 + * indent-tabs-mode: nil + * End: + */