From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ingo Molnar Subject: Re: [PATCH 1/3] ftrace: add function tracing to single thread Date: Wed, 26 Nov 2008 07:32:01 +0100 Message-ID: <20081126063201.GF9732@elte.hu> References: <20081125223456.976670734@goodmis.org> <20081125144221.66eb99ad.akpm@linux-foundation.org> <20081125153104.ecdceed4.akpm@linux-foundation.org> <1227660839.12109.52.camel@nimitz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: "Eric W. Biederman" Cc: Steven Rostedt , Dave Hansen , Andrew Morton , containers@lists.osdl.org, fweisbec@gmail.com, LKML , srostedt@redhat.com, Sukadev Bhattiprolu , "Serge E. Hallyn" List-Id: containers.vger.kernel.org * Eric W. Biederman wrote: > Steven Rostedt writes: > > > On Tue, 25 Nov 2008, Eric W. Biederman wrote: > > > > I'm speechless too. > > I'm a bit tired so probably am pushing to hard. > > At the same time I don't see a single reason not to use struct pid > for what it was designed for. Identifying tasks. pid_t's really > only belong at the border. > > I can see in the tracer when grabbing numbers you might not be able > to follow pointers. For that I see justification for using > task->pid. For the comparison I just don't see it. i dont see the point of the complexity you are advocating. 99.9% of the users run a unique PID space. Tracing is about keeping stuff simple. On containers we could also trace the namespace ID (is there an easy ID for the namespace, as an easy extension to the very nice PID concept that Unix introduced decades ago?) and be done with it. Ingo