Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/54435/?format=api
{ "id": 54435, "url": "http://patches.dpdk.org/api/patches/54435/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/1559765196-250614-3-git-send-email-nicolas.chautru@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": "<1559765196-250614-3-git-send-email-nicolas.chautru@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1559765196-250614-3-git-send-email-nicolas.chautru@intel.com", "date": "2019-06-05T20:06:35", "name": "[v2,2/3] baseband/turbo_sw: option to build turbosw PMD without SDK", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "0d37ab056f47c4a36a9094674652d5fea9295a3c", "submitter": { "id": 1314, "url": "http://patches.dpdk.org/api/people/1314/?format=api", "name": "Chautru, Nicolas", "email": "nicolas.chautru@intel.com" }, "delegate": null, "mbox": "http://patches.dpdk.org/project/dpdk/patch/1559765196-250614-3-git-send-email-nicolas.chautru@intel.com/mbox/", "series": [ { "id": 4913, "url": "http://patches.dpdk.org/api/series/4913/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=4913", "date": "2019-06-05T20:06:33", "name": "BBDEV turbo_sw PMD compilation fix", "version": 2, "mbox": "http://patches.dpdk.org/series/4913/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/54435/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/54435/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@dpdk.org", "Delivered-To": "patchwork@dpdk.org", "Received": [ "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id DAE9F1B9B0;\n\tWed, 5 Jun 2019 22:08:29 +0200 (CEST)", "from mga06.intel.com (mga06.intel.com [134.134.136.31])\n\tby dpdk.org (Postfix) with ESMTP id 775931B9A1\n\tfor <dev@dpdk.org>; Wed, 5 Jun 2019 22:08:27 +0200 (CEST)", "from orsmga005.jf.intel.com ([10.7.209.41])\n\tby orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n\t05 Jun 2019 13:08:26 -0700", "from skx-5gnr-sc12-1.sc.intel.com ([172.25.69.194])\n\tby orsmga005.jf.intel.com with ESMTP; 05 Jun 2019 13:08:26 -0700" ], "X-Amp-Result": "SKIPPED(no attachment in message)", "X-Amp-File-Uploaded": "False", "X-ExtLoop1": "1", "From": "Nicolas Chautru <nicolas.chautru@intel.com>", "To": "thomas@monjalon.net, akhil.goyal@nxp.com, dev@dpdk.org\u001b\b\b", "Cc": "ferruh.yigit@intel.com,\n\tNicolas Chautru <nicolas.chautru@intel.com>", "Date": "Wed, 5 Jun 2019 13:06:35 -0700", "Message-Id": "<1559765196-250614-3-git-send-email-nicolas.chautru@intel.com>", "X-Mailer": "git-send-email 1.8.3.1", "In-Reply-To": "<1559765196-250614-1-git-send-email-nicolas.chautru@intel.com>", "References": "<1559756323-179855-2-git-send-email-nicolas.chautru@intel.com>\n\t<1559765196-250614-1-git-send-email-nicolas.chautru@intel.com>", "Subject": "[dpdk-dev] [PATCH v2 2/3] baseband/turbo_sw: option to build\n\tturbosw PMD without SDK", "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\t<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\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "Adding compile flag to allow to build the turbo_sw PMD\nwithout dependency to have the SDK libraries installed.\n\nSigned-off-by: Nicolas Chautru <nicolas.chautru@intel.com>\n---\n config/common_base | 3 +-\n drivers/baseband/turbo_sw/Makefile | 13 ++++----\n drivers/baseband/turbo_sw/bbdev_turbo_software.c | 39 +++++++++++++++++++++++-\n mk/rte.app.mk | 3 ++\n 4 files changed, 50 insertions(+), 8 deletions(-)", "diff": "diff --git a/config/common_base b/config/common_base\nindex 6b96e0e..bc80209 100644\n--- a/config/common_base\n+++ b/config/common_base\n@@ -523,6 +523,7 @@ CONFIG_RTE_PMD_PACKET_PREFETCH=y\n CONFIG_RTE_LIBRTE_BBDEV=y\n CONFIG_RTE_BBDEV_MAX_DEVS=128\n CONFIG_RTE_BBDEV_OFFLOAD_COST=y\n+CONFIG_RTE_BBDEV_SDK_AVX2=n\n \n #\n # Compile PMD for NULL bbdev device\n@@ -532,7 +533,7 @@ CONFIG_RTE_LIBRTE_PMD_BBDEV_NULL=y\n #\n # Compile PMD for turbo software bbdev device\n #\n-CONFIG_RTE_LIBRTE_PMD_BBDEV_TURBO_SW=n\n+CONFIG_RTE_LIBRTE_PMD_BBDEV_TURBO_SW=y\n \n #\n # Compile generic crypto device library\ndiff --git a/drivers/baseband/turbo_sw/Makefile b/drivers/baseband/turbo_sw/Makefile\nindex d364677..414d0d9 100644\n--- a/drivers/baseband/turbo_sw/Makefile\n+++ b/drivers/baseband/turbo_sw/Makefile\n@@ -3,9 +3,6 @@\n \n include $(RTE_SDK)/mk/rte.vars.mk\n \n-ifeq ($(FLEXRAN_SDK),)\n-$(error \"Please define FLEXRAN_SDK environment variable\")\n-endif\n \n # library name\n LIB = librte_pmd_bbdev_turbo_sw.a\n@@ -21,17 +18,21 @@ LDLIBS += -lrte_bus_vdev\n # versioning export map\n EXPORT_MAP := rte_pmd_bbdev_turbo_sw_version.map\n \n-# external library dependencies\n+# external library dependencies if available\n+ifeq ($(CONFIG_RTE_BBDEV_SDK_AVX2),y)\n+ifeq ($(FLEXRAN_SDK),)\n+$(error \"Please define FLEXRAN_SDK environment variable\")\n+endif\n CFLAGS += -I$(FLEXRAN_SDK)/lib_common\n CFLAGS += -I$(FLEXRAN_SDK)/lib_turbo\n CFLAGS += -I$(FLEXRAN_SDK)/lib_crc\n CFLAGS += -I$(FLEXRAN_SDK)/lib_rate_matching\n-\n LDLIBS += -L$(FLEXRAN_SDK)/lib_turbo -lturbo\n LDLIBS += -L$(FLEXRAN_SDK)/lib_crc -lcrc\n LDLIBS += -L$(FLEXRAN_SDK)/lib_rate_matching -lrate_matching\n LDLIBS += -L$(FLEXRAN_SDK)/lib_common -lcommon\n-LDLIBS += -lstdc++ -lirc -limf -lipps\n+LDLIBS += -lstdc++ -lirc -limf -lipps -lsvml\n+endif\n \n # library version\n LIBABIVER := 1\ndiff --git a/drivers/baseband/turbo_sw/bbdev_turbo_software.c b/drivers/baseband/turbo_sw/bbdev_turbo_software.c\nindex 5204a77..4781de0 100644\n--- a/drivers/baseband/turbo_sw/bbdev_turbo_software.c\n+++ b/drivers/baseband/turbo_sw/bbdev_turbo_software.c\n@@ -14,10 +14,11 @@\n #include <rte_bbdev.h>\n #include <rte_bbdev_pmd.h>\n \n+#ifdef RTE_BBDEV_SDK_AVX2\n #include <phy_turbo.h>\n #include <phy_crc.h>\n #include <phy_rate_match.h>\n-#include <divide.h>\n+#endif\n \n #define DRIVER_NAME baseband_turbo_sw\n \n@@ -143,6 +144,7 @@ struct turbo_sw_queue {\n \tstruct bbdev_private *internals = dev->data->dev_private;\n \n \tstatic const struct rte_bbdev_op_cap bbdev_capabilities[] = {\n+#ifdef RTE_BBDEV_SDK_AVX2\n \t\t{\n \t\t\t.type = RTE_BBDEV_OP_TURBO_DEC,\n \t\t\t.cap.turbo_dec = {\n@@ -172,6 +174,7 @@ struct turbo_sw_queue {\n \t\t\t\t.num_buffers_dst = RTE_BBDEV_MAX_CODE_BLOCKS,\n \t\t\t}\n \t\t},\n+#endif\n \t\tRTE_BBDEV_END_OF_CAPABILITIES_LIST()\n \t};\n \n@@ -472,6 +475,7 @@ struct turbo_sw_queue {\n \t\tstruct rte_mbuf *m_out, uint16_t in_offset, uint16_t out_offset,\n \t\tuint16_t in_length, struct rte_bbdev_stats *q_stats)\n {\n+#ifdef RTE_BBDEV_SDK_AVX2\n \tint ret;\n \tint16_t k_idx;\n \tuint16_t m;\n@@ -724,6 +728,22 @@ struct turbo_sw_queue {\n \t\t}\n \t\t*tmp_out = 0;\n \t}\n+#else\n+\tRTE_SET_USED(q);\n+\tRTE_SET_USED(op);\n+\tRTE_SET_USED(r);\n+\tRTE_SET_USED(c);\n+\tRTE_SET_USED(k);\n+\tRTE_SET_USED(ncb);\n+\tRTE_SET_USED(e);\n+\tRTE_SET_USED(m_in);\n+\tRTE_SET_USED(m_out_head);\n+\tRTE_SET_USED(m_out);\n+\tRTE_SET_USED(in_offset);\n+\tRTE_SET_USED(out_offset);\n+\tRTE_SET_USED(in_length);\n+\tRTE_SET_USED(q_stats);\n+#endif\n }\n \n static inline void\n@@ -856,6 +876,7 @@ struct turbo_sw_queue {\n \t\tuint16_t crc24_overlap, uint16_t in_length,\n \t\tstruct rte_bbdev_stats *q_stats)\n {\n+#ifdef RTE_BBDEV_SDK_AVX2\n \tint ret;\n \tint32_t k_idx;\n \tint32_t iter_cnt;\n@@ -972,6 +993,22 @@ struct turbo_sw_queue {\n \t\trte_bbdev_log(ERR, \"Turbo Decoder failed\");\n \t\treturn;\n \t}\n+#else\n+\tRTE_SET_USED(q);\n+\tRTE_SET_USED(op);\n+\tRTE_SET_USED(c);\n+\tRTE_SET_USED(k);\n+\tRTE_SET_USED(kw);\n+\tRTE_SET_USED(m_in);\n+\tRTE_SET_USED(m_out_head);\n+\tRTE_SET_USED(m_out);\n+\tRTE_SET_USED(in_offset);\n+\tRTE_SET_USED(out_offset);\n+\tRTE_SET_USED(check_crc_24b);\n+\tRTE_SET_USED(crc24_overlap);\n+\tRTE_SET_USED(in_length);\n+\tRTE_SET_USED(q_stats);\n+#endif\n }\n \n static inline void\ndiff --git a/mk/rte.app.mk b/mk/rte.app.mk\nindex 7c9b4b5..878cc31 100644\n--- a/mk/rte.app.mk\n+++ b/mk/rte.app.mk\n@@ -221,11 +221,14 @@ _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_BBDEV_NULL) += -lrte_pmd_bbdev_null\n \n # TURBO SOFTWARE PMD is dependent on the FLEXRAN library\n _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_BBDEV_TURBO_SW) += -lrte_pmd_bbdev_turbo_sw\n+ifeq ($(CONFIG_RTE_BBDEV_SDK_AVX2),y)\n+# Dependency on the FLEXRAN SDK library if available\n _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_BBDEV_TURBO_SW) += -L$(FLEXRAN_SDK)/lib_crc -lcrc\n _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_BBDEV_TURBO_SW) += -L$(FLEXRAN_SDK)/lib_turbo -lturbo\n _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_BBDEV_TURBO_SW) += -L$(FLEXRAN_SDK)/lib_rate_matching -lrate_matching\n _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_BBDEV_TURBO_SW) += -L$(FLEXRAN_SDK)/lib_common -lcommon\n _LDLIBS-$(CONFIG_RTE_LIBRTE_PMD_BBDEV_TURBO_SW) += -lirc -limf -lstdc++ -lipps\n+endif # CONFIG_RTE_BBDEV_SDK_AVX2\n endif # CONFIG_RTE_LIBRTE_BBDEV\n \n ifeq ($(CONFIG_RTE_LIBRTE_CRYPTODEV),y)\n", "prefixes": [ "v2", "2/3" ] }