Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/82378/?format=api
{ "id": 82378, "url": "http://patches.dpdk.org/api/patches/82378/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20201027210115.2529025-14-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": "<20201027210115.2529025-14-thomas@monjalon.net>", "list_archive_url": "https://inbox.dpdk.org/dev/20201027210115.2529025-14-thomas@monjalon.net", "date": "2020-10-27T21:01:13", "name": "[v3,13/15] examples/bbdev: switch to dynamic mbuf field", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "5e0e93c3ab4431d661edb3b3eca7a078df4aeeb5", "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/20201027210115.2529025-14-thomas@monjalon.net/mbox/", "series": [ { "id": 13393, "url": "http://patches.dpdk.org/api/series/13393/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=13393", "date": "2020-10-27T21:01:00", "name": "remove mbuf userdata", "version": 3, "mbox": "http://patches.dpdk.org/series/13393/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/82378/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/82378/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 4B217A04B5;\n\tTue, 27 Oct 2020 22:05:56 +0100 (CET)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 1CA962C1A;\n\tTue, 27 Oct 2020 22:02:25 +0100 (CET)", "from out5-smtp.messagingengine.com (out5-smtp.messagingengine.com\n [66.111.4.29]) by dpdk.org (Postfix) with ESMTP id C6A4F58CD\n for <dev@dpdk.org>; Tue, 27 Oct 2020 22:02:15 +0100 (CET)", "from compute2.internal (compute2.nyi.internal [10.202.2.42])\n by mailout.nyi.internal (Postfix) with ESMTP id 76B2E5C00B4;\n Tue, 27 Oct 2020 17:02:14 -0400 (EDT)", "from mailfrontend1 ([10.202.2.162])\n by compute2.internal (MEProxy); Tue, 27 Oct 2020 17:02:14 -0400", "from xps.monjalon.net (184.203.134.77.rev.sfr.net [77.134.203.184])\n by mail.messagingengine.com (Postfix) with ESMTPA id 6E1113280059;\n Tue, 27 Oct 2020 17:02:13 -0400 (EDT)" ], "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=tQuKWbiliUVAj\n SSPhKYtMVm5sa1lzLavUnMWeV38LP4=; b=htWG0Rh/ng015pCAFtew70qzyhsMQ\n tVWch3O0spqVG4eF7VcWVuCPYUc1n43VJh0YUqsc2Ahe7YPkK8XGstwrUd6E2eLM\n xltiO3jW8dGgXIOyTulOtxA11CKSLf7FtKoum04fsIx0P2XMOMakBrEAAx55BUDK\n di5lD3ZmN4POpntGVBuywYsriK6KdDDe3yr6cT8H+yAsk2yCVApbC/UyLmbBv/cw\n owMA1w7bFRglOPeT1WDEHHxIBf2axvySFPYmJ937YbauY+rBqixETXEze99mEUtP\n UUl4/lVFkAgcZAdBegkW3biZyECY2OOc7PrkTa9vVZbO4JCQ+YstVdeEQ==", "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=tQuKWbiliUVAjSSPhKYtMVm5sa1lzLavUnMWeV38LP4=; b=ch03CNJE\n pHjVYuwjOAhcaRfgCihMhDkemkXc5BntXVxb+GVsDxLSJHVCnQ+j1Vo1uTDRUv76\n 2Qstxk5zoS/IfAjo30vaeY590wDZSDR5qpEIPNdNoKICj3qrvFQTghMcJlunU9Yq\n SxBEtEei/rx7AqQemHSKzXLhvREOjanAD/wKD2yiy7BPnsP9v4SiRkXPbw01q/Z5\n YckQZg0ez3rYjHzXBgbFf7SugZFF7R4V7bovTpmBhWXgg6gfbQ4jSQn37lbGju6b\n qkuRj8TTgWc8EBHe0VWxD/+LmE/An/51RlHncno9HO1N+EKw2KP9sdDgiA48wX/R\n F0CBJSKXaMTIrw==" ], "X-ME-Sender": "<xms:1oqYX6nXqiBamnunJtSTi8bWlrMAcWwh3nDZnmLvw-voCxoz3p32HA>\n <xme:1oqYXx3FQQyyVTN6uBRS8Uu-RRHgXS5YNvt44B3AHsCnD6B1Juw-67Ob-YGwHJV-O\n wM2S3kTZPG2Evqf6Q>", "X-ME-Proxy-Cause": "\n gggruggvucftvghtrhhoucdtuddrgedujedrkeelgddugeegucetufdoteggodetrfdotf\n fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen\n uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne\n cujfgurhephffvufffkffojghfggfgsedtkeertdertddtnecuhfhrohhmpefvhhhomhgr\n shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecugg\n ftrfgrthhtvghrnhepvdehgfeivdejgedtveehfefhteelfefgieevgfffveefjeegtdfg\n uedthedtgeevnecukfhppeejjedrudefgedrvddtfedrudekgeenucevlhhushhtvghruf\n hiiigvpedutdenucfrrghrrghmpehmrghilhhfrhhomhepthhhohhmrghssehmohhnjhgr\n lhhonhdrnhgvth", "X-ME-Proxy": "<xmx:1oqYX4pTNSMwn-j0uRgKCyOOxoDjtIADnFM1peLplY5YpW9O3Vbvmg>\n <xmx:1oqYX-n53doargNRUc_V1B3LsOer50Tg9ZE7xkk3LrZCoFPbKuaPoA>\n <xmx:1oqYX42CGkyJ9oQ2jTtNXJzU5LTzHTxlDQRA4sXT5-J4ewW7hmYvVg>\n <xmx:1oqYX8-WGdzu4BTrSC9rVtxIIlzWcKf43hCIW8a0P3HInRW37cPswg>", "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, akhil.goyal@nxp.com,\n Nicolas Chautru <nicolas.chautru@intel.com>", "Date": "Tue, 27 Oct 2020 22:01:13 +0100", "Message-Id": "<20201027210115.2529025-14-thomas@monjalon.net>", "X-Mailer": "git-send-email 2.28.0", "In-Reply-To": "<20201027210115.2529025-1-thomas@monjalon.net>", "References": "<20201026052105.1561859-1-thomas@monjalon.net>\n <20201027210115.2529025-1-thomas@monjalon.net>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH v3 13/15] examples/bbdev: switch to dynamic mbuf\n\tfield", "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 example used the deprecated mbuf field udata64 as input mbuf pointer.\nIt is moved to a dynamic field in order to allow removal of udata64.\n\nSigned-off-by: Thomas Monjalon <thomas@monjalon.net>\n---\n examples/bbdev_app/main.c | 49 +++++++++++++++++++++++++++++++--------\n 1 file changed, 39 insertions(+), 10 deletions(-)", "diff": "diff --git a/examples/bbdev_app/main.c b/examples/bbdev_app/main.c\nindex e512c807cd..63e8b18e44 100644\n--- a/examples/bbdev_app/main.c\n+++ b/examples/bbdev_app/main.c\n@@ -28,6 +28,7 @@\n #include <rte_lcore.h>\n #include <rte_malloc.h>\n #include <rte_mbuf.h>\n+#include <rte_mbuf_dyn.h>\n #include <rte_memory.h>\n #include <rte_mempool.h>\n #include <rte_log.h>\n@@ -59,6 +60,22 @@\n \t} \\\n } while (0)\n \n+static int input_dynfield_offset = -1;\n+\n+static inline struct rte_mbuf *\n+get_mbuf_input(struct rte_mbuf *mbuf)\n+{\n+\treturn *RTE_MBUF_DYNFIELD(mbuf,\n+\t\t\tinput_dynfield_offset, struct rte_mbuf **);\n+}\n+\n+static inline void\n+set_mbuf_input(struct rte_mbuf *mbuf, struct rte_mbuf *input)\n+{\n+\t*RTE_MBUF_DYNFIELD(mbuf,\n+\t\t\tinput_dynfield_offset, struct rte_mbuf **) = input;\n+}\n+\n static const struct rte_eth_conf port_conf = {\n \t.rxmode = {\n \t\t.mq_mode = ETH_MQ_RX_NONE,\n@@ -294,11 +311,11 @@ pktmbuf_free_bulk(struct rte_mbuf **mbufs, unsigned int nb_to_free)\n }\n \n static inline void\n-pktmbuf_userdata_free_bulk(struct rte_mbuf **mbufs, unsigned int nb_to_free)\n+pktmbuf_input_free_bulk(struct rte_mbuf **mbufs, unsigned int nb_to_free)\n {\n \tunsigned int i;\n \tfor (i = 0; i < nb_to_free; ++i) {\n-\t\tstruct rte_mbuf *rx_pkt = mbufs[i]->userdata;\n+\t\tstruct rte_mbuf *rx_pkt = get_mbuf_input(mbufs[i]);\n \t\trte_pktmbuf_free(rx_pkt);\n \t\trte_pktmbuf_free(mbufs[i]);\n \t}\n@@ -429,7 +446,7 @@ verify_data(struct rte_mbuf **mbufs, uint16_t num_pkts)\n \tuint16_t i;\n \tfor (i = 0; i < num_pkts; ++i) {\n \t\tstruct rte_mbuf *out = mbufs[i];\n-\t\tstruct rte_mbuf *in = out->userdata;\n+\t\tstruct rte_mbuf *in = get_mbuf_input(out);\n \n \t\tif (memcmp(rte_pktmbuf_mtod_offset(in, uint8_t *,\n \t\t\t\tsizeof(struct rte_ether_hdr)),\n@@ -731,7 +748,7 @@ run_encoding(struct lcore_conf *lcore_conf)\n \t\t\t\trte_pktmbuf_data_len(rx_pkts_burst[i]) -\n \t\t\t\tsizeof(struct rte_ether_hdr);\n \t\t/* save input mbuf pointer for later comparison */\n-\t\tenc_out_pkts[i]->userdata = rx_pkts_burst[i];\n+\t\tset_mbuf_input(enc_out_pkts[i], rx_pkts_burst[i]);\n \n \t\t/* copy ethernet header */\n \t\trte_pktmbuf_reset(enc_out_pkts[i]);\n@@ -775,7 +792,7 @@ run_encoding(struct lcore_conf *lcore_conf)\n \tnb_enq = rte_bbdev_enqueue_enc_ops(bbdev_id, enc_queue_id,\n \t\t\tbbdev_ops_burst, nb_rx);\n \tif (unlikely(nb_enq < nb_rx)) {\n-\t\tpktmbuf_userdata_free_bulk(&enc_out_pkts[nb_enq],\n+\t\tpktmbuf_input_free_bulk(&enc_out_pkts[nb_enq],\n \t\t\t\tnb_rx - nb_enq);\n \t\trte_bbdev_enc_op_free_bulk(&bbdev_ops_burst[nb_enq],\n \t\t\t\tnb_rx - nb_enq);\n@@ -805,7 +822,7 @@ run_encoding(struct lcore_conf *lcore_conf)\n \tnb_sent = rte_ring_enqueue_burst(enc_to_dec_ring, (void **)enc_out_pkts,\n \t\t\tnb_deq, NULL);\n \tif (unlikely(nb_sent < nb_deq)) {\n-\t\tpktmbuf_userdata_free_bulk(&enc_out_pkts[nb_sent],\n+\t\tpktmbuf_input_free_bulk(&enc_out_pkts[nb_sent],\n \t\t\t\tnb_deq - nb_sent);\n \t\tlcore_stats->enc_to_dec_lost_packets += nb_deq - nb_sent;\n \t}\n@@ -842,7 +859,7 @@ run_decoding(struct lcore_conf *lcore_conf)\n \n \tif (unlikely(rte_bbdev_dec_op_alloc_bulk(bbdev_op_pool, bbdev_ops_burst,\n \t\t\tnb_recv) != 0)) {\n-\t\tpktmbuf_userdata_free_bulk(recv_pkts_burst, nb_recv);\n+\t\tpktmbuf_input_free_bulk(recv_pkts_burst, nb_recv);\n \t\tlcore_stats->rx_lost_packets += nb_recv;\n \t\treturn;\n \t}\n@@ -871,7 +888,7 @@ run_decoding(struct lcore_conf *lcore_conf)\n \tnb_enq = rte_bbdev_enqueue_dec_ops(bbdev_id, bbdev_queue_id,\n \t\t\tbbdev_ops_burst, nb_recv);\n \tif (unlikely(nb_enq < nb_recv)) {\n-\t\tpktmbuf_userdata_free_bulk(&recv_pkts_burst[nb_enq],\n+\t\tpktmbuf_input_free_bulk(&recv_pkts_burst[nb_enq],\n \t\t\t\tnb_recv - nb_enq);\n \t\trte_bbdev_dec_op_free_bulk(&bbdev_ops_burst[nb_enq],\n \t\t\t\tnb_recv - nb_enq);\n@@ -898,12 +915,12 @@ run_decoding(struct lcore_conf *lcore_conf)\n \n \t/* Free the RX mbufs after verification */\n \tfor (i = 0; i < nb_deq; ++i)\n-\t\trte_pktmbuf_free(recv_pkts_burst[i]->userdata);\n+\t\trte_pktmbuf_free(get_mbuf_input(recv_pkts_burst[i]));\n \n \t/* Transmit the packets */\n \tnb_tx = rte_eth_tx_burst(port_id, tx_queue_id, recv_pkts_burst, nb_deq);\n \tif (unlikely(nb_tx < nb_deq)) {\n-\t\tpktmbuf_userdata_free_bulk(&recv_pkts_burst[nb_tx],\n+\t\tpktmbuf_input_free_bulk(&recv_pkts_burst[nb_tx],\n \t\t\t\tnb_deq - nb_tx);\n \t\tlcore_stats->tx_lost_packets += nb_deq - nb_tx;\n \t}\n@@ -1046,6 +1063,12 @@ main(int argc, char **argv)\n \tbool stats_thread_started = false;\n \tunsigned int main_lcore_id = rte_get_main_lcore();\n \n+\tstatic const struct rte_mbuf_dynfield input_dynfield_desc = {\n+\t\t.name = \"example_bbdev_dynfield_input\",\n+\t\t.size = sizeof(struct rte_mbuf *),\n+\t\t.align = __alignof__(struct rte_mbuf *),\n+\t};\n+\n \trte_atomic16_init(&global_exit_flag);\n \n \tsigret = signal(SIGTERM, signal_handler);\n@@ -1115,6 +1138,12 @@ main(int argc, char **argv)\n \tif (bbdev_mbuf_mempool == NULL)\n \t\trte_exit(EXIT_FAILURE, \"Cannot create ethdev mbuf mempool\\n\");\n \n+\t/* register mbuf field to store input pointer */\n+\tinput_dynfield_offset =\n+\t\trte_mbuf_dynfield_register(&input_dynfield_desc);\n+\tif (input_dynfield_offset < 0)\n+\t\trte_exit(EXIT_FAILURE, \"Cannot register mbuf field\\n\");\n+\n \t/* initialize ports */\n \tret = initialize_ports(&app_params, ethdev_mbuf_mempool);\n \n", "prefixes": [ "v3", "13/15" ] }