Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/111869/?format=api
{ "id": 111869, "url": "https://patches.dpdk.org/api/patches/111869/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/1653526523-68839-5-git-send-email-nicolas.chautru@intel.com/", "project": { "id": 1, "url": "https://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": "<1653526523-68839-5-git-send-email-nicolas.chautru@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1653526523-68839-5-git-send-email-nicolas.chautru@intel.com", "date": "2022-05-26T00:55:22", "name": "[v6,4/5] baseband/acc100: modify validation code for ACC101", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "df2acd0208dbae85968ac8ff2bbe7f816b15a1b9", "submitter": { "id": 1314, "url": "https://patches.dpdk.org/api/people/1314/?format=api", "name": "Chautru, Nicolas", "email": "nicolas.chautru@intel.com" }, "delegate": { "id": 6690, "url": "https://patches.dpdk.org/api/users/6690/?format=api", "username": "akhil", "first_name": "akhil", "last_name": "goyal", "email": "gakhil@marvell.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/1653526523-68839-5-git-send-email-nicolas.chautru@intel.com/mbox/", "series": [ { "id": 23167, "url": "https://patches.dpdk.org/api/series/23167/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=23167", "date": "2022-05-26T00:55:19", "name": "[v6,1/5] baseband/acc100: update companion PF configure function", "version": 6, "mbox": "https://patches.dpdk.org/series/23167/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/111869/comments/", "check": "success", "checks": "https://patches.dpdk.org/api/patches/111869/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 69FE0A0548;\n\tThu, 26 May 2022 03:04:53 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id EA46242B6D;\n\tThu, 26 May 2022 03:04:32 +0200 (CEST)", "from mga14.intel.com (mga14.intel.com [192.55.52.115])\n by mails.dpdk.org (Postfix) with ESMTP id 8CE4040150\n for <dev@dpdk.org>; Thu, 26 May 2022 03:04:28 +0200 (CEST)", "from orsmga006.jf.intel.com ([10.7.209.51])\n by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 25 May 2022 18:04:24 -0700", "from skx-5gnr-sc12-4.sc.intel.com ([172.25.69.210])\n by orsmga006.jf.intel.com with ESMTP; 25 May 2022 18:04:24 -0700" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1653527068; x=1685063068;\n h=from:to:cc:subject:date:message-id:in-reply-to: references;\n bh=qz7cl5cAeiostfqsmc8K0TOFKiOW5752fz69WVYUqtY=;\n b=kwovt/Sus/5vUUUoN5D0pge0v1fwD1xoIGrt1KNfy60VIKnIjlN5BU4V\n BkV27UV9z8px8AVVuDt+GNd/n/QOo9pv1i3Wm41YWRX/pkp6K6x5xy/RH\n ybMMmQ+91X+tMHRmkFEdCuc+0Xw122Av531Jy5judJgFduqsSWov/xbJA\n uTUaUPoaK4SuZThtJST2NXzumpkHVuPyMMcKO6pfNUwJ2JYyQ3gXy86k6\n VMarHuT88B66eT6Q++YXiDe0KQMF7Yh9ag2haGAOcGhsDIhptUqICedUR\n IelEnofw9N288njZ9x8Betet4f1cMuo2zxhYjwa2Trh8z9j2wcmeTyyQu w==;", "X-IronPort-AV": [ "E=McAfee;i=\"6400,9594,10358\"; a=\"274089456\"", "E=Sophos;i=\"5.91,252,1647327600\"; d=\"scan'208\";a=\"274089456\"", "E=Sophos;i=\"5.91,252,1647327600\"; d=\"scan'208\";a=\"549294878\"" ], "X-ExtLoop1": "1", "From": "Nicolas Chautru <nicolas.chautru@intel.com>", "To": "dev@dpdk.org, gakhil@marvell.com, trix@redhat.com,\n maxime.coquelin@redhat.com", "Cc": "thomas@monjalon.net, ray.kinsella@intel.com, bruce.richardson@intel.com,\n hemant.agrawal@nxp.com, hernan.vargas@intel.com, david.marchand@redhat.com,\n Nicolas Chautru <nicolas.chautru@intel.com>", "Subject": "[PATCH v6 4/5] baseband/acc100: modify validation code for ACC101", "Date": "Wed, 25 May 2022 17:55:22 -0700", "Message-Id": "<1653526523-68839-5-git-send-email-nicolas.chautru@intel.com>", "X-Mailer": "git-send-email 1.8.3.1", "In-Reply-To": "<1653526523-68839-1-git-send-email-nicolas.chautru@intel.com>", "References": "<1653350912-53876-1-git-send-email-nicolas.chautru@intel.com>\n <1653526523-68839-1-git-send-email-nicolas.chautru@intel.com>", "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": "The validation requirement is different for the two\ndevices.\n\nSigned-off-by: Nicolas Chautru <nicolas.chautru@intel.com>\n---\n drivers/baseband/acc100/rte_acc100_pmd.c | 47 ++++++++++++++++++++++++--------\n 1 file changed, 35 insertions(+), 12 deletions(-)", "diff": "diff --git a/drivers/baseband/acc100/rte_acc100_pmd.c b/drivers/baseband/acc100/rte_acc100_pmd.c\nindex 6a2123b..a057edf 100644\n--- a/drivers/baseband/acc100/rte_acc100_pmd.c\n+++ b/drivers/baseband/acc100/rte_acc100_pmd.c\n@@ -1295,6 +1295,21 @@\n \t\t\tRTE_BBDEV_TURBO_HALF_ITERATION_EVEN);\n }\n \n+#ifdef RTE_LIBRTE_BBDEV_DEBUG\n+\n+static inline bool\n+is_acc100(struct acc100_queue *q)\n+{\n+\treturn (q->d->device_variant == ACC100_VARIANT);\n+}\n+\n+static inline bool\n+validate_op_required(struct acc100_queue *q)\n+{\n+\treturn is_acc100(q);\n+}\n+#endif\n+\n /* Fill in a frame control word for LDPC decoding. */\n static inline void\n acc100_fcw_ld_fill(struct rte_bbdev_dec_op *op, struct acc100_fcw_ld *fcw,\n@@ -2182,8 +2197,10 @@ static inline uint32_t hq_index(uint32_t offset)\n #ifdef RTE_LIBRTE_BBDEV_DEBUG\n /* Validates turbo encoder parameters */\n static inline int\n-validate_enc_op(struct rte_bbdev_enc_op *op)\n+validate_enc_op(struct rte_bbdev_enc_op *op, struct acc100_queue *q)\n {\n+\tif (!validate_op_required(q))\n+\t\treturn 0;\n \tstruct rte_bbdev_op_turbo_enc *turbo_enc = &op->turbo_enc;\n \tstruct rte_bbdev_op_enc_turbo_cb_params *cb = NULL;\n \tstruct rte_bbdev_op_enc_turbo_tb_params *tb = NULL;\n@@ -2320,8 +2337,10 @@ static inline uint32_t hq_index(uint32_t offset)\n }\n /* Validates LDPC encoder parameters */\n static inline int\n-validate_ldpc_enc_op(struct rte_bbdev_enc_op *op)\n+validate_ldpc_enc_op(struct rte_bbdev_enc_op *op, struct acc100_queue *q)\n {\n+\tif (!validate_op_required(q))\n+\t\treturn 0;\n \tstruct rte_bbdev_op_ldpc_enc *ldpc_enc = &op->ldpc_enc;\n \n \tif (op->mempool == NULL) {\n@@ -2373,8 +2392,10 @@ static inline uint32_t hq_index(uint32_t offset)\n \n /* Validates LDPC decoder parameters */\n static inline int\n-validate_ldpc_dec_op(struct rte_bbdev_dec_op *op)\n+validate_ldpc_dec_op(struct rte_bbdev_dec_op *op, struct acc100_queue *q)\n {\n+\tif (!validate_op_required(q))\n+\t\treturn 0;\n \tstruct rte_bbdev_op_ldpc_dec *ldpc_dec = &op->ldpc_dec;\n \n \tif (op->mempool == NULL) {\n@@ -2429,7 +2450,7 @@ static inline uint32_t hq_index(uint32_t offset)\n \n #ifdef RTE_LIBRTE_BBDEV_DEBUG\n \t/* Validate op structure */\n-\tif (validate_enc_op(op) == -1) {\n+\tif (validate_enc_op(op, q) == -1) {\n \t\trte_bbdev_log(ERR, \"Turbo encoder validation failed\");\n \t\treturn -EINVAL;\n \t}\n@@ -2483,7 +2504,7 @@ static inline uint32_t hq_index(uint32_t offset)\n \n #ifdef RTE_LIBRTE_BBDEV_DEBUG\n \t/* Validate op structure */\n-\tif (validate_ldpc_enc_op(ops[0]) == -1) {\n+\tif (validate_ldpc_enc_op(ops[0], q) == -1) {\n \t\trte_bbdev_log(ERR, \"LDPC encoder validation failed\");\n \t\treturn -EINVAL;\n \t}\n@@ -2545,7 +2566,7 @@ static inline uint32_t hq_index(uint32_t offset)\n \n #ifdef RTE_LIBRTE_BBDEV_DEBUG\n \t/* Validate op structure */\n-\tif (validate_ldpc_enc_op(op) == -1) {\n+\tif (validate_ldpc_enc_op(op, q) == -1) {\n \t\trte_bbdev_log(ERR, \"LDPC encoder validation failed\");\n \t\treturn -EINVAL;\n \t}\n@@ -2602,7 +2623,7 @@ static inline uint32_t hq_index(uint32_t offset)\n \n #ifdef RTE_LIBRTE_BBDEV_DEBUG\n \t/* Validate op structure */\n-\tif (validate_enc_op(op) == -1) {\n+\tif (validate_enc_op(op, q) == -1) {\n \t\trte_bbdev_log(ERR, \"Turbo encoder validation failed\");\n \t\treturn -EINVAL;\n \t}\n@@ -2675,8 +2696,10 @@ static inline uint32_t hq_index(uint32_t offset)\n #ifdef RTE_LIBRTE_BBDEV_DEBUG\n /* Validates turbo decoder parameters */\n static inline int\n-validate_dec_op(struct rte_bbdev_dec_op *op)\n+validate_dec_op(struct rte_bbdev_dec_op *op, struct acc100_queue *q)\n {\n+\tif (!validate_op_required(q))\n+\t\treturn 0;\n \tstruct rte_bbdev_op_turbo_dec *turbo_dec = &op->turbo_dec;\n \tstruct rte_bbdev_op_dec_turbo_cb_params *cb = NULL;\n \tstruct rte_bbdev_op_dec_turbo_tb_params *tb = NULL;\n@@ -2822,7 +2845,7 @@ static inline uint32_t hq_index(uint32_t offset)\n \n #ifdef RTE_LIBRTE_BBDEV_DEBUG\n \t/* Validate op structure */\n-\tif (validate_dec_op(op) == -1) {\n+\tif (validate_dec_op(op, q) == -1) {\n \t\trte_bbdev_log(ERR, \"Turbo decoder validation failed\");\n \t\treturn -EINVAL;\n \t}\n@@ -3047,7 +3070,7 @@ static inline uint32_t hq_index(uint32_t offset)\n \n #ifdef RTE_LIBRTE_BBDEV_DEBUG\n \t/* Validate op structure */\n-\tif (validate_ldpc_dec_op(op) == -1) {\n+\tif (validate_ldpc_dec_op(op, q) == -1) {\n \t\trte_bbdev_log(ERR, \"LDPC decoder validation failed\");\n \t\treturn -EINVAL;\n \t}\n@@ -3151,7 +3174,7 @@ static inline uint32_t hq_index(uint32_t offset)\n \n #ifdef RTE_LIBRTE_BBDEV_DEBUG\n \t/* Validate op structure */\n-\tif (validate_ldpc_dec_op(op) == -1) {\n+\tif (validate_ldpc_dec_op(op, q) == -1) {\n \t\trte_bbdev_log(ERR, \"LDPC decoder validation failed\");\n \t\treturn -EINVAL;\n \t}\n@@ -3241,7 +3264,7 @@ static inline uint32_t hq_index(uint32_t offset)\n \n #ifdef RTE_LIBRTE_BBDEV_DEBUG\n \t/* Validate op structure */\n-\tif (validate_dec_op(op) == -1) {\n+\tif (validate_dec_op(op, q) == -1) {\n \t\trte_bbdev_log(ERR, \"Turbo decoder validation failed\");\n \t\treturn -EINVAL;\n \t}\n", "prefixes": [ "v6", "4/5" ] }