|
Subject: Re: [PATCH 2/4] tracing: add event trace infrastructure Newsgroups: gmane.linux.kernel Date: 2009-02-25 08:28:52 GMT (3 years, 12 weeks, 1 day, 6 hours and 1 minute ago) On Wed, 25 Feb 2009 09:11:18 +0100 Ingo Molnar <mingo <at> elte.hu> wrote: > > * Andrew Morton <akpm <at> linux-foundation.org> wrote: > > > On Tue, 24 Feb 2009 23:08:56 -0500 (EST) Steven Rostedt <rostedt <at> goodmis.org> wrote: > > > > > > Gad, what a lot of stuff. > > > > > > > > Use strncpy_from_user()? > > > > > > > > Use strstrip()? > > > > > > > > Why do we care about leading and trailing whitespace - user error! > > > > > > This is because i want: > > > > > > cat available_events > set_event > > > > > > to work. > > > > :( > > > > Why on earth do we keep on putting all these pretty-printers > > and pretty-parsers into the kernel? I mean, how hard is it > > for userspace to read a text file, do some basic substitutions > > and print it out again? > > Note that there's no mandatory user-space component here - the > final destination is the kernel developer's eyes in 90% of the > cases. These traces get pasted into email, etc. etc. > > So leading spaces, meaningful formatting and general hands-on > usability is important. [ I know, it's a strange concept in the > kernel, we tend to have a perversion for the most unusable and > most inconsistent user interfaces |
|