From: "Jörn Engel" <joern@logfs.org>
To: Borislav Petkov <bp@alien8.de>
Cc: Jeff Moyer <jmoyer@redhat.com>,
linux-kernel@vger.kernel.org,
Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH 5/9] blockconsole: use CON_ALLDATA
Date: Fri, 1 Mar 2013 11:31:28 -0500 [thread overview]
Message-ID: <20130301163128.GA16713@logfs.org> (raw)
In-Reply-To: <20130301175220.GF30938@pd.tnic>
On Fri, 1 March 2013 18:52:20 +0100, Borislav Petkov wrote:
> On Fri, Mar 01, 2013 at 11:20:29AM -0500, Jörn Engel wrote:
> > If you and/or other people are strongly opposed to the CON_ALLDATA
> > patches, I can either keep them private (we will never drop them from
> > our kernel) or add a config option.
>
> We'll do that when someone comes up with a really persuasive reason why
> blockconsole shouldn't log *all* messages. And, whatever we do, I don't
> want to configure/set anything so that blockconsole logs everything - it
> should be the default.
For blockconsole I agree. Netconsole is the interesting one, because
CON_ALLDATA could be considered a regression for it.
Jörn
--
The key to performance is elegance, not battalions of special cases.
-- Jon Bentley and Doug McIlroy
next prev parent reply other threads:[~2013-03-01 17:55 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-28 21:39 [PATCH 0/9] Add blockconsole version 1.1 (try 2) Joern Engel
2013-02-28 21:39 ` [PATCH 1/9] do_mounts: constify name_to_dev_t parameter Joern Engel
2013-02-28 21:39 ` [PATCH 2/9] add blockconsole version 1.1 Joern Engel
2013-02-28 21:39 ` [PATCH 3/9] printk: add CON_ALLDATA console flag Joern Engel
2013-03-01 14:52 ` Jeff Moyer
2013-03-01 14:58 ` Borislav Petkov
2013-03-01 15:08 ` Jeff Moyer
2013-03-01 15:31 ` Borislav Petkov
2013-03-01 15:42 ` Jeff Moyer
2013-03-01 15:58 ` Borislav Petkov
2013-03-01 16:15 ` Jörn Engel
2013-02-28 21:39 ` [PATCH 4/9] netconsole: use CON_ALLDATA Joern Engel
2013-02-28 21:39 ` [PATCH 5/9] blockconsole: " Joern Engel
2013-03-01 16:11 ` Jeff Moyer
2013-03-01 16:20 ` Jörn Engel
2013-03-01 17:52 ` Borislav Petkov
2013-03-01 16:31 ` Jörn Engel [this message]
2013-03-01 17:56 ` Borislav Petkov
2013-02-28 21:39 ` [PATCH 6/9] bcon: add a release work struct Joern Engel
2013-02-28 21:40 ` [PATCH 7/9] bcon: check for hdparm in bcon_tail Joern Engel
2013-02-28 21:40 ` [PATCH 8/9] blockconsole: Allow to pass a device file path to bcon_tail Joern Engel
2013-02-28 21:40 ` [PATCH 9/9] bcon: remove version 1.0 support Joern Engel
2013-03-01 17:15 ` [PATCH 0/9] Add blockconsole version 1.1 (try 2) Takashi Iwai
2013-03-01 16:22 ` Jörn Engel
2013-03-05 17:36 ` Takashi Iwai
2013-03-06 19:49 ` Jörn Engel
2013-03-07 18:46 ` Takashi Iwai
2013-03-07 18:51 ` [PATCH 0/3] blockconsole: make it a module Takashi Iwai
2013-03-07 18:51 ` [PATCH 1/3] Export sched_setscheduler_nocheck() Takashi Iwai
2013-03-07 18:51 ` [PATCH 2/3] block/partitions: Support dynamic addition of partition checkers Takashi Iwai
2013-03-07 18:51 ` [PATCH 3/3] blockconsole: Allow to be a module Takashi Iwai
2013-03-01 17:17 ` [PATCH 1/3] blockconsole: Fix undefined MAX_RT_PRIO Takashi Iwai
2013-03-01 17:17 ` [PATCH 2/3] blockconsole: Rename device_lock with bc_device_lock Takashi Iwai
2013-03-01 17:17 ` [PATCH 3/3] blockconsole: Mark a local work struct static Takashi Iwai
2013-03-20 22:52 ` [PATCH 0/9] Add blockconsole version 1.1 (try 2) Borislav Petkov
2013-03-21 6:30 ` Takashi Iwai
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=20130301163128.GA16713@logfs.org \
--to=joern@logfs.org \
--cc=akpm@linux-foundation.org \
--cc=bp@alien8.de \
--cc=jmoyer@redhat.com \
--cc=linux-kernel@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.