Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758056AbZKJUQv (ORCPT ); Tue, 10 Nov 2009 15:16:51 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1758036AbZKJUQu (ORCPT ); Tue, 10 Nov 2009 15:16:50 -0500 Received: from e4.ny.us.ibm.com ([32.97.182.144]:36457 "EHLO e4.ny.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757992AbZKJUQs (ORCPT ); Tue, 10 Nov 2009 15:16:48 -0500 Subject: Re: [RFC][PATCH] Add prctl to set sibling thread names From: john stultz To: KOSAKI Motohiro Cc: Sean Foley , Andrew Morton , Andi Kleen , Arjan van de Ven , Darren Hart , lkml , Mike Fulton In-Reply-To: <20091110141406.3603.A69D9226@jp.fujitsu.com> References: <20091105143959.2093.A69D9226@jp.fujitsu.com> <20091110141406.3603.A69D9226@jp.fujitsu.com> Content-Type: text/plain; charset="ISO-8859-1" Date: Tue, 10 Nov 2009 12:16:49 -0800 Message-ID: <1257884209.3139.17.camel@localhost> Mime-Version: 1.0 X-Mailer: Evolution 2.28.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2024 Lines: 53 On Tue, 2009-11-10 at 14:27 +0900, KOSAKI Motohiro wrote: > Hi Sean, John, > > > Kosaki, > > Here are a couple of use cases previously posted to this thread on the linux kernel mailing list: > > > > dispatch thread adds context to thread names: > > http://marc.info/?l=linux-kernel&m=125660141231348&w=2 > > > > java language support: > > http://marc.info/?l=linux-kernel&m=125666430720863&w=2 > > > > > > > > Here are some various specific use cases from the web: > > > > Attaching additional info to thread names when used for different purposes: > > http://osdir.com/ml/java.jsr.166-concurrency/2006-12/msg00105.html > > > > Threads obtained from thread pools being reassigned new names: > > http://haacked.com/archive/2004/06/07/546.aspx > > http://bytes.com/topic/c-sharp/answers/637152-naming-backgroundworker-thread > > > > Renaming threads scattered across third-party libraries by enumerating them and renaming them dynamically: > > http://stackoverflow.com/questions/467224/renaming-threads-in-java > > Okey, good explanation. thanks! > > So, I would suggested to extend /proc/{pid}/cmdline instead using task->comm. > because > - task->comm has nasty locking rule. It is harder to change SMP safe. I cc'ed you on the updated patch which addresses this. Please let me know if you have specific concerns there. > - ps (and other procps tools) already support /proc/{pid}/cmdline. > - task->comm is restrected 16 character length, /proc/cmdline isn't. Part of the reason to use comm is that most tools like perf or oprofile, use comm, instead of cmd. Additionally, looking at cmdline, that's per mm not per task, right? So it wouldn't really work for thread names. Please correct me if I'm not seeing what you're really suggesting. thanks -john -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/