Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/88476/?format=api
{ "id": 88476, "url": "http://patches.dpdk.org/api/patches/88476/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/1614868228-13685-7-git-send-email-xuemingl@nvidia.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<1614868228-13685-7-git-send-email-xuemingl@nvidia.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1614868228-13685-7-git-send-email-xuemingl@nvidia.com", "date": "2021-03-04T14:30:22", "name": "[v8,6/9] ethdev: support multi-host in representor", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "b33b59848ba6d4b20b167faa69d27d1638edfbab", "submitter": { "id": 1904, "url": "http://patches.dpdk.org/api/people/1904/?format=api", "name": "Xueming Li", "email": "xuemingl@nvidia.com" }, "delegate": { "id": 319, "url": "http://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/1614868228-13685-7-git-send-email-xuemingl@nvidia.com/mbox/", "series": [ { "id": 15491, "url": "http://patches.dpdk.org/api/series/15491/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=15491", "date": "2021-03-04T14:30:16", "name": "ethdev: support SubFunction representor", "version": 8, "mbox": "http://patches.dpdk.org/series/15491/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/88476/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/88476/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 40636A0561;\n\tThu, 4 Mar 2021 15:31:54 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id ADF2322A300;\n\tThu, 4 Mar 2021 15:31:18 +0100 (CET)", "from mellanox.co.il (mail-il-dmz.mellanox.com [193.47.165.129])\n by mails.dpdk.org (Postfix) with ESMTP id 3B94C22A2FB\n for <dev@dpdk.org>; Thu, 4 Mar 2021 15:31:16 +0100 (CET)", "from Internal Mail-Server by MTLPINE1 (envelope-from\n xuemingl@nvidia.com) with SMTP; 4 Mar 2021 16:31:11 +0200", "from nvidia.com (pegasus05.mtr.labs.mlnx [10.210.16.100])\n by labmailer.mlnx (8.13.8/8.13.8) with ESMTP id 124EUkoU027668;\n Thu, 4 Mar 2021 16:31:10 +0200" ], "From": "Xueming Li <xuemingl@nvidia.com>", "To": "Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>", "Cc": "dev@dpdk.org, Viacheslav Ovsiienko <viacheslavo@nvidia.com>,\n xuemingl@nvidia.com, Asaf Penso <asafp@nvidia.com>,\n Bruce Richardson <bruce.richardson@intel.com>,\n Thomas Monjalon <thomas@monjalon.net>,\n Ferruh Yigit <ferruh.yigit@intel.com>", "Date": "Thu, 4 Mar 2021 14:30:22 +0000", "Message-Id": "<1614868228-13685-7-git-send-email-xuemingl@nvidia.com>", "X-Mailer": "git-send-email 1.8.3.1", "In-Reply-To": [ "<1614868228-13685-1-git-send-email-xuemingl@nvidia.com>", "<1608303356-13089-2-git-send-email-xuemingl@nvidia.com>" ], "References": [ "<1614868228-13685-1-git-send-email-xuemingl@nvidia.com>", "<1608303356-13089-2-git-send-email-xuemingl@nvidia.com>" ], "Subject": "[dpdk-dev] [PATCH v8 6/9] ethdev: support multi-host in representor", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "The NIC can have multiple PCIe links and can be attached to the multiple\nhosts, for example the same single NIC can be shared for multiple server\nunits in the rack. On each PCIe link NIC can provide multiple PFs and\nVFs/SFs based on these ones. To provide the unambiguous identification\nof the PCIe function the controller index is added. The full representor\nidentifier consists of three indices - controller index, PF index, and\nVF or SF index (if any).\n\nThis patch introduces controller index to ethdev representor syntax,\nexamples:\n\n[[c#]pf#]vf#: VF port representor/s, example: pf0vf1\n[[c#]pf#]sf#: SF port representor/s, example: c1pf1sf[0-3]\n\nc# is controller(host) ID/range in case of multi-host, optional.\n\nFor user application (e.g. OVS), PMD is responsible to interpret and\nlocate representor device based on controller ID, PF ID and VF/SF ID in\nrepresentor syntax.\n\nSigned-off-by: Xueming Li <xuemingl@nvidia.com>\nAcked-by: Viacheslav Ovsiienko <viacheslavo@nvidia.com>\nAcked-by: Thomas Monjalon <thomas@monjalon.net>\nAcked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>\n---\n config/rte_config.h | 1 +\n lib/librte_ethdev/ethdev_driver.h | 4 ++++\n lib/librte_ethdev/ethdev_private.c | 18 +++++++++++++++---\n 3 files changed, 20 insertions(+), 3 deletions(-)", "diff": "diff --git a/config/rte_config.h b/config/rte_config.h\nindex 55a2fc50ed..904a40b3c1 100644\n--- a/config/rte_config.h\n+++ b/config/rte_config.h\n@@ -57,6 +57,7 @@\n #define RTE_MAX_QUEUES_PER_PORT 1024\n #define RTE_ETHDEV_QUEUE_STAT_CNTRS 16 /* max 256 */\n #define RTE_ETHDEV_RXTX_CALLBACKS 1\n+#define RTE_MAX_MULTI_HOST_CTRLS 4\n \n /* cryptodev defines */\n #define RTE_CRYPTO_MAX_DEVS 64\ndiff --git a/lib/librte_ethdev/ethdev_driver.h b/lib/librte_ethdev/ethdev_driver.h\nindex d68b8401e7..06ff35266f 100644\n--- a/lib/librte_ethdev/ethdev_driver.h\n+++ b/lib/librte_ethdev/ethdev_driver.h\n@@ -1222,6 +1222,10 @@ rte_eth_switch_domain_free(uint16_t domain_id);\n * One type of representor each structure.\n */\n struct rte_eth_devargs {\n+\tuint16_t mh_controllers[RTE_MAX_MULTI_HOST_CTRLS];\n+\t/** controller/s number in case of multi-host */\n+\tuint16_t nb_mh_controllers;\n+\t/** number of controllers in multi-host controllers field */\n \tuint16_t ports[RTE_MAX_ETHPORTS];\n \t/** port/s number to enable on a multi-port single function */\n \tuint16_t nb_ports;\ndiff --git a/lib/librte_ethdev/ethdev_private.c b/lib/librte_ethdev/ethdev_private.c\nindex 35a57c56ec..012cf73ca2 100644\n--- a/lib/librte_ethdev/ethdev_private.c\n+++ b/lib/librte_ethdev/ethdev_private.c\n@@ -118,9 +118,9 @@ rte_eth_devargs_process_list(char *str, uint16_t *list, uint16_t *len_list,\n *\n * Representor format:\n * #: range or single number of VF representor - legacy\n- * [pf#]vf#: VF port representor/s\n- * [pf#]sf#: SF port representor/s\n- * pf#: PF port representor/s\n+ * [[c#]pf#]vf#: VF port representor/s\n+ * [[c#]pf#]sf#: SF port representor/s\n+ * [c#]pf#: PF port representor/s\n *\n * Examples of #:\n * 2 - single\n@@ -132,6 +132,14 @@ rte_eth_devargs_parse_representor_ports(char *str, void *data)\n {\n \tstruct rte_eth_devargs *eth_da = data;\n \n+\tif (str[0] == 'c') {\n+\t\tstr += 1;\n+\t\tstr = rte_eth_devargs_process_list(str, eth_da->mh_controllers,\n+\t\t\t\tð_da->nb_mh_controllers,\n+\t\t\t\tRTE_DIM(eth_da->mh_controllers));\n+\t\tif (str == NULL)\n+\t\t\tgoto done;\n+\t}\n \tif (str[0] == 'p' && str[1] == 'f') {\n \t\teth_da->type = RTE_ETH_REPRESENTOR_PF;\n \t\tstr += 2;\n@@ -139,6 +147,10 @@ rte_eth_devargs_parse_representor_ports(char *str, void *data)\n \t\t\t\tð_da->nb_ports, RTE_DIM(eth_da->ports));\n \t\tif (str == NULL || str[0] == '\\0')\n \t\t\tgoto done;\n+\t} else if (eth_da->nb_mh_controllers > 0) {\n+\t\t/* 'c' must followed by 'pf'. */\n+\t\tstr = NULL;\n+\t\tgoto done;\n \t}\n \tif (str[0] == 'v' && str[1] == 'f') {\n \t\teth_da->type = RTE_ETH_REPRESENTOR_VF;\n", "prefixes": [ "v8", "6/9" ] }