Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/83566/?format=api
https://patches.dpdk.org/api/patches/83566/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20201103122205.183472-6-thomas@monjalon.net/", "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": "<20201103122205.183472-6-thomas@monjalon.net>", "list_archive_url": "https://inbox.dpdk.org/dev/20201103122205.183472-6-thomas@monjalon.net", "date": "2020-11-03T12:21:54", "name": "[v4,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": "https://patches.dpdk.org/api/people/685/?format=api", "name": "Thomas Monjalon", "email": "thomas@monjalon.net" }, "delegate": { "id": 1, "url": "https://patches.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20201103122205.183472-6-thomas@monjalon.net/mbox/", "series": [ { "id": 13616, "url": "https://patches.dpdk.org/api/series/13616/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=13616", "date": "2020-11-03T12:21:49", "name": "remove mbuf timestamp", "version": 4, "mbox": "https://patches.dpdk.org/series/13616/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/83566/comments/", "check": "warning", "checks": "https://patches.dpdk.org/api/patches/83566/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 0C6B6A0521;\n\tTue, 3 Nov 2020 13:24:16 +0100 (CET)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 6CCDAC9FA;\n\tTue, 3 Nov 2020 13:22:32 +0100 (CET)", "from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com\n [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id 787B8C9E8\n for <dev@dpdk.org>; Tue, 3 Nov 2020 13:22:29 +0100 (CET)", "from compute2.internal (compute2.nyi.internal [10.202.2.42])\n by mailout.nyi.internal (Postfix) with ESMTP id 033E75C010A;\n Tue, 3 Nov 2020 07:22:28 -0500 (EST)", "from mailfrontend2 ([10.202.2.163])\n by compute2.internal (MEProxy); Tue, 03 Nov 2020 07:22:28 -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 86BF23064684;\n Tue, 3 Nov 2020 07:22:26 -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=KwtLzXKgZMiD1\n Zg5UliWNRCSI0+Xd/1/iHcT6A+5bwo=; b=M0EtAtzr0X6VULNZLA06Pd4jJsM+J\n L/UEsKCHoFnXT9Z5lLY/yytDQEp0ZRtdtdB04ddaUxk6QH/XaDWaMXX/Y+2AFRxA\n 08RmMR1JEE8LK3lN3CBfidb/O6WqgzUU/20EfFAVncSMKaVk30YYGzJz7D805JfA\n YIU7/1sFRYSvB+E7JIJICzaMhPYI6hcfW3pMwPSfm9SB6Bdh2gcpZCCju38YGkCD\n AweDJyW3secX2fgO1DaiLnVMqfk5Nj9klWv8f5QvyT6JEgjf44oLQ6tRhIqtlYna\n nmNaAbMmNujkIdTkMvxkZfRouuw1DkfEHM+gVdGBJ9HT7RfzYGW2I/0iA==", "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=KwtLzXKgZMiD1Zg5UliWNRCSI0+Xd/1/iHcT6A+5bwo=; b=HxmDwFlk\n ZHHW3IqTPBi9Y/6DtcDm4uxDOFMyCA/0Nd7pIsE4FXdyiiHXWCCesifR8cDsD394\n p39nQNq+BOXl6m0AhMW5XH8WGOR5JnXLoGQXxviyqKNxPTQ6fT1ltzqFpeHFW1OQ\n ZbUziJJ2Gt2xPfFBiBiWU/in9h7QLqSM+pQoxh+9bBKtIBohr3q/iiH8cy1Fy43Z\n tAI++9U6JjRaqQsl3/d8dyCEe+NL4T83ju2UYmwtHii58R5uodCHMAahxFwIbeiB\n h451gxRGCFbCzFR6f3QGp+bm0WUF3utquibGUwsSXnir2XVAW5NgvwHCnpZFI8Mt\n BEV5BEfwq8hGsA==" ], "X-ME-Sender": "<xms:g0uhX3ApU5EO5rpizoT6XQ0FPy-ue2nVLrlYAq5flzK944QyYmLDSw>\n <xme:g0uhX9hg2TIUvmD8YlfYCr3_WaKxuU9R7iIHCdr5dLjBLz4PRtU4aQ0ApYR10kAnC\n SiiTdMwVjNFWWA8dg>", "X-ME-Proxy-Cause": "\n gggruggvucftvghtrhhoucdtuddrgedujedruddtfedgfeelucetufdoteggodetrfdotf\n fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen\n uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne\n cujfgurhephffvufffkffojghfggfgsedtkeertdertddtnecuhfhrohhmpefvhhhomhgr\n shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecugg\n ftrfgrthhtvghrnhepvdehgfeivdejgedtveehfefhteelfefgieevgfffveefjeegtdfg\n uedthedtgeevnecukfhppeejjedrudefgedrvddtfedrudekgeenucevlhhushhtvghruf\n hiiigvpeegnecurfgrrhgrmhepmhgrihhlfhhrohhmpehthhhomhgrshesmhhonhhjrghl\n ohhnrdhnvght", "X-ME-Proxy": "<xmx:g0uhXymkEIIe01Bawx-4r9AHlT-Z9zWUKrfCbVuPh1DO8Lk7pMlO7g>\n <xmx:g0uhX5wnQn2DHjnH_4MtX2H4Se5jGQhgOQ95D7u_oXtExqJnctznvQ>\n <xmx:g0uhX8TJ1MylHKFlFGqlG77ugfHONjAJD9WaC3pZlLGGXyut7r2PyQ>\n <xmx:g0uhXxTRPHDx2igfG1b0VttCys5m8AUfee3pgUcrlQMCKYkKyOq2eQ>", "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 13:21:54 +0100", "Message-Id": "<20201103122205.183472-6-thomas@monjalon.net>", "X-Mailer": "git-send-email 2.28.0", "In-Reply-To": "<20201103122205.183472-1-thomas@monjalon.net>", "References": "<20201029092751.3837177-1-thomas@monjalon.net>\n <20201103122205.183472-1-thomas@monjalon.net>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH v4 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>\nAcked-by: Hemant Agrawal <hemant.agrawal@nxp.com>\nAcked-by: David Marchand <david.marchand@redhat.com>\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": [ "v4", "05/16" ] }{ "id": 83566, "url": "