Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/99638/?format=api
{ "id": 99638, "url": "http://patchwork.dpdk.org/api/patches/99638/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20210924155740.20510-4-olivier.matz@6wind.com/", "project": { "id": 1, "url": "http://patchwork.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": "<20210924155740.20510-4-olivier.matz@6wind.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20210924155740.20510-4-olivier.matz@6wind.com", "date": "2021-09-24T15:57:38", "name": "[3/5] kvargs: new function to get from key and value", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "4fc00283cd352c281c0d0f83c9d1d161b9a84115", "submitter": { "id": 8, "url": "http://patchwork.dpdk.org/api/people/8/?format=api", "name": "Olivier Matz", "email": "olivier.matz@6wind.com" }, "delegate": { "id": 24651, "url": "http://patchwork.dpdk.org/api/users/24651/?format=api", "username": "dmarchand", "first_name": "David", "last_name": "Marchand", "email": "david.marchand@redhat.com" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/20210924155740.20510-4-olivier.matz@6wind.com/mbox/", "series": [ { "id": 19147, "url": "http://patchwork.dpdk.org/api/series/19147/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=19147", "date": "2021-09-24T15:57:35", "name": "kvargs: promote or remove experimental api", "version": 1, "mbox": "http://patchwork.dpdk.org/series/19147/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/99638/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/99638/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 043DBA0548;\n\tFri, 24 Sep 2021 17:58:27 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id F3E1541340;\n\tFri, 24 Sep 2021 17:58:11 +0200 (CEST)", "from mail-wr1-f51.google.com (mail-wr1-f51.google.com\n [209.85.221.51]) by mails.dpdk.org (Postfix) with ESMTP id C726B41329\n for <dev@dpdk.org>; Fri, 24 Sep 2021 17:58:08 +0200 (CEST)", "by mail-wr1-f51.google.com with SMTP id t8so28972228wrq.4\n for <dev@dpdk.org>; Fri, 24 Sep 2021 08:58:08 -0700 (PDT)", "from gojira.dev.6wind.com (host.78.145.23.62.rev.coltfrance.com.\n [62.23.145.78])\n by smtp.gmail.com with ESMTPSA id a75sm9280829wmd.4.2021.09.24.08.58.07\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Fri, 24 Sep 2021 08:58:08 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind.com; s=google;\n h=from:to:cc:subject:date:message-id:in-reply-to:references\n :mime-version:content-transfer-encoding;\n bh=mG4OZqEBoW7KZ2M7s7ssYRmQvKyO3lGkA6SRDKo6ft0=;\n b=fbxWlUKYeU6HA49+hQobP0J2hcUV3xOaHQoYOjJRMo8Emfi8UsxjuwaFcl6DCsRD/+\n 8aeH86hyELUwQiJUZK5mBgaq0ywXsbYJGDl2nTWO71v5hqnNtdkwNjF0GsXrl+X9qb7o\n TYRFplWeHXz/N83auuKHa7lFTJTf9K/1xqvM8STl0uB1zz1r9m1Xeb7ZBDQqpxd4ye7P\n ySxyHYLIe3dGh+casZ8kaNCn90HWcBsV3h2end0GNBPj9wop166LCgRpi8rl6VTbRXJ+\n 0m9rtzowZj01D3JqHjOwsbiFv4G4wO5IRI9CJ0xLJY9VZCdnFCm36xWbNNCd2UmegbSY\n 52XQ==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20210112;\n h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n :references:mime-version:content-transfer-encoding;\n bh=mG4OZqEBoW7KZ2M7s7ssYRmQvKyO3lGkA6SRDKo6ft0=;\n b=KcW/6vi0LwJmgMqvofvAf4Yx3nEnfBW3jw7NSsu2EU24IGHCqxburkNOzqXUTuVnLa\n tLZuQ5+FM3BYGv2KpJtLc/15fSnP9/NYVv6hCi2ZznYmT3laygjIUui0z/ueNaqWvwuZ\n w6nCzMhKWOsdaDZgbC/bWOT296hkpE94TDcQ+jyp8GL8pJ5mUOIuiUQ1F0egKbkFU4c8\n 7WloGQnz6dmS4FYCCovdMj3cRYMC5ouyQwxrkbNpZ66q82V8q1p1Bd2WJzocbjmCnCTx\n tPV127KSuw0uQYcKlAhdKj4ZA20+WPuQdudVUHXsWZCuWdEmyrn97H3uw0souN0sQ7ZP\n MVTg==", "X-Gm-Message-State": "AOAM533B3dwDLgdEDNhs88wVcjV1Y1GVhmNn9DmEcVNcfGeH3KCVZaqt\n hDJd0QC6/R9zaS3qnJ5esLbrpCMkzQBPmA==", "X-Google-Smtp-Source": "\n ABdhPJzNLJf62OA7W33G7X7uAlyiUcor+i7TQyUVCsSYsHVUjKLK6W88a7kn+xzGeVQjq/3sFn6RCQ==", "X-Received": "by 2002:adf:f4c6:: with SMTP id h6mr1720823wrp.397.1632499088530;\n Fri, 24 Sep 2021 08:58:08 -0700 (PDT)", "From": "Olivier Matz <olivier.matz@6wind.com>", "To": "dev@dpdk.org", "Cc": "Olivier Matz <olivier.matz@6wind.com>,\n Bruce Richardson <bruce.richardson@intel.com>,\n Xueming Li <xuemingl@nvidia.com>, Gaetan Rivet <grive@u256.net>,\n Ray Kinsella <mdr@ashroe.eu>", "Date": "Fri, 24 Sep 2021 17:57:38 +0200", "Message-Id": "<20210924155740.20510-4-olivier.matz@6wind.com>", "X-Mailer": "git-send-email 2.30.2", "In-Reply-To": "<20210924155740.20510-1-olivier.matz@6wind.com>", "References": "<20210924155740.20510-1-olivier.matz@6wind.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH 3/5] kvargs: new function to get from key and\n value", "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": "A quite common scenario with kvargs is to lookup for a <key>=<value> in\na kvlist. For instance, check if name=foo is present in\nname=toto,name=foo,name=bar. This is currently done in drivers/bus with\nrte_kvargs_process() + the rte_kvargs_strcmp() handler.\n\nThis approach is not straightforward, and can be replaced by this new\nfunction.\n\nSigned-off-by: Olivier Matz <olivier.matz@6wind.com>\n---\n drivers/bus/auxiliary/auxiliary_params.c | 9 ++++----\n drivers/bus/vdev/vdev_params.c | 13 ++---------\n lib/kvargs/rte_kvargs.c | 23 +++++++++++++++----\n lib/kvargs/rte_kvargs.h | 29 +++++++++++++++++++++++-\n lib/kvargs/version.map | 2 ++\n 5 files changed, 54 insertions(+), 22 deletions(-)", "diff": "diff --git a/drivers/bus/auxiliary/auxiliary_params.c b/drivers/bus/auxiliary/auxiliary_params.c\nindex cd3fa56cb4..a9c7853ed1 100644\n--- a/drivers/bus/auxiliary/auxiliary_params.c\n+++ b/drivers/bus/auxiliary/auxiliary_params.c\n@@ -25,13 +25,12 @@ auxiliary_dev_match(const struct rte_device *dev,\n \t const void *_kvlist)\n {\n \tconst struct rte_kvargs *kvlist = _kvlist;\n-\tint ret;\n+\tconst char *key = auxiliary_params_keys[RTE_AUXILIARY_PARAM_NAME];\n \n-\tret = rte_kvargs_process(kvlist,\n-\t\t\tauxiliary_params_keys[RTE_AUXILIARY_PARAM_NAME],\n-\t\t\trte_kvargs_strcmp, (void *)(uintptr_t)dev->name);\n+\tif (rte_kvargs_get_with_value(kvlist, key, dev->name) == NULL)\n+\t\treturn -1;\n \n-\treturn ret != 0 ? -1 : 0;\n+\treturn 0;\n }\n \n void *\ndiff --git a/drivers/bus/vdev/vdev_params.c b/drivers/bus/vdev/vdev_params.c\nindex 6f74704d1c..37d95395e7 100644\n--- a/drivers/bus/vdev/vdev_params.c\n+++ b/drivers/bus/vdev/vdev_params.c\n@@ -26,19 +26,10 @@ static int\n vdev_dev_match(const struct rte_device *dev,\n \t const void *_kvlist)\n {\n-\tint ret;\n \tconst struct rte_kvargs *kvlist = _kvlist;\n-\tchar *name;\n+\tconst char *key = vdev_params_keys[RTE_VDEV_PARAM_NAME];\n \n-\t/* cannot pass const dev->name to rte_kvargs_process() */\n-\tname = strdup(dev->name);\n-\tif (name == NULL)\n-\t\treturn -1;\n-\tret = rte_kvargs_process(kvlist,\n-\t\tvdev_params_keys[RTE_VDEV_PARAM_NAME],\n-\t\trte_kvargs_strcmp, name);\n-\tfree(name);\n-\tif (ret != 0)\n+\tif (rte_kvargs_get_with_value(kvlist, key, dev->name) == NULL)\n \t\treturn -1;\n \n \treturn 0;\ndiff --git a/lib/kvargs/rte_kvargs.c b/lib/kvargs/rte_kvargs.c\nindex 38e9d5c1ca..20abb23183 100644\n--- a/lib/kvargs/rte_kvargs.c\n+++ b/lib/kvargs/rte_kvargs.c\n@@ -204,21 +204,34 @@ rte_kvargs_free(struct rte_kvargs *kvlist)\n \tfree(kvlist);\n }\n \n-/* Lookup a value in an rte_kvargs list by its key. */\n+/* Lookup a value in an rte_kvargs list by its key and value. */\n const char *\n-rte_kvargs_get(const struct rte_kvargs *kvlist, const char *key)\n+rte_kvargs_get_with_value(const struct rte_kvargs *kvlist, const char *key,\n+\t\t\t const char *value)\n {\n \tunsigned int i;\n \n-\tif (kvlist == NULL || key == NULL)\n+\tif (kvlist == NULL)\n \t\treturn NULL;\n \tfor (i = 0; i < kvlist->count; ++i) {\n-\t\tif (strcmp(kvlist->pairs[i].key, key) == 0)\n-\t\t\treturn kvlist->pairs[i].value;\n+\t\tif (key != NULL && strcmp(kvlist->pairs[i].key, key) != 0)\n+\t\t\tcontinue;\n+\t\tif (value != NULL && strcmp(kvlist->pairs[i].value, value) != 0)\n+\t\t\tcontinue;\n+\t\treturn kvlist->pairs[i].value;\n \t}\n \treturn NULL;\n }\n \n+/* Lookup a value in an rte_kvargs list by its key. */\n+const char *\n+rte_kvargs_get(const struct rte_kvargs *kvlist, const char *key)\n+{\n+\tif (kvlist == NULL || key == NULL)\n+\t\treturn NULL;\n+\treturn rte_kvargs_get_with_value(kvlist, key, NULL);\n+}\n+\n /*\n * Parse the arguments \"key=value,key=value,...\" string and return\n * an allocated structure that contains a key/value list. Also\ndiff --git a/lib/kvargs/rte_kvargs.h b/lib/kvargs/rte_kvargs.h\nindex 328f0d3cc6..e40df1bd2a 100644\n--- a/lib/kvargs/rte_kvargs.h\n+++ b/lib/kvargs/rte_kvargs.h\n@@ -116,7 +116,7 @@ void rte_kvargs_free(struct rte_kvargs *kvlist);\n /**\n * Get the value associated with a given key.\n *\n- * If multiple key matches, the value of the first one is returned.\n+ * If multiple keys match, the value of the first one is returned.\n *\n * The memory returned is allocated as part of the rte_kvargs structure,\n * it must never be modified.\n@@ -132,6 +132,33 @@ void rte_kvargs_free(struct rte_kvargs *kvlist);\n */\n const char *rte_kvargs_get(const struct rte_kvargs *kvlist, const char *key);\n \n+/**\n+ * @warning\n+ * @b EXPERIMENTAL: this API may change without prior notice\n+ *\n+ * Get the value associated with a given key and value.\n+ *\n+ * Find the first entry in the kvlist whose key and value match the\n+ * ones passed as argument.\n+ *\n+ * The memory returned is allocated as part of the rte_kvargs structure,\n+ * it must never be modified.\n+ *\n+ * @param kvlist\n+ * A list of rte_kvargs pair of 'key=value'.\n+ * @param key\n+ * The matching key. If NULL, any key will match.\n+ * @param value\n+ * The matching value. If NULL, any value will match.\n+ *\n+ * @return\n+ * NULL if no key matches the input,\n+ * a value associated with a matching key otherwise.\n+ */\n+__rte_experimental\n+const char *rte_kvargs_get_with_value(const struct rte_kvargs *kvlist,\n+\t\t\t\t const char *key, const char *value);\n+\n /**\n * Call a handler function for each key/value matching the key\n *\ndiff --git a/lib/kvargs/version.map b/lib/kvargs/version.map\nindex 236f35c02b..82879b7140 100644\n--- a/lib/kvargs/version.map\n+++ b/lib/kvargs/version.map\n@@ -16,4 +16,6 @@ EXPERIMENTAL {\n \n \trte_kvargs_strcmp;\n \n+\t# added in 21.11\n+\trte_kvargs_get_with_value;\n };\n", "prefixes": [ "3/5" ] }