lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 26 Nov 2008 07:32:01 +0100 From: Ingo Molnar <mingo@...e.hu> To: "Eric W. Biederman" <ebiederm@...ssion.com> Cc: Steven Rostedt <rostedt@...dmis.org>, Dave Hansen <dave@...ux.vnet.ibm.com>, Andrew Morton <akpm@...ux-foundation.org>, containers@...ts.osdl.org, fweisbec@...il.com, LKML <linux-kernel@...r.kernel.org>, srostedt@...hat.com, Sukadev Bhattiprolu <sukadev@...ux.vnet.ibm.com>, "Serge E. Hallyn" <serue@...ibm.com> Subject: Re: [PATCH 1/3] ftrace: add function tracing to single thread * Eric W. Biederman <ebiederm@...ssion.com> wrote: > Steven Rostedt <rostedt@...dmis.org> 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists