Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/138984/?format=api
{ "id": 138984, "url": "http://patches.dpdk.org/api/patches/138984/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20240330030429.4630-5-stephen@networkplumber.org/", "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": "<20240330030429.4630-5-stephen@networkplumber.org>", "list_archive_url": "https://inbox.dpdk.org/dev/20240330030429.4630-5-stephen@networkplumber.org", "date": "2024-03-30T03:00:47", "name": "[v19,04/15] windows: common wrapper for vasprintf and asprintf", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "582ccd4ba5558a6005de0f30c96883cf48708a3a", "submitter": { "id": 27, "url": "http://patches.dpdk.org/api/people/27/?format=api", "name": "Stephen Hemminger", "email": "stephen@networkplumber.org" }, "delegate": { "id": 1, "url": "http://patches.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20240330030429.4630-5-stephen@networkplumber.org/mbox/", "series": [ { "id": 31647, "url": "http://patches.dpdk.org/api/series/31647/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=31647", "date": "2024-03-30T03:00:44", "name": "Logging unification and improvements", "version": 19, "mbox": "http://patches.dpdk.org/series/31647/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/138984/comments/", "check": "warning", "checks": "http://patches.dpdk.org/api/patches/138984/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 2FD4743D8B;\n\tSat, 30 Mar 2024 04:05:09 +0100 (CET)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 3DEEB40A6B;\n\tSat, 30 Mar 2024 04:04:50 +0100 (CET)", "from mail-pg1-f171.google.com (mail-pg1-f171.google.com\n [209.85.215.171])\n by mails.dpdk.org (Postfix) with ESMTP id 712C3402EC\n for <dev@dpdk.org>; Sat, 30 Mar 2024 04:04:44 +0100 (CET)", "by mail-pg1-f171.google.com with SMTP id\n 41be03b00d2f7-5d81b08d6f2so1835711a12.0\n for <dev@dpdk.org>; Fri, 29 Mar 2024 20:04:44 -0700 (PDT)", "from hermes.local (204-195-123-203.wavecable.com. [204.195.123.203])\n by smtp.gmail.com with ESMTPSA id\n o3-20020a056a00214300b006e6288ef4besm3655486pfk.54.2024.03.29.20.04.42\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Fri, 29 Mar 2024 20:04:43 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1711767883;\n x=1712372683; darn=dpdk.org;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=YwHfWcdLNfli8f+JFimP9JJamA7SW0tTDfnLjdRpUKc=;\n b=bxw4jAAaqsxGku4lpq45STN5dgeO8yP8V9jfx/LvUZas6ulotOzX8adyCoc+kcdLqb\n itYfgxgUwuoJDOsPjHr0CMVMRJHRAJW5/HYdljyHpHh/AUme/E8p/50vmNakIDXBgTFm\n RY3PDlretEV/dmwF2+3vkq6YxqsRSyE3s7eMruZimTxRZwlvzZ6tU+hHRrf1Y3HHz9oT\n 8kyu2SpPteWiEj7Dmq8/QmvHH94mLbkk5rHdwePRYoYd16GYAkgxsL8UhG89S0SJt6o+\n tEUUfscZutYuj6EHPa5BhgpR9TGFMem2sH6oX8NQGpu8yIyykFuUjE6mYA/IQIioj8so\n gGKg==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20230601; t=1711767883; x=1712372683;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc\n :subject:date:message-id:reply-to;\n bh=YwHfWcdLNfli8f+JFimP9JJamA7SW0tTDfnLjdRpUKc=;\n b=UxZGgaHlFJnrazj5rydLra4nuIYFXWVQSJnIpOf3SLIikWcJzB4SEOl3c+JmqR3Qhv\n OZvPnD3Ptj8XjGJs/np+OGmGVY1Vc/BphPe2Ref7CbzmZawJ6i3MpFIghMb4KZc9YiMY\n /SAQg0XsOEKJhpb437PMbXvuFfJYV64BrnDKh2vhEnAutFCZm6cWgfv44XmSZLewSLBL\n 2XJkpY6EUxQrKgZ8a2cIjLPSnE1+DTaCN5wRRQYoKir2aOxNpDu/GO3s70dczXrTwsZS\n X7NTiaFvueeM+5JSgQdL3R+t54Xx1uB6Z4nadhabx+5AlwaygzmkLpLr3MsXvpiMW28w\n GDtg==", "X-Gm-Message-State": "AOJu0Yx4gvh15SpTY/NU0/Dwg2uRW1E9ZVhps9pUb0mwgaB3QsHvIsz7\n 7rUbYrC6d8ET1j5iMrOtZiBVDR8sTxRYZ0fFXTFsgkLpOCJZNXfTXk3NKCKksccSe6y6B3OHPjm\n U", "X-Google-Smtp-Source": "\n AGHT+IE28d8uac5RJFvvGmRYq0jwFucpohQ8dfz9fxkNIe7Kg94Ds8tWEmLCnfxsBTRHK/TRO79Esw==", "X-Received": "by 2002:a05:6a20:4323:b0:1a3:c8a9:6deb with SMTP id\n h35-20020a056a20432300b001a3c8a96debmr4427287pzk.48.1711767883598;\n Fri, 29 Mar 2024 20:04:43 -0700 (PDT)", "From": "Stephen Hemminger <stephen@networkplumber.org>", "To": "dev@dpdk.org", "Cc": "Stephen Hemminger <stephen@networkplumber.org>,\n Tyler Retzlaff <roretzla@linux.microsoft.com>,\n Chenbo Xia <chenbox@nvidia.com>, Nipun Gupta <nipun.gupta@amd.com>,\n Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>,\n Pallavi Kadam <pallavi.kadam@intel.com>,\n Bruce Richardson <bruce.richardson@intel.com>,\n David Marchand <david.marchand@redhat.com>,\n Nick Connolly <nick.connolly@mayadata.io>,\n Ranjit Menon <ranjit.menon@intel.com>, Khoa To <khot@microsoft.com>", "Subject": "[PATCH v19 04/15] windows: common wrapper for vasprintf and asprintf", "Date": "Fri, 29 Mar 2024 20:00:47 -0700", "Message-ID": "<20240330030429.4630-5-stephen@networkplumber.org>", "X-Mailer": "git-send-email 2.43.0", "In-Reply-To": "<20240330030429.4630-1-stephen@networkplumber.org>", "References": "<20200814173441.23086-1-stephen@networkplumber.org>\n <20240330030429.4630-1-stephen@networkplumber.org>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "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" }, "content": "Replace the windows version of asprintf() that was only usable\nin eal. With a more generic one that supports both vasprintf()\nand asprintf(). This also eliminates duplicate code.\n\nFixes: 8f4de2dba9b9 (\"bus/pci: fill bus specific information\")\nFixes: 9ec521006db0 (\"eal/windows: hide asprintf shim\")\n\nSigned-off-by: Stephen Hemminger <stephen@networkplumber.org>\nAcked-by: Tyler Retzlaff <roretzla@linux.microsoft.com>\n---\n drivers/bus/pci/pci_common.c | 32 ------------------\n lib/eal/common/eal_private.h | 10 ------\n lib/eal/windows/eal.c | 28 ----------------\n lib/eal/windows/include/rte_os_shim.h | 48 +++++++++++++++++++++++++++\n 4 files changed, 48 insertions(+), 70 deletions(-)", "diff": "diff --git a/drivers/bus/pci/pci_common.c b/drivers/bus/pci/pci_common.c\nindex 889a48d2af..80691c75a3 100644\n--- a/drivers/bus/pci/pci_common.c\n+++ b/drivers/bus/pci/pci_common.c\n@@ -45,38 +45,6 @@ const char *rte_pci_get_sysfs_path(void)\n \treturn path;\n }\n \n-#ifdef RTE_EXEC_ENV_WINDOWS\n-#define asprintf pci_asprintf\n-\n-static int\n-__rte_format_printf(2, 3)\n-pci_asprintf(char **buffer, const char *format, ...)\n-{\n-\tint size, ret;\n-\tva_list arg;\n-\n-\tva_start(arg, format);\n-\tsize = vsnprintf(NULL, 0, format, arg);\n-\tva_end(arg);\n-\tif (size < 0)\n-\t\treturn -1;\n-\tsize++;\n-\n-\t*buffer = malloc(size);\n-\tif (*buffer == NULL)\n-\t\treturn -1;\n-\n-\tva_start(arg, format);\n-\tret = vsnprintf(*buffer, size, format, arg);\n-\tva_end(arg);\n-\tif (ret != size - 1) {\n-\t\tfree(*buffer);\n-\t\treturn -1;\n-\t}\n-\treturn ret;\n-}\n-#endif /* RTE_EXEC_ENV_WINDOWS */\n-\n static struct rte_devargs *\n pci_devargs_lookup(const struct rte_pci_addr *pci_addr)\n {\ndiff --git a/lib/eal/common/eal_private.h b/lib/eal/common/eal_private.h\nindex 71523cfdb8..da8d77a134 100644\n--- a/lib/eal/common/eal_private.h\n+++ b/lib/eal/common/eal_private.h\n@@ -737,16 +737,6 @@ void __rte_thread_init(unsigned int lcore_id, rte_cpuset_t *cpuset);\n */\n void __rte_thread_uninit(void);\n \n-/**\n- * asprintf(3) replacement for Windows.\n- */\n-#ifdef RTE_EXEC_ENV_WINDOWS\n-__rte_format_printf(2, 3)\n-int eal_asprintf(char **buffer, const char *format, ...);\n-\n-#define asprintf(buffer, format, ...) \\\n-\t\teal_asprintf(buffer, format, ##__VA_ARGS__)\n-#endif\n \n #define EAL_LOG(level, ...) \\\n \tRTE_LOG_LINE(level, EAL, \"\" __VA_ARGS__)\ndiff --git a/lib/eal/windows/eal.c b/lib/eal/windows/eal.c\nindex 52f0e7462d..8ca00c0f95 100644\n--- a/lib/eal/windows/eal.c\n+++ b/lib/eal/windows/eal.c\n@@ -503,34 +503,6 @@ rte_eal_init(int argc, char **argv)\n \treturn fctret;\n }\n \n-/* Don't use MinGW asprintf() to have identical code with all toolchains. */\n-int\n-eal_asprintf(char **buffer, const char *format, ...)\n-{\n-\tint size, ret;\n-\tva_list arg;\n-\n-\tva_start(arg, format);\n-\tsize = vsnprintf(NULL, 0, format, arg);\n-\tva_end(arg);\n-\tif (size < 0)\n-\t\treturn -1;\n-\tsize++;\n-\n-\t*buffer = malloc(size);\n-\tif (*buffer == NULL)\n-\t\treturn -1;\n-\n-\tva_start(arg, format);\n-\tret = vsnprintf(*buffer, size, format, arg);\n-\tva_end(arg);\n-\tif (ret != size - 1) {\n-\t\tfree(*buffer);\n-\t\treturn -1;\n-\t}\n-\treturn ret;\n-}\n-\n int\n rte_vfio_container_dma_map(__rte_unused int container_fd,\n \t\t\t__rte_unused uint64_t vaddr,\ndiff --git a/lib/eal/windows/include/rte_os_shim.h b/lib/eal/windows/include/rte_os_shim.h\nindex e9741a9df2..65153fdb38 100644\n--- a/lib/eal/windows/include/rte_os_shim.h\n+++ b/lib/eal/windows/include/rte_os_shim.h\n@@ -3,6 +3,7 @@\n #ifndef _RTE_OS_SHIM_\n #define _RTE_OS_SHIM_\n \n+#include <stdio.h>\n #include <time.h>\n \n #include <rte_os.h>\n@@ -120,4 +121,51 @@ rte_localtime_r(const time_t *timer, struct tm *buf)\n }\n #define localtime_r(timer, buf) rte_localtime_r(timer, buf)\n \n+/* print to allocated string */\n+__rte_format_printf(2, 0)\n+static inline int\n+rte_vasprintf(char **strp, const char *fmt, va_list ap)\n+{\n+\tchar *str;\n+\tint len, ret;\n+\n+\t*strp = NULL;\n+\n+\t/* determine size of buffer needed */\n+\tlen = _vscprintf(fmt, ap);\n+\tif (len < 0)\n+\t\treturn -1;\n+\n+\tlen += 1;\t/* for nul termination */\n+\tstr = malloc(len);\n+\tif (str == NULL)\n+\t\treturn -1;\n+\n+\tret = vsnprintf(str, len, fmt, ap);\n+\tif (ret < 0) {\n+\t\tfree(str);\n+\t\treturn -1;\n+\t} else {\n+\t\t*strp = str;\n+\t\treturn ret;\n+\t}\n+}\n+#define vasprintf(strp, fmt, ap) rte_vasprintf(strp, fmt, ap)\n+\n+__rte_format_printf(2, 3)\n+static inline int\n+rte_asprintf(char **strp, const char *fmt, ...)\n+{\n+\tint ret;\n+\n+\tva_list ap;\n+\n+\tva_start(ap, fmt);\n+\tret = rte_vasprintf(strp, fmt, ap);\n+\tva_end(ap);\n+\n+\treturn ret;\n+}\n+\n+#define asprintf(strp, fmt, ...) rte_asprintf(strp, fmt, __VA_ARGS__)\n #endif /* _RTE_OS_SHIM_ */\n", "prefixes": [ "v19", "04/15" ] }