Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/75985/?format=api
{ "id": 75985, "url": "http://patches.dpdk.org/api/patches/75985/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20200826095552.82525-2-radu.nicolau@intel.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": "<20200826095552.82525-2-radu.nicolau@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20200826095552.82525-2-radu.nicolau@intel.com", "date": "2020-08-26T09:55:48", "name": "[v11,1/5] eal: add WC store functions", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "abcc8bfe4219065c9791eef50d4b44e5287e3618", "submitter": { "id": 743, "url": "http://patches.dpdk.org/api/people/743/?format=api", "name": "Radu Nicolau", "email": "radu.nicolau@intel.com" }, "delegate": { "id": 24651, "url": "http://patches.dpdk.org/api/users/24651/?format=api", "username": "dmarchand", "first_name": "David", "last_name": "Marchand", "email": "david.marchand@redhat.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20200826095552.82525-2-radu.nicolau@intel.com/mbox/", "series": [ { "id": 11800, "url": "http://patches.dpdk.org/api/series/11800/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=11800", "date": "2020-08-26T09:55:47", "name": "eal: add WC store functions", "version": 11, "mbox": "http://patches.dpdk.org/series/11800/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/75985/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/75985/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 dpdk.org (dpdk.org [92.243.14.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id C558DA04B2;\n\tWed, 26 Aug 2020 11:56:12 +0200 (CEST)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 35DAE1BEBA;\n\tWed, 26 Aug 2020 11:56:05 +0200 (CEST)", "from mga09.intel.com (mga09.intel.com [134.134.136.24])\n by dpdk.org (Postfix) with ESMTP id 3712814581\n for <dev@dpdk.org>; Wed, 26 Aug 2020 11:56:03 +0200 (CEST)", "from orsmga003.jf.intel.com ([10.7.209.27])\n by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 26 Aug 2020 02:56:02 -0700", "from silpixa00400567.ir.intel.com ([10.237.214.190])\n by orsmga003.jf.intel.com with ESMTP; 26 Aug 2020 02:55:59 -0700" ], "IronPort-SDR": [ "\n NuYI3rLxVVWYXpjqRxHph5AoLVU0DHkwJOZc79TDF4czv1/qee9mwPk8vLNYcsqPB1pxn4UDvl\n cKJ0tfTQlsYw==", "\n JAUJ8jXHp2Wx0gvGTBEUL1+vT0oFkf6nBJP3TTe3G1Z/VS/lYjBON2zCgO46iwkyD4iYM+5RMW\n Ecf0+XK8QbqQ==" ], "X-IronPort-AV": [ "E=McAfee;i=\"6000,8403,9724\"; a=\"157301339\"", "E=Sophos;i=\"5.76,355,1592895600\"; d=\"scan'208\";a=\"157301339\"", "E=Sophos;i=\"5.76,355,1592895600\"; d=\"scan'208\";a=\"295319478\"" ], "X-Amp-Result": "SKIPPED(no attachment in message)", "X-Amp-File-Uploaded": "False", "X-ExtLoop1": "1", "From": "Radu Nicolau <radu.nicolau@intel.com>", "To": "dev@dpdk.org", "Cc": "beilei.xing@intel.com, jia.guo@intel.com, bruce.richardson@intel.com,\n konstantin.ananyev@intel.com, jerinjacobk@gmail.com,\n david.marchand@redhat.com, fiona.trahe@intel.com, wei.zhao1@intel.com,\n ruifeng.wang@arm.com, qiming.yang@intel.com, qi.z.zhang@intel.com,\n Radu Nicolau <radu.nicolau@intel.com>", "Date": "Wed, 26 Aug 2020 10:55:48 +0100", "Message-Id": "<20200826095552.82525-2-radu.nicolau@intel.com>", "X-Mailer": "git-send-email 2.17.1", "In-Reply-To": "<20200826095552.82525-1-radu.nicolau@intel.com>", "References": "<1591870283-7776-1-git-send-email-radu.nicolau@intel.com>\n <20200826095552.82525-1-radu.nicolau@intel.com>", "Subject": "[dpdk-dev] [PATCH v11 1/5] eal: add WC store functions", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "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": "Add rte_write32_wc and rte_write32_wc_relaxed functions\nthat implement 32bit stores using write combining memory protocol.\nProvided generic stubs and x86 implementation.\n\nSigned-off-by: Radu Nicolau <radu.nicolau@intel.com>\nAcked-by: Bruce Richardson <bruce.richardson@intel.com>\n---\n lib/librte_eal/arm/include/rte_io_64.h | 12 +++++++\n lib/librte_eal/include/generic/rte_io.h | 48 +++++++++++++++++++++++++\n lib/librte_eal/x86/include/rte_io.h | 42 ++++++++++++++++++++++\n 3 files changed, 102 insertions(+)", "diff": "diff --git a/lib/librte_eal/arm/include/rte_io_64.h b/lib/librte_eal/arm/include/rte_io_64.h\nindex e5346240e..d07d9cb22 100644\n--- a/lib/librte_eal/arm/include/rte_io_64.h\n+++ b/lib/librte_eal/arm/include/rte_io_64.h\n@@ -164,6 +164,18 @@ rte_write64(uint64_t value, volatile void *addr)\n \trte_write64_relaxed(value, addr);\n }\n \n+static __rte_always_inline void\n+rte_write32_wc(uint32_t value, volatile void *addr)\n+{\n+\trte_write32(value, addr);\n+}\n+\n+static __rte_always_inline void\n+rte_write32_wc_relaxed(uint32_t value, volatile void *addr)\n+{\n+\trte_write32_relaxed(value, addr);\n+}\n+\n #ifdef __cplusplus\n }\n #endif\ndiff --git a/lib/librte_eal/include/generic/rte_io.h b/lib/librte_eal/include/generic/rte_io.h\nindex da457f7f7..0669baa0b 100644\n--- a/lib/librte_eal/include/generic/rte_io.h\n+++ b/lib/librte_eal/include/generic/rte_io.h\n@@ -229,6 +229,40 @@ rte_write32(uint32_t value, volatile void *addr);\n static inline void\n rte_write64(uint64_t value, volatile void *addr);\n \n+/**\n+ * Write a 32-bit value to I/O device memory address addr using write\n+ * combining memory write protocol. Depending on the platform write combining\n+ * may not be available and/or may be treated as a hint and the behavior may\n+ * fallback to a regular store.\n+ *\n+ * @param value\n+ * Value to write\n+ * @param addr\n+ * I/O memory address to write the value to\n+ */\n+__rte_experimental\n+static inline void\n+rte_write32_wc(uint32_t value, volatile void *addr);\n+\n+/**\n+ * Write a 32-bit value to I/O device memory address addr using write\n+ * combining memory write protocol. Depending on the platform write combining\n+ * may not be available and/or may be treated as a hint and the behavior may\n+ * fallback to a regular store.\n+ *\n+ * The relaxed version does not have additional I/O memory barrier, useful in\n+ * accessing the device registers of integrated controllers which implicitly\n+ * strongly ordered with respect to memory access.\n+ *\n+ * @param value\n+ * Value to write\n+ * @param addr\n+ * I/O memory address to write the value to\n+ */\n+__rte_experimental\n+static inline void\n+rte_write32_wc_relaxed(uint32_t value, volatile void *addr);\n+\n #endif /* __DOXYGEN__ */\n \n #ifndef RTE_OVERRIDE_IO_H\n@@ -345,6 +379,20 @@ rte_write64(uint64_t value, volatile void *addr)\n \trte_write64_relaxed(value, addr);\n }\n \n+#ifndef RTE_NATIVE_WRITE32_WC\n+static __rte_always_inline void\n+rte_write32_wc(uint32_t value, volatile void *addr)\n+{\n+\trte_write32(value, addr);\n+}\n+\n+static __rte_always_inline void\n+rte_write32_wc_relaxed(uint32_t value, volatile void *addr)\n+{\n+\trte_write32_relaxed(value, addr);\n+}\n+#endif /* RTE_NATIVE_WRITE32_WC */\n+\n #endif /* RTE_OVERRIDE_IO_H */\n \n #endif /* _RTE_IO_H_ */\ndiff --git a/lib/librte_eal/x86/include/rte_io.h b/lib/librte_eal/x86/include/rte_io.h\nindex 2db71b1b0..4f4ff8b87 100644\n--- a/lib/librte_eal/x86/include/rte_io.h\n+++ b/lib/librte_eal/x86/include/rte_io.h\n@@ -9,8 +9,50 @@\n extern \"C\" {\n #endif\n \n+#include \"rte_cpuflags.h\"\n+\n+#define RTE_NATIVE_WRITE32_WC\n #include \"generic/rte_io.h\"\n \n+/**\n+ * @internal\n+ * MOVDIRI wrapper.\n+ */\n+static __rte_always_inline void\n+_rte_x86_movdiri(uint32_t value, volatile void *addr)\n+{\n+\tasm volatile(\n+\t\t/* MOVDIRI */\n+\t\t\".byte 0x40, 0x0f, 0x38, 0xf9, 0x02\"\n+\t\t:\n+\t\t: \"a\" (value), \"d\" (addr));\n+}\n+\n+static __rte_always_inline void\n+rte_write32_wc_relaxed(uint32_t value, volatile void *addr)\n+{\n+\tstatic int _x86_movdiri_flag = -1;\n+\tif (_x86_movdiri_flag == 1) {\n+\t\t_rte_x86_movdiri(value, addr);\n+\t} else if (_x86_movdiri_flag == 0) {\n+\t\trte_write32_relaxed(value, addr);\n+\t} else {\n+\t\t_x86_movdiri_flag =\n+\t\t\t(rte_cpu_get_flag_enabled(RTE_CPUFLAG_MOVDIRI) > 0);\n+\t\tif (_x86_movdiri_flag == 1)\n+\t\t\t_rte_x86_movdiri(value, addr);\n+\t\telse\n+\t\t\trte_write32_relaxed(value, addr);\n+\t}\n+}\n+\n+static __rte_always_inline void\n+rte_write32_wc(uint32_t value, volatile void *addr)\n+{\n+\trte_wmb();\n+\trte_write32_wc_relaxed(value, addr);\n+}\n+\n #ifdef __cplusplus\n }\n #endif\n", "prefixes": [ "v11", "1/5" ] }