Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/64533/?format=api
http://patches.dpdk.org/api/patches/64533/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/1578895241-22229-7-git-send-email-mchalla@marvell.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": "<1578895241-22229-7-git-send-email-mchalla@marvell.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1578895241-22229-7-git-send-email-mchalla@marvell.com", "date": "2020-01-13T06:00:41", "name": "[v5,6/6] raw/octeontx2_ep: add driver self test", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "e0cbbf0c45e3167bfa515598afb9aae9587ac8ad", "submitter": { "id": 1532, "url": "http://patches.dpdk.org/api/people/1532/?format=api", "name": "Mahipal Challa", "email": "mchalla@marvell.com" }, "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/1578895241-22229-7-git-send-email-mchalla@marvell.com/mbox/", "series": [ { "id": 8071, "url": "http://patches.dpdk.org/api/series/8071/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=8071", "date": "2020-01-13T06:00:35", "name": "OCTEON TX2 End Point Driver", "version": 5, "mbox": "http://patches.dpdk.org/series/8071/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/64533/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/64533/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 C73CDA04F0;\n\tMon, 13 Jan 2020 07:02:06 +0100 (CET)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id BEA031D63D;\n\tMon, 13 Jan 2020 07:01:20 +0100 (CET)", "from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com\n [67.231.156.173]) by dpdk.org (Postfix) with ESMTP id AC2C61D631\n for <dev@dpdk.org>; Mon, 13 Jan 2020 07:01:19 +0100 (CET)", "from pps.filterd (m0045851.ppops.net [127.0.0.1])\n by mx0b-0016f401.pphosted.com (8.16.0.42/8.16.0.42) with SMTP id\n 00D60NWb023005; Sun, 12 Jan 2020 22:01:17 -0800", "from sc-exch02.marvell.com ([199.233.58.182])\n by mx0b-0016f401.pphosted.com with ESMTP id 2xfert4ceu-5\n (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT);\n Sun, 12 Jan 2020 22:01:17 -0800", "from SC-EXCH01.marvell.com (10.93.176.81) by SC-EXCH02.marvell.com\n (10.93.176.82) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Sun, 12 Jan\n 2020 22:01:15 -0800", "from maili.marvell.com (10.93.176.43) by SC-EXCH01.marvell.com\n (10.93.176.81) with Microsoft SMTP Server id 15.0.1497.2 via Frontend\n Transport; Sun, 12 Jan 2020 22:01:15 -0800", "from hyd1244.marvell.com (hyd1244.marvell.com [10.29.20.28])\n by maili.marvell.com (Postfix) with ESMTP id C02373F7041;\n Sun, 12 Jan 2020 22:01:13 -0800 (PST)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com;\n h=from : to : cc :\n subject : date : message-id : in-reply-to : references : mime-version :\n content-type; s=pfpt0818; bh=YtBf66bGS5/cGHLHe91kHjyzxspSKlBKy3FTPjNjcfw=;\n b=uWnc6r7Ra2i6iIz5oOzwtFZNN5QgGstHWNta3DbGVPSWaMEm0DsjkXkhZrIArL1h20He\n JcbP1RqEHLJsIFunSfGxgHOq23WYud6YWsa6q/xcG/j9TEK089ldXFaLC1OpbZ19qFo4\n f6nlIippdakg9o0NsGew/jBjL7SLIBo2lrsNwNDFCpu/0GpfuFXpP9yQOUd+9aTS4xqh\n 6ALIO+DRV6ezWcbtq+MQs9CXRuHS74/a8YWoZRNS0okCIr+aWOj6pKF0pt0nU4zhELP9\n o/g7MpXoFS/kBLQ5NWjOozEieTtBXzFMxKpbaEDfv9FzCU7Hoe6I/dNuk5DaiZIgco1B NA==", "From": "Mahipal Challa <mchalla@marvell.com>", "To": "<dev@dpdk.org>, <thomas@monjalon.net>", "CC": "<Gavin.Hu@arm.com>, <jerinj@marvell.com>, <pathreya@marvell.com>,\n <snilla@marvell.com>, <venkatn@marvell.com>", "Date": "Mon, 13 Jan 2020 11:30:41 +0530", "Message-ID": "<1578895241-22229-7-git-send-email-mchalla@marvell.com>", "X-Mailer": "git-send-email 1.8.3.1", "In-Reply-To": "<1578895241-22229-1-git-send-email-mchalla@marvell.com>", "References": "<1578895241-22229-1-git-send-email-mchalla@marvell.com>", "MIME-Version": "1.0", "Content-Type": "text/plain", "X-Proofpoint-Virus-Version": "vendor=fsecure engine=2.50.10434:6.0.138, 18.0.572\n definitions=2020-01-13_01:2020-01-13,\n 2020-01-13 signatures=0", "Subject": "[dpdk-dev] [PATCH v5 6/6] raw/octeontx2_ep: add driver self test", "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 rawdev's selftest feature in SDP VF driver, which\nverifies the EP mode functionality test.\n\nSigned-off-by: Mahipal Challa <mchalla@marvell.com>\n---\n doc/guides/rawdevs/octeontx2_ep.rst | 13 +++\n drivers/raw/octeontx2_ep/Makefile | 1 +\n drivers/raw/octeontx2_ep/meson.build | 1 +\n drivers/raw/octeontx2_ep/otx2_ep_rawdev.c | 1 +\n drivers/raw/octeontx2_ep/otx2_ep_rawdev.h | 2 +\n drivers/raw/octeontx2_ep/otx2_ep_test.c | 173 ++++++++++++++++++++++++++++++\n 6 files changed, 191 insertions(+)", "diff": "diff --git a/doc/guides/rawdevs/octeontx2_ep.rst b/doc/guides/rawdevs/octeontx2_ep.rst\nindex 39a7c29..bbcf530 100644\n--- a/doc/guides/rawdevs/octeontx2_ep.rst\n+++ b/doc/guides/rawdevs/octeontx2_ep.rst\n@@ -74,3 +74,16 @@ Performing Data Transfer\n To perform data transfer using SDP VF EP rawdev devices use standard\n ``rte_rawdev_enqueue_buffers()`` and ``rte_rawdev_dequeue_buffers()`` APIs.\n \n+Self test\n+---------\n+\n+On EAL initialization, SDP VF devices will be probed and populated into the\n+raw devices. The rawdev ID of the device can be obtained using\n+\n+* Invoke ``rte_rawdev_get_dev_id(\"SDPEP:x\")`` from the test application\n+ where x is the VF device's bus id specified in \"bus:device.func\"(BDF)\n+ format. Use this index for further rawdev function calls.\n+\n+* The driver's selftest rawdev API can be used to verify the SDP EP mode\n+ functional tests which can send/receive the raw data packets to/from the\n+ EP device.\ndiff --git a/drivers/raw/octeontx2_ep/Makefile b/drivers/raw/octeontx2_ep/Makefile\nindex 02853fb..44fdf89 100644\n--- a/drivers/raw/octeontx2_ep/Makefile\n+++ b/drivers/raw/octeontx2_ep/Makefile\n@@ -37,6 +37,7 @@ LIBABIVER := 1\n #\n SRCS-$(CONFIG_RTE_LIBRTE_PMD_OCTEONTX2_EP_RAWDEV) += otx2_ep_rawdev.c\n SRCS-$(CONFIG_RTE_LIBRTE_PMD_OCTEONTX2_EP_RAWDEV) += otx2_ep_enqdeq.c\n+SRCS-$(CONFIG_RTE_LIBRTE_PMD_OCTEONTX2_EP_RAWDEV) += otx2_ep_test.c\n SRCS-$(CONFIG_RTE_LIBRTE_PMD_OCTEONTX2_EP_RAWDEV) += otx2_ep_vf.c\n \n \ndiff --git a/drivers/raw/octeontx2_ep/meson.build b/drivers/raw/octeontx2_ep/meson.build\nindex 99e6c6d..0e6338f 100644\n--- a/drivers/raw/octeontx2_ep/meson.build\n+++ b/drivers/raw/octeontx2_ep/meson.build\n@@ -5,4 +5,5 @@\n deps += ['bus_pci', 'common_octeontx2', 'rawdev']\n sources = files('otx2_ep_rawdev.c',\n \t\t'otx2_ep_enqdeq.c',\n+\t\t'otx2_ep_test.c',\n \t\t'otx2_ep_vf.c')\ndiff --git a/drivers/raw/octeontx2_ep/otx2_ep_rawdev.c b/drivers/raw/octeontx2_ep/otx2_ep_rawdev.c\nindex 7158b97..0778603 100644\n--- a/drivers/raw/octeontx2_ep/otx2_ep_rawdev.c\n+++ b/drivers/raw/octeontx2_ep/otx2_ep_rawdev.c\n@@ -253,6 +253,7 @@\n \t.dev_close = sdp_rawdev_close,\n \t.enqueue_bufs = sdp_rawdev_enqueue,\n \t.dequeue_bufs = sdp_rawdev_dequeue,\n+\t.dev_selftest = sdp_rawdev_selftest,\n };\n \n static int\ndiff --git a/drivers/raw/octeontx2_ep/otx2_ep_rawdev.h b/drivers/raw/octeontx2_ep/otx2_ep_rawdev.h\nindex a77cbab..dab2fb7 100644\n--- a/drivers/raw/octeontx2_ep/otx2_ep_rawdev.h\n+++ b/drivers/raw/octeontx2_ep/otx2_ep_rawdev.h\n@@ -494,4 +494,6 @@ int sdp_rawdev_enqueue(struct rte_rawdev *dev, struct rte_rawdev_buf **buffers,\n int sdp_rawdev_dequeue(struct rte_rawdev *dev, struct rte_rawdev_buf **buffers,\n \t\t unsigned int count, rte_rawdev_obj_t context);\n \n+int sdp_rawdev_selftest(uint16_t dev_id);\n+\n #endif /* _OTX2_EP_RAWDEV_H_ */\ndiff --git a/drivers/raw/octeontx2_ep/otx2_ep_test.c b/drivers/raw/octeontx2_ep/otx2_ep_test.c\nnew file mode 100644\nindex 0000000..9746840\n--- /dev/null\n+++ b/drivers/raw/octeontx2_ep/otx2_ep_test.c\n@@ -0,0 +1,173 @@\n+/* SPDX-License-Identifier: BSD-3-Clause\n+ * Copyright(C) 2019 Marvell International Ltd.\n+ */\n+\n+#include <stdio.h>\n+#include <stdlib.h>\n+#include <string.h>\n+#include <unistd.h>\n+\n+#include <rte_common.h>\n+#include <rte_eal.h>\n+#include <rte_lcore.h>\n+#include <rte_mempool.h>\n+\n+#include <rte_rawdev.h>\n+#include <rte_rawdev_pmd.h>\n+\n+#include \"otx2_common.h\"\n+#include \"otx2_ep_rawdev.h\"\n+\n+#define SDP_IOQ_NUM_BUFS (4 * 1024)\n+#define SDP_IOQ_BUF_SIZE (2 * 1024)\n+\n+#define SDP_TEST_PKT_FSZ (0)\n+#define SDP_TEST_PKT_SIZE (1024)\n+\n+static int\n+sdp_validate_data(struct sdp_droq_pkt *oq_pkt, uint8_t *iq_pkt,\n+\t\t uint32_t pkt_len)\n+{\n+\tif (!oq_pkt)\n+\t\treturn -EINVAL;\n+\n+\tif (pkt_len != oq_pkt->len) {\n+\t\totx2_err(\"Invalid packet length\");\n+\t\treturn -EINVAL;\n+\t}\n+\n+\tif (memcmp(oq_pkt->data, iq_pkt, pkt_len) != 0) {\n+\t\totx2_err(\"Data validation failed\");\n+\t\treturn -EINVAL;\n+\t}\n+\totx2_sdp_dbg(\"Data validation successful\");\n+\n+\treturn 0;\n+}\n+\n+static void\n+sdp_ioq_buffer_fill(uint8_t *addr, uint32_t len)\n+{\n+\tuint32_t idx;\n+\n+\tmemset(addr, 0, len);\n+\n+\tfor (idx = 0; idx < len; idx++)\n+\t\taddr[idx] = idx;\n+}\n+\n+static struct rte_mempool*\n+sdp_ioq_mempool_create(void)\n+{\n+\tstruct rte_mempool *mpool;\n+\n+\tmpool = rte_mempool_create(\"ioqbuf_pool\",\n+\t\t\t\t SDP_IOQ_NUM_BUFS /*num elt*/,\n+\t\t\t\t SDP_IOQ_BUF_SIZE /*elt size*/,\n+\t\t\t\t 0 /*cache_size*/,\n+\t\t\t\t 0 /*private_data_size*/,\n+\t\t\t\t NULL /*mp_init*/,\n+\t\t\t\t NULL /*mp_init arg*/,\n+\t\t\t\t NULL /*obj_init*/,\n+\t\t\t\t NULL /*obj_init arg*/,\n+\t\t\t\t rte_socket_id() /*socket id*/,\n+\t\t\t\t (MEMPOOL_F_SP_PUT | MEMPOOL_F_SC_GET));\n+\n+\treturn mpool;\n+}\n+\n+\n+int\n+sdp_rawdev_selftest(uint16_t dev_id)\n+{\n+\tstruct sdp_rawdev_info app_info = {0};\n+\tstruct rte_rawdev_info dev_info = {0};\n+\n+\tstruct rte_rawdev_buf *d_buf[1];\n+\tstruct sdp_droq_pkt oq_pkt;\n+\tstruct sdp_soft_instr si;\n+\tstruct sdp_device sdpvf;\n+\n+\tuint32_t buf_size;\n+\tint ret = 0;\n+\tvoid *buf;\n+\n+\totx2_info(\"SDP RAWDEV Self Test: Started\");\n+\n+\tmemset(&oq_pkt, 0x00, sizeof(oq_pkt));\n+\td_buf[0] = (struct rte_rawdev_buf *)&oq_pkt;\n+\n+\tstruct rte_mempool *ioq_mpool = sdp_ioq_mempool_create();\n+\tif (!ioq_mpool) {\n+\t\totx2_err(\"IOQ mpool creation failed\");\n+\t\treturn -ENOMEM;\n+\t}\n+\n+\tapp_info.enqdeq_mpool = ioq_mpool;\n+\tapp_info.app_conf = NULL; /* Use default conf */\n+\n+\tdev_info.dev_private = &app_info;\n+\n+\tret = rte_rawdev_configure(dev_id, &dev_info);\n+\tif (ret) {\n+\t\totx2_err(\"Unable to configure SDP_VF %d\", dev_id);\n+\t\trte_mempool_free(ioq_mpool);\n+\t\treturn -ENOMEM;\n+\t}\n+\totx2_info(\"SDP VF rawdev[%d] configured successfully\", dev_id);\n+\n+\tmemset(&si, 0x00, sizeof(si));\n+\tmemset(&sdpvf, 0x00, sizeof(sdpvf));\n+\n+\tbuf_size = SDP_TEST_PKT_SIZE;\n+\n+\tsi.q_no = 0;\n+\tsi.reqtype = SDP_REQTYPE_NORESP;\n+\tsi.rptr = NULL;\n+\n+\tsi.ih.fsz = SDP_TEST_PKT_FSZ;\n+\tsi.ih.tlen = buf_size;\n+\tsi.ih.gather = 0;\n+\n+\t/* Enqueue raw pkt data */\n+\trte_mempool_get(ioq_mpool, &buf);\n+\tif (!buf) {\n+\t\totx2_err(\"Buffer allocation failed\");\n+\t\trte_mempool_free(ioq_mpool);\n+\t\trte_rawdev_close(dev_id);\n+\t\treturn -ENOMEM;\n+\t}\n+\n+\tsdp_ioq_buffer_fill(buf, buf_size);\n+\tsi.dptr = (uint8_t *)buf;\n+\n+\trte_rawdev_enqueue_buffers(dev_id, NULL, 1, &si);\n+\tusleep(10000);\n+\n+\t/* Dequeue raw pkt data */\n+\tret = 0;\n+\twhile (ret < 1) {\n+\t\tret = rte_rawdev_dequeue_buffers(dev_id, &d_buf[0], 1, &si);\n+\t\trte_pause();\n+\t}\n+\n+\t/* Validate the dequeued raw pkt data */\n+\tif (sdp_validate_data((struct sdp_droq_pkt *)d_buf[0],\n+\t\t\t buf, buf_size) != 0) {\n+\t\totx2_err(\"Data invalid\");\n+\t\trte_mempool_put(ioq_mpool,\n+\t\t\t\t((struct sdp_droq_pkt *)d_buf[0])->data);\n+\t\trte_mempool_free(ioq_mpool);\n+\t\trte_rawdev_close(dev_id);\n+\t\treturn -EINVAL;\n+\t}\n+\n+\trte_mempool_put(ioq_mpool, ((struct sdp_droq_pkt *)d_buf[0])->data);\n+\trte_mempool_free(ioq_mpool);\n+\trte_rawdev_close(dev_id);\n+\n+\totx2_info(\"SDP RAWDEV Self Test: Successful\");\n+\n+\treturn 0;\n+}\n+\n", "prefixes": [ "v5", "6/6" ] }{ "id": 64533, "url": "