From: Ingo Molnar <mingo@kernel.org>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: Kay Sievers <kay@vrfy.org>, LKML <linux-kernel@vger.kernel.org>,
Linus Torvalds <torvalds@linux-foundation.org>,
Ingo Molnar <mingo@elte.hu>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Wu Fengguang <fengguang.wu@intel.com>,
Andrew Morton <akpm@linux-foundation.org>,
Joe Perches <joe@perches.com>,
"Paul E. McKenney" <paulmck@us.ibm.com>,
Peter Zijlstra <a.p.zijlstra@chello.nl>
Subject: Re: [PATCH v2] printk: Have printk() never buffer its data
Date: Tue, 26 Jun 2012 22:43:58 +0200 [thread overview]
Message-ID: <20120626204358.GA11771@gmail.com> (raw)
In-Reply-To: <1340639741.27036.337.camel@gandalf.stny.rr.com>
* Steven Rostedt <rostedt@goodmis.org> wrote:
> > It also sounds like quite a lot of wasted headers in the
> > buffer, which we need to carry around and throw away when we
> > reconstruct the line for output again. If we merge them
> > internally we mess around with the sequence numbers, if we
> > merge them in userspace we would need to export the flags to
> > do that.
>
> As I'm still on debian and fedora 14, I have to admit that I
> do not quite understand exactly what you are trying to do with
> systemd or your logging facility. But it seems to become clear
> that printk isn't a tool for it.
>
> What exactly are you trying to record from the kernel? Device
> information, kernel diagnostics, or something else? Maybe
> there should be another facility besides printk that can pass
> what you want to your utilities.
>
> printk is the first line of attack for kernel developers to
> find their bugs. If it becomes bloated and focused on users,
> it will make development of the kernel much more difficult.
>
> Perhaps we should have added a hook or a tracepoint at the
> beginning of printk that your logging facility could attach
> to, [...]
That was my immediate suggestion very early on in the printk
discussion, months ago, when Kay was still giving us nightmares
about UUID printk's and other nonsense.
Adding the printk tracepoint and feeding it to kmesg would have
been a few trivial lines of code plus tracing code reuse, it
would have made a ton of sense and it would have enhanced
tracing and profiling as well, not just kmdesg/syslogd-ng.
I guess the triviality of it must have been one of the reasons
why it wasn't implemented that way ;-)
Today I'd be happy with at least having clean separation of a
direct, relatively simple printk() from whatever special purpose
new logging facilities feed off of printk().
Thanks,
Ingo
prev parent reply other threads:[~2012-06-26 20:44 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-25 13:21 [PATCH v2] printk: Have printk() never buffer its data Steven Rostedt
2012-06-25 13:33 ` Steven Rostedt
2012-06-25 13:56 ` Ingo Molnar
2012-06-25 14:26 ` Steven Rostedt
2012-06-25 15:22 ` Kay Sievers
2012-06-25 15:55 ` Steven Rostedt
2012-06-26 20:43 ` Ingo Molnar [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120626204358.GA11771@gmail.com \
--to=mingo@kernel.org \
--cc=a.p.zijlstra@chello.nl \
--cc=akpm@linux-foundation.org \
--cc=fengguang.wu@intel.com \
--cc=gregkh@linuxfoundation.org \
--cc=joe@perches.com \
--cc=kay@vrfy.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=paulmck@us.ibm.com \
--cc=rostedt@goodmis.org \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox