Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/83456/?format=api
{ "id": 83456, "url": "http://patches.dpdk.org/api/patches/83456/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20201103001407.2931963-6-thomas@monjalon.net/", "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": "<20201103001407.2931963-6-thomas@monjalon.net>", "list_archive_url": "https://inbox.dpdk.org/dev/20201103001407.2931963-6-thomas@monjalon.net", "date": "2020-11-03T00:13:56", "name": "[v3,05/16] net/dpaa2: switch Rx timestamp to dynamic mbuf field", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "2d2a2fefc72b146458932543ef8fa78ca736b933", "submitter": { "id": 685, "url": "http://patches.dpdk.org/api/people/685/?format=api", "name": "Thomas Monjalon", "email": "thomas@monjalon.net" }, "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/20201103001407.2931963-6-thomas@monjalon.net/mbox/", "series": [ { "id": 13585, "url": "http://patches.dpdk.org/api/series/13585/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=13585", "date": "2020-11-03T00:13:51", "name": "remove mbuf timestamp", "version": 3, "mbox": "http://patches.dpdk.org/series/13585/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/83456/comments/", "check": "warning", "checks": "http://patches.dpdk.org/api/patches/83456/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 4C2CBA04E7;\n\tTue, 3 Nov 2020 01:16:12 +0100 (CET)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 828EC6A6F;\n\tTue, 3 Nov 2020 01:14:38 +0100 (CET)", "from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com\n [66.111.4.26]) by dpdk.org (Postfix) with ESMTP id A096768C3\n for <dev@dpdk.org>; Tue, 3 Nov 2020 01:14:34 +0100 (CET)", "from compute2.internal (compute2.nyi.internal [10.202.2.42])\n by mailout.nyi.internal (Postfix) with ESMTP id 1D6615C0216;\n Mon, 2 Nov 2020 19:14:33 -0500 (EST)", "from mailfrontend2 ([10.202.2.163])\n by compute2.internal (MEProxy); Mon, 02 Nov 2020 19:14:33 -0500", "from xps.monjalon.net (184.203.134.77.rev.sfr.net [77.134.203.184])\n by mail.messagingengine.com (Postfix) with ESMTPA id 965013064684;\n Mon, 2 Nov 2020 19:14:31 -0500 (EST)" ], "DKIM-Signature": [ "v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h=\n from:to:cc:subject:date:message-id:in-reply-to:references\n :mime-version:content-transfer-encoding; s=fm2; bh=PRDcruSUbTI7w\n HF1JOHwJ9EdEduNkMC3YvarIn3Koyg=; b=S5pF3xf1TGnbU4+5wpaq2yR7oHuZJ\n FUcQpITPKl47i+6rp5ukRHWZQll6IlABcryEEkn1vpj5oafJSXPRKGRU35CkP0C8\n bHELVHHAY3v1K2bICgmqv/6oUsFcF6xy7ojzRFQIGKhdMK2jsYdfJWKueuMTu1Dy\n PwibN2PihhLSnCfsJ08IusO8Kho0hYJua1f/mSg2OcMhC9rfln19rPxVZ6K/4zWM\n 1IqabG52z3pDKTCEYCeUIOYzDATTiZ64h5UlCsOPpC+i7pT5aqHfFEXRBWCpOhps\n B6W4SIQBBmrQlMWZgLPKNJcXACgkODlOkFAoGAG4ZyGUIbjp8ZTXVfZaQ==", "v=1; a=rsa-sha256; c=relaxed/relaxed; d=\n messagingengine.com; h=cc:content-transfer-encoding:date:from\n :in-reply-to:message-id:mime-version:references:subject:to\n :x-me-proxy:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=\n fm1; bh=PRDcruSUbTI7wHF1JOHwJ9EdEduNkMC3YvarIn3Koyg=; b=nW5d6FHy\n EkIn8isGR8nE6AVhMQ7JtwSES1zjCEKnHB6KDZ3Z/2/lI6qnZaI2EkjxfGKpyjSe\n MPqSiJJG6+CUG3FOo+Srb6nxIgxDNicFkrKog9uCp/9UH2nRN2xJXUD4NXasHgQU\n eDQIVfrMYJAun5nlKSbWUOUuaBg+jJAauA++ns8fQCfw6dCb48JQgGDBPEXsafA3\n dTBgbFDDUZ3gy8nNijNs24CW7SOFtATQonQpkPkUVrSFfo4tuSaYSkxivLivOdVy\n 9WreEZDGrCKiC+HHXpwkqq7Q+Zqe7FTSbpI1gcrHsKz9o0RyUmoTXzNW0+RCPRrr\n kscDkx+4yD3C6g==" ], "X-ME-Sender": "<xms:6KCgX1HNrN3butnpAtwpfjskpu1nPCGnqRm5AUHyvWJ_2e9g3IR0bg>\n <xme:6KCgX6VNrG8y3eKNTEP2VZnrGHwhL0f2IZAI3LqDRFoqn3Lso8EFUDk1lf9US4JAf\n BLPr0tg2P0XiuGcfQ>", "X-ME-Proxy-Cause": "\n gggruggvucftvghtrhhoucdtuddrgedujedruddtvddgudejucetufdoteggodetrfdotf\n fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen\n uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne\n cujfgurhephffvufffkffojghfggfgsedtkeertdertddtnecuhfhrohhmpefvhhhomhgr\n shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecugg\n ftrfgrthhtvghrnhepvdehgfeivdejgedtveehfefhteelfefgieevgfffveefjeegtdfg\n uedthedtgeevnecukfhppeejjedrudefgedrvddtfedrudekgeenucevlhhushhtvghruf\n hiiigvpeehnecurfgrrhgrmhepmhgrihhlfhhrohhmpehthhhomhgrshesmhhonhhjrghl\n ohhnrdhnvght", "X-ME-Proxy": "<xmx:6KCgX3Klm_vi5X3xdoyqOZ0QSuyOjibXZwJUjMmT23FU8eo_kHvDEQ>\n <xmx:6KCgX7FJ5Z36VPm8M7XjgmMUON9XQ4ulQFs0oLQwDeo1kXl_olWhLA>\n <xmx:6KCgX7UqReTPlA5AFqmZYEVHRXGEPZNu-nve3vmxLGOARzuRLRbX5w>\n <xmx:6aCgXyGLiLRw4vAij2p8i21RwGEsA__HKHaibXWePeSo0Hl1PFxQIQ>", "From": "Thomas Monjalon <thomas@monjalon.net>", "To": "dev@dpdk.org", "Cc": "ferruh.yigit@intel.com, david.marchand@redhat.com,\n bruce.richardson@intel.com, olivier.matz@6wind.com,\n andrew.rybchenko@oktetlabs.ru, jerinj@marvell.com, viacheslavo@nvidia.com,\n Hemant Agrawal <hemant.agrawal@nxp.com>,\n Sachin Saxena <sachin.saxena@oss.nxp.com>", "Date": "Tue, 3 Nov 2020 01:13:56 +0100", "Message-Id": "<20201103001407.2931963-6-thomas@monjalon.net>", "X-Mailer": "git-send-email 2.28.0", "In-Reply-To": "<20201103001407.2931963-1-thomas@monjalon.net>", "References": "<20201029092751.3837177-1-thomas@monjalon.net>\n <20201103001407.2931963-1-thomas@monjalon.net>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH v3 05/16] net/dpaa2: switch Rx timestamp to\n\tdynamic mbuf field", "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": "The mbuf timestamp is moved to a dynamic field\nin order to allow removal of the deprecated static field.\nThe related mbuf flag is also replaced.\n\nSigned-off-by: Thomas Monjalon <thomas@monjalon.net>\n---\n drivers/net/dpaa2/dpaa2_ethdev.c | 11 +++++++++++\n drivers/net/dpaa2/dpaa2_ethdev.h | 2 ++\n drivers/net/dpaa2/dpaa2_rxtx.c | 25 ++++++++++++++++++-------\n 3 files changed, 31 insertions(+), 7 deletions(-)", "diff": "diff --git a/drivers/net/dpaa2/dpaa2_ethdev.c b/drivers/net/dpaa2/dpaa2_ethdev.c\nindex 04e60c56f2..3b0c7717b6 100644\n--- a/drivers/net/dpaa2/dpaa2_ethdev.c\n+++ b/drivers/net/dpaa2/dpaa2_ethdev.c\n@@ -65,6 +65,8 @@ static uint64_t dev_tx_offloads_nodis =\n \n /* enable timestamp in mbuf */\n bool dpaa2_enable_ts[RTE_MAX_ETHPORTS];\n+uint64_t dpaa2_timestamp_rx_dynflag;\n+int dpaa2_timestamp_dynfield_offset = -1;\n \n struct rte_dpaa2_xstats_name_off {\n \tchar name[RTE_ETH_XSTATS_NAME_SIZE];\n@@ -587,7 +589,16 @@ dpaa2_eth_dev_configure(struct rte_eth_dev *dev)\n #if !defined(RTE_LIBRTE_IEEE1588)\n \tif (rx_offloads & DEV_RX_OFFLOAD_TIMESTAMP)\n #endif\n+\t{\n+\t\tret = rte_mbuf_dyn_rx_timestamp_register(\n+\t\t\t\t&dpaa2_timestamp_dynfield_offset,\n+\t\t\t\t&dpaa2_timestamp_rx_dynflag);\n+\t\tif (ret != 0) {\n+\t\t\tDPAA2_PMD_ERR(\"Error to register timestamp field/flag\");\n+\t\t\treturn -rte_errno;\n+\t\t}\n \t\tdpaa2_enable_ts[dev->data->port_id] = true;\n+\t}\n \n \tif (tx_offloads & DEV_TX_OFFLOAD_IPV4_CKSUM)\n \t\ttx_l3_csum_offload = true;\ndiff --git a/drivers/net/dpaa2/dpaa2_ethdev.h b/drivers/net/dpaa2/dpaa2_ethdev.h\nindex 94cf253827..8d82f74684 100644\n--- a/drivers/net/dpaa2/dpaa2_ethdev.h\n+++ b/drivers/net/dpaa2/dpaa2_ethdev.h\n@@ -92,6 +92,8 @@\n \n /* enable timestamp in mbuf*/\n extern bool dpaa2_enable_ts[];\n+extern uint64_t dpaa2_timestamp_rx_dynflag;\n+extern int dpaa2_timestamp_dynfield_offset;\n \n #define DPAA2_QOS_TABLE_RECONFIGURE\t1\n #define DPAA2_FS_TABLE_RECONFIGURE\t2\ndiff --git a/drivers/net/dpaa2/dpaa2_rxtx.c b/drivers/net/dpaa2/dpaa2_rxtx.c\nindex 6201de4606..9cca6d16c3 100644\n--- a/drivers/net/dpaa2/dpaa2_rxtx.c\n+++ b/drivers/net/dpaa2/dpaa2_rxtx.c\n@@ -31,6 +31,13 @@ dpaa2_dev_rx_parse_slow(struct rte_mbuf *mbuf,\n \n static void enable_tx_tstamp(struct qbman_fd *fd) __rte_unused;\n \n+static inline rte_mbuf_timestamp_t *\n+dpaa2_timestamp_dynfield(struct rte_mbuf *mbuf)\n+{\n+\treturn RTE_MBUF_DYNFIELD(mbuf,\n+\t\tdpaa2_timestamp_dynfield_offset, rte_mbuf_timestamp_t *);\n+}\n+\n #define DPAA2_MBUF_TO_CONTIG_FD(_mbuf, _fd, _bpid) do { \\\n \tDPAA2_SET_FD_ADDR(_fd, DPAA2_MBUF_VADDR_TO_IOVA(_mbuf)); \\\n \tDPAA2_SET_FD_LEN(_fd, _mbuf->data_len); \\\n@@ -109,9 +116,10 @@ dpaa2_dev_rx_parse_new(struct rte_mbuf *m, const struct qbman_fd *fd,\n \tm->ol_flags |= PKT_RX_RSS_HASH;\n \n \tif (dpaa2_enable_ts[m->port]) {\n-\t\tm->timestamp = annotation->word2;\n-\t\tm->ol_flags |= PKT_RX_TIMESTAMP;\n-\t\tDPAA2_PMD_DP_DEBUG(\"pkt timestamp:0x%\" PRIx64 \"\", m->timestamp);\n+\t\t*dpaa2_timestamp_dynfield(m) = annotation->word2;\n+\t\tm->ol_flags |= dpaa2_timestamp_rx_dynflag;\n+\t\tDPAA2_PMD_DP_DEBUG(\"pkt timestamp:0x%\" PRIx64 \"\",\n+\t\t\t\t*dpaa2_timestamp_dynfield(m));\n \t}\n \n \tDPAA2_PMD_DP_DEBUG(\"HW frc = 0x%x\\t packet type =0x%x \"\n@@ -223,9 +231,12 @@ dpaa2_dev_rx_parse(struct rte_mbuf *mbuf, void *hw_annot_addr)\n \telse if (BIT_ISSET_AT_POS(annotation->word8, DPAA2_ETH_FAS_L4CE))\n \t\tmbuf->ol_flags |= PKT_RX_L4_CKSUM_BAD;\n \n-\tmbuf->ol_flags |= PKT_RX_TIMESTAMP;\n-\tmbuf->timestamp = annotation->word2;\n-\tDPAA2_PMD_DP_DEBUG(\"pkt timestamp: 0x%\" PRIx64 \"\", mbuf->timestamp);\n+\tif (dpaa2_enable_ts[mbuf->port]) {\n+\t\t*dpaa2_timestamp_dynfield(mbuf) = annotation->word2;\n+\t\tmbuf->ol_flags |= dpaa2_timestamp_rx_dynflag;\n+\t\tDPAA2_PMD_DP_DEBUG(\"pkt timestamp: 0x%\" PRIx64 \"\",\n+\t\t\t\t*dpaa2_timestamp_dynfield(mbuf));\n+\t}\n \n \t/* Check detailed parsing requirement */\n \tif (annotation->word3 & 0x7FFFFC3FFFF)\n@@ -629,7 +640,7 @@ dpaa2_dev_prefetch_rx(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts)\n \t\telse\n \t\t\tbufs[num_rx] = eth_fd_to_mbuf(fd, eth_data->port_id);\n #if defined(RTE_LIBRTE_IEEE1588)\n-\t\tpriv->rx_timestamp = bufs[num_rx]->timestamp;\n+\t\tpriv->rx_timestamp = *dpaa2_timestamp_dynfield(bufs[num_rx]);\n #endif\n \n \t\tif (eth_data->dev_conf.rxmode.offloads &\n", "prefixes": [ "v3", "05/16" ] }