public inbox for linuxppc-dev@ozlabs.org 
 help / color / mirror / Atom feed
From: Grant Erickson <gerickson@nuovations•com>
To: Benjamin Herrenschmidt <benh@kernel•crashing.org>
Cc: linuxppc-dev@ozlabs•org, Stefan Roese <sr@denx•de>,
	Wolfgang Denx <wd@denx•de>,
	linux-embedded@vger•kernel.org
Subject: Re: [PATCH/RFC] Add Alternative Log Buffer Support for printk Messages
Date: Tue, 06 Jan 2009 18:11:11 -0800	[thread overview]
Message-ID: <C5894F3F.13C20%gerickson@nuovations.com> (raw)
In-Reply-To: <1231286642.14860.35.camel@pasglop>

On 1/6/09 4:04 PM, Benjamin Herrenschmidt wrote:
> On Tue, 2008-11-25 at 10:34 -0800, Grant Erickson wrote:
>> This merges support for the previously DENX-only kernel feature of
>> specifying an alternative, "external" buffer for kernel printk
>> messages and their associated metadata. In addition, this ports
>> architecture support for this feature from arch/ppc to arch/powerpc.
>> 
>> Signed-off-by: Grant Erickson <gerickson@nuovations•com>
> 
> Considering the extensive changes to generic code, this patch will
> have to be submitted via the linux-kernel mailing list.
> 
> I suggest you split the generic core change from the powerpc specific
> implementation.
> 
> I'm not sure whether I like the idea myself or not there, so you'll have
> to convince the powers that be to take it.

Ben:

Thanks for the feedback. Matt Sealey had some good feedback
<http://ozlabs.org/pipermail/linuxppc-dev/2008-November/065594.html> which I
have on my to-do list to evaluate.

In the interim, I'll hold off on pushing up to linux-kernel until I've done
that.

Regards,

Grant

      reply	other threads:[~2009-01-07  2:11 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-25 18:34 [PATCH/RFC] Add Alternative Log Buffer Support for printk Messages Grant Erickson
2008-11-25 18:53 ` Matt Sealey
2008-11-25 18:55   ` Josh Boyer
2008-11-25 19:01     ` Matt Sealey
2008-11-25 19:04     ` Grant Erickson
2008-11-25 19:31       ` Matt Sealey
2008-11-25 19:51         ` Bill Gatliff
2008-11-25 20:07           ` Matt Sealey
2008-11-25 20:17             ` Grant Likely
2008-11-25 20:46               ` Matt Sealey
2008-11-25 20:19             ` Bill Gatliff
2008-11-25 20:54               ` David VomLehn
2008-11-25 21:45                 ` David Brownell
2008-11-26 20:57                   ` Matt Sealey
2008-11-25 20:14           ` Grant Likely
2008-11-25 21:05         ` Wolfgang Denk
2008-11-25 21:14           ` Matt Sealey
2008-11-26  1:23 ` Mike Frysinger
2009-01-07  0:04 ` Benjamin Herrenschmidt
2009-01-07  2:11   ` Grant Erickson [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=C5894F3F.13C20%gerickson@nuovations.com \
    --to=gerickson@nuovations$(echo .)com \
    --cc=benh@kernel$(echo .)crashing.org \
    --cc=linux-embedded@vger$(echo .)kernel.org \
    --cc=linuxppc-dev@ozlabs$(echo .)org \
    --cc=sr@denx$(echo .)de \
    --cc=wd@denx$(echo .)de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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