Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/131384/?format=api
{ "id": 131384, "url": "http://patchwork.dpdk.org/api/patches/131384/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20230913103600.1935292-10-thomas@monjalon.net/", "project": { "id": 1, "url": "http://patchwork.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": "<20230913103600.1935292-10-thomas@monjalon.net>", "list_archive_url": "https://inbox.dpdk.org/dev/20230913103600.1935292-10-thomas@monjalon.net", "date": "2023-09-13T10:34:28", "name": "[v2,09/11] test: convert threads creation", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "2788c05589198c83b661596058ed7d0dc93499d3", "submitter": { "id": 685, "url": "http://patchwork.dpdk.org/api/people/685/?format=api", "name": "Thomas Monjalon", "email": "thomas@monjalon.net" }, "delegate": { "id": 1, "url": "http://patchwork.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/20230913103600.1935292-10-thomas@monjalon.net/mbox/", "series": [ { "id": 29493, "url": "http://patchwork.dpdk.org/api/series/29493/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=29493", "date": "2023-09-13T10:34:19", "name": "rework thread management", "version": 2, "mbox": "http://patchwork.dpdk.org/series/29493/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/131384/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/131384/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 AF98742588;\n\tWed, 13 Sep 2023 12:38:08 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id D540940A73;\n\tWed, 13 Sep 2023 12:38:02 +0200 (CEST)", "from wout5-smtp.messagingengine.com (wout5-smtp.messagingengine.com\n [64.147.123.21]) by mails.dpdk.org (Postfix) with ESMTP id 66B794064A\n for <dev@dpdk.org>; Wed, 13 Sep 2023 12:38:00 +0200 (CEST)", "from compute4.internal (compute4.nyi.internal [10.202.2.44])\n by mailout.west.internal (Postfix) with ESMTP id F357F3200413;\n Wed, 13 Sep 2023 06:37:58 -0400 (EDT)", "from mailfrontend1 ([10.202.2.162])\n by compute4.internal (MEProxy); Wed, 13 Sep 2023 06:37:59 -0400", "by mail.messagingengine.com (Postfix) with ESMTPA; Wed,\n 13 Sep 2023 06:37:56 -0400 (EDT)" ], "DKIM-Signature": [ "v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h=\n cc:cc:content-transfer-encoding:content-type:content-type:date\n :date:from:from:in-reply-to:in-reply-to:message-id:mime-version\n :references:reply-to:sender:subject:subject:to:to; s=fm2; t=\n 1694601478; x=1694687878; bh=yuYb/FScGlE53o8sIBUUC4AbBWAHIA9MsQv\n NXxUbr+k=; b=SsHTuO/ctEtEqbs58CbNHRl8pR4D5YbMhiVtAmSq/+cQDlMvs6c\n wHXAOrcwCRJseSkpp6t0DVoMhlQgEw/NajWAy9kGL9F9uHKsGIPN4FJlSltOHJzY\n mcVPLMs3MEDR5pX1+aMPDF1I3lG1Oid25UNLeOh1YgQEJkTq+UWVTRjEAj7YjXxT\n WhsksrX7bQSLIfSOYP1WyNoUeLHp+3rYRLkhu7Jrzu2IBo1kipoDEgwXOfgP3h3L\n xMS6kgBDJyNhS0gGmkM3bjwdc4jaO8pjBLU/N97ay9ljCNB9fOIRE596Hl1jHhpC\n Qr2/zjHad1u83UT1s4Ma30jqCdG/+iamwow==", "v=1; a=rsa-sha256; c=relaxed/relaxed; d=\n messagingengine.com; h=cc:cc:content-transfer-encoding\n :content-type:content-type:date:date:feedback-id:feedback-id\n :from:from:in-reply-to:in-reply-to:message-id:mime-version\n :references:reply-to:sender:subject:subject:to:to:x-me-proxy\n :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm2; t=\n 1694601478; x=1694687878; bh=yuYb/FScGlE53o8sIBUUC4AbBWAHIA9MsQv\n NXxUbr+k=; b=pLoMKEeEd71/ug/91NS/zEfwfGj1Gi+OTyIAdTQGNam9a8+lz7L\n fJHvdL2UhNbNtkZcaZhqbb3QO6i2gLv+LHao2RL8VYYn+SpoamtH0WzOgSneO1Hc\n H5okxwcXsIB1Gm6iH2jny88blTD4hWBXJT4PpB4M/1Hq1ao/htWpNEI2CIshEPEx\n HMlBapq+/pPdBEYzRRIsky5ON+bYWoOaBYtgqvoVRvqTMMqq/KyrLgFMWwn8pKxv\n 0qkJNvYTJBi1x71vbLRy4GD778JsnDCwfmhp+g8jD+3vnhIgiwfwKu6Hqc8Vlu0K\n Gz4MX8wDqyNvsoGrufD7+UGTDtvoOOxfDWQ==" ], "X-ME-Sender": "<xms:BpEBZTzEMtwEDIV0mHzbh4S9yrDIlGDgwhgPoV1j8NGvDXMxIAZrzg>\n <xme:BpEBZbSRMZUJy46dRVBCe5sqBBaycZ5h4A6SjAXy5dwHX5qaTeGwGVnYASEwNmccG\n Lne9yXmY6aRcFg31w>", "X-ME-Received": "\n <xmr:BpEBZdUZU3CnS5pgHAFeOOQDMMGgFuIAh0LsLQzcjDlo6uEn0bKHtkCiDXd457n8qSlnnQ>", "X-ME-Proxy-Cause": "\n gggruggvucftvghtrhhoucdtuddrgedviedrudeikedgfedtucetufdoteggodetrfdotf\n fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen\n uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne\n cujfgurhephffvvefufffkofgjfhggtgfgsehtkeertdertdejnecuhfhrohhmpefvhhho\n mhgrshcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqne\n cuggftrfgrthhtvghrnheptdeiffffgeefteelhfekueetueduffejgfelvedtieejudek\n gfevheetheehgeejnecuvehluhhsthgvrhfuihiivgepudenucfrrghrrghmpehmrghilh\n hfrhhomhepthhhohhmrghssehmohhnjhgrlhhonhdrnhgvth", "X-ME-Proxy": "<xmx:BpEBZdjJ1keL0In9sD-nW_-e-m4IDJqbTKEBZ8U5yVrnltpCjzkMCw>\n <xmx:BpEBZVDFLy9YhKn0A5hfWwCR-8WXCk4WTxcZ-o_M91NrrY5B9dDCqQ>\n <xmx:BpEBZWJ_Q8mybf6sJpc05clX5QeooUF5v-ZBiA7lOKhFqb-Ff9OM_A>\n <xmx:BpEBZY6T86QKEJpCXhHMNKXX7b6m6AGobuImF_Owe1vod79fczR6Wg>", "Feedback-ID": "i47234305:Fastmail", "From": "Thomas Monjalon <thomas@monjalon.net>", "To": "dev@dpdk.org", "Cc": "Tyler Retzlaff <roretzla@linux.microsoft.com>,\n David Marchand <david.marchand@redhat.com>,\n Ferruh Yigit <ferruh.yigit@amd.com>,\n =?utf-8?q?Morten_Br=C3=B8rup?= <mb@smartsharesystems.com>,\n Reshma Pattan <reshma.pattan@intel.com>,\n Stephen Hemminger <stephen@networkplumber.org>", "Subject": "[PATCH v2 09/11] test: convert threads creation", "Date": "Wed, 13 Sep 2023 12:34:28 +0200", "Message-ID": "<20230913103600.1935292-10-thomas@monjalon.net>", "X-Mailer": "git-send-email 2.42.0", "In-Reply-To": "<20230913103600.1935292-1-thomas@monjalon.net>", "References": "<20230906162226.1618088-1-thomas@monjalon.net>\n <20230913103600.1935292-1-thomas@monjalon.net>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "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": "Calls to pthread for thread creation are replaced with the rte_thread API.\n\nSigned-off-by: Thomas Monjalon <thomas@monjalon.net>\nAcked-by: Morten Brørup <mb@smartsharesystems.com>\nAcked-by: Tyler Retzlaff <roretzla@linux.microsoft.com>\n---\n app/test/process.h | 10 +++++-----\n app/test/test_lcores.c | 9 ++++-----\n app/test/test_pdump.c | 6 +++---\n app/test/test_pdump.h | 2 +-\n 4 files changed, 13 insertions(+), 14 deletions(-)", "diff": "diff --git a/app/test/process.h b/app/test/process.h\nindex 1f073b9c5c..af7bc3e0de 100644\n--- a/app/test/process.h\n+++ b/app/test/process.h\n@@ -28,8 +28,8 @@\n \n #ifdef RTE_LIB_PDUMP\n #ifdef RTE_NET_RING\n-#include <pthread.h>\n-extern void *send_pkts(void *empty);\n+#include <rte_thread.h>\n+extern uint32_t send_pkts(void *empty);\n extern uint16_t flag_for_send_pkts;\n #endif\n #endif\n@@ -49,7 +49,7 @@ process_dup(const char *const argv[], int numargs, const char *env_value)\n \tchar path[32];\n #ifdef RTE_LIB_PDUMP\n #ifdef RTE_NET_RING\n-\tpthread_t thread;\n+\trte_thread_t thread;\n \tint rc;\n #endif\n #endif\n@@ -136,7 +136,7 @@ process_dup(const char *const argv[], int numargs, const char *env_value)\n #ifdef RTE_LIB_PDUMP\n #ifdef RTE_NET_RING\n \tif ((strcmp(env_value, \"run_pdump_server_tests\") == 0)) {\n-\t\trc = pthread_create(&thread, NULL, &send_pkts, NULL);\n+\t\trc = rte_thread_create(&thread, NULL, send_pkts, NULL);\n \t\tif (rc != 0) {\n \t\t\trte_panic(\"Cannot start send pkts thread: %s\\n\",\n \t\t\t\t strerror(rc));\n@@ -151,7 +151,7 @@ process_dup(const char *const argv[], int numargs, const char *env_value)\n #ifdef RTE_NET_RING\n \tif ((strcmp(env_value, \"run_pdump_server_tests\") == 0)) {\n \t\tflag_for_send_pkts = 0;\n-\t\tpthread_join(thread, NULL);\n+\t\trte_thread_join(thread, NULL);\n \t}\n #endif\n #endif\ndiff --git a/app/test/test_lcores.c b/app/test/test_lcores.c\nindex 0c96faa4cb..22225a9fd3 100644\n--- a/app/test/test_lcores.c\n+++ b/app/test/test_lcores.c\n@@ -2,7 +2,6 @@\n * Copyright (c) 2020 Red Hat, Inc.\n */\n \n-#include <pthread.h>\n #include <string.h>\n \n #include <rte_common.h>\n@@ -341,7 +340,7 @@ test_non_eal_lcores_callback(unsigned int eal_threads_count)\n \treturn -1;\n }\n \n-static void *ctrl_thread_loop(void *arg)\n+static uint32_t ctrl_thread_loop(void *arg)\n {\n \tstruct thread_context *t = arg;\n \n@@ -350,7 +349,7 @@ static void *ctrl_thread_loop(void *arg)\n \t/* Set the thread state to DONE */\n \tt->state = Thread_DONE;\n \n-\treturn NULL;\n+\treturn 0;\n }\n \n static int\n@@ -362,8 +361,8 @@ test_ctrl_thread(void)\n \t/* Create one control thread */\n \tt = &ctrl_thread_context;\n \tt->state = Thread_INIT;\n-\tif (rte_ctrl_thread_create((pthread_t *)&t->id, \"dpdk-test-ctrlt\",\n-\t\t\t\t\tNULL, ctrl_thread_loop, t) != 0)\n+\tif (rte_thread_create_control(&t->id, \"dpdk-test-ctrlt\",\n+\t\t\t\tctrl_thread_loop, t) != 0)\n \t\treturn -1;\n \n \t/* Wait till the control thread exits.\ndiff --git a/app/test/test_pdump.c b/app/test/test_pdump.c\nindex ea03056b47..adf49f0b54 100644\n--- a/app/test/test_pdump.c\n+++ b/app/test/test_pdump.c\n@@ -136,8 +136,8 @@ test_pdump_uninit(void)\n \treturn ret;\n }\n \n-void *\n-send_pkts(void *empty)\n+uint32_t\n+send_pkts(void *empty __rte_unused)\n {\n \tint ret = 0;\n \tstruct rte_mbuf *pbuf[NUM_PACKETS] = { };\n@@ -161,7 +161,7 @@ send_pkts(void *empty)\n \n \trte_eth_dev_stop(portid);\n \ttest_put_mbuf_to_pool(mp, pbuf);\n-\treturn empty;\n+\treturn 0;\n }\n \n /*\ndiff --git a/app/test/test_pdump.h b/app/test/test_pdump.h\nindex abef9a85ec..8746d61269 100644\n--- a/app/test/test_pdump.h\n+++ b/app/test/test_pdump.h\n@@ -9,7 +9,7 @@\n #define NUM_ITR 3\n \n /* sample test to send packets to the pdump client recursively */\n-void *send_pkts(void *port);\n+uint32_t send_pkts(void *empty);\n \n /* Sample test to create setup for the pdump server tests */\n int test_pdump_init(void);\n", "prefixes": [ "v2", "09/11" ] }