From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 89A97A0A02; Fri, 16 Apr 2021 20:40:33 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6C1AA161D46; Fri, 16 Apr 2021 20:40:33 +0200 (CEST) Received: from wnew1-smtp.messagingengine.com (wnew1-smtp.messagingengine.com [64.147.123.26]) by mails.dpdk.org (Postfix) with ESMTP id 2DCC8161D3F for ; Fri, 16 Apr 2021 20:40:32 +0200 (CEST) Received: from compute2.internal (compute2.nyi.internal [10.202.2.42]) by mailnew.west.internal (Postfix) with ESMTP id 1FD03149F; Fri, 16 Apr 2021 14:40:29 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute2.internal (MEProxy); Fri, 16 Apr 2021 14:40:30 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=fm1; bh= 8ep4ffKWFRKgGWhk9+qFyH9Wv4QZ5r6DOzvhSAce8KQ=; b=QQEmwD6ShqsWQn4o aYdSLqrbjaQ3pvKJZl2+t9zMgvYGQEiohbKchivWyh2Da4Ep1VdUbMlz9NHoBqYB wyzYkFd9FnA8bBDEgsd5opImJ6v0QQ1spel64Tf7BKXme1peF5n7wqOrFDtv54Vp YVKTbXMxu0LlQI9w4GU0l7lbQRqNHbWAwvZj4MRQqU4HmB6dGlPVBngsC8xYeGib 6LAIgnstq/2Fy4QMwSYxb2Rj0wwcr3uZf/9gRnkbE+pOVdaurQ8Om7EiPdidgC/m p5Zxtm8x1hPFdoKhcUROcMwLmLxv8WoYqj1N0TbfG+/CAJS7cNyDLfMdqxZFD2MK YUJEAg== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; bh=8ep4ffKWFRKgGWhk9+qFyH9Wv4QZ5r6DOzvhSAce8 KQ=; b=S3C3eGINIG9zCzxKZ+2CgbX3TRKtqCGdXx6uqv4xhNiwLLteh0dO+BYQH vewMHTe6ForQy2EQlOXqLre4VpM027/KBEf1DuIYVy0pY29bU/lYwOm6yGblIjqf HO6SmUqrSdQ3bxyCh4doM2Mnyt/v2eyFEp6qQJSy06lEnoiwFW0pxV7CCCbMwCIy W+tAXyoG71+mJ4CTMod9AMYgLzXe/ZGX4h6zuv6K5vnPmrW9vVxihKXeTbZjS5Wi Vptk7xzRCdPB1Tbx7tA4wx7mJOzOpBKLGgIhnmRwnCgiA1W9wizNwAbTduXZ5Gf6 j+XH0FjSNjNSkFRSujye2fkcbJWKQ== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduledrudelhedgudefudcutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd enucfjughrpefhvffufffkjghfggfgtgesthhqredttddtjeenucfhrhhomhepvfhhohhm rghsucfoohhnjhgrlhhonhcuoehthhhomhgrshesmhhonhhjrghlohhnrdhnvghtqeenuc ggtffrrghtthgvrhhnpeekteehtdeivefhieegjeelgedufeejheekkeetueevieeuvdev uedtjeevheevteenucfkphepjeejrddufeegrddvtdefrddukeegnecuvehluhhsthgvrh fuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepthhhohhmrghssehmohhnjhgr lhhonhdrnhgvth X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id D378B108005B; Fri, 16 Apr 2021 14:40:26 -0400 (EDT) From: Thomas Monjalon To: David Marchand , Shepard Siegel , Qiming Yang , Xiao Wang , Rosen Xu , Maxime Coquelin , dev@dpdk.org, "Yigit, Ferruh" Cc: "Mcnamara, John" , Ciara Power , "Min Hu (Connor)" Date: Fri, 16 Apr 2021 20:40:25 +0200 Message-ID: <9694923.JHvA49bgKg@thomas> In-Reply-To: References: <1618051221-19962-1-git-send-email-humin29@huawei.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-dev] [PATCH 0/7] support set thread name X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 13/04/2021 03:12, Min Hu (Connor): > =E5=9C=A8 2021/4/12 15:19, David Marchand =E5=86=99=E9=81=93: > > On Sat, Apr 10, 2021 at 12:40 PM Min Hu (Connor) w= rote: > >> > >> This set of patches support set thread name for debugging. > >> > >> Chengwen Feng (7): > >> net/ark: support set thread name > >> net/ice: support set VSI reset thread name > >> vdpa/ifc: support set notify and vring relay thread name > >> raw/ifpga: support set monitor thread name > >> examples/performance-thread: support set thread name > >> telemetry: support set init threads name > >> examples/vhost_blk: support set ctrl worker thread name > >=20 > > Rather than add those calls, can maintainers check if their component > > can use ctrl threads instead? > > rte_ctrl_thread_create ensures both that the name is set, and that the > > ctrl thread won't run on the same cpu as "datapath" threads. > >=20 > > I also saw some issues with components creating threads. > > I'll post a series addressing those later (net/ark, net/ice not > > detaching/joining created threads + telemetry not checking > > pthread_create failures). > >=20 > Totally agree with David. > By the way, for "telemetry not checking pthread_create failures", I have > sent patches to fix it. Please check it out. > Thanks. Do I understand correctly that we prefer switching to rte_ctrl_thread_create() instead of this patch series adding rte_thread_setname()? Any volunteer to add the use of rte_ctrl_thread_create() in these drivers and examples?