mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Vegard Nossum" <vegard.nossum@gmail.com>
To: "Jan Engelhardt" <jengelh@computergmbh.de>
Cc: LKML <linux-kernel@vger.kernel.org>,
	"Kyle Moffett" <mrmacman_g4@mac.com>,
	"Michael Holzheu" <holzheu@linux.vnet.ibm.com>,
	"Joe Perches" <joe@perches.com>, "Rob Landley" <rob@landley.net>,
	"Dick Streefland" <dick.streefland@altium.nl>,
	"Geert Uytterhoeven" <Geert.Uytterhoeven@sonycom.com>,
	"Jesse Barnes" <jesse.barnes@intel.com>,
	"Arnd Bergmann" <arnd@arndb.de>,
	"Emil Medve" <Emilian.Medve@freescale.com>,
	"Stephen Hemminger" <shemminger@linux-foundation.org>,
	"linux@horizon.com" <linux@horizon.com>
Subject: Re: [RFC] New kernel-message logging API (take 2)
Date: Fri, 28 Sep 2007 13:59:19 +0200	[thread overview]
Message-ID: <19f34abd0709280459w13266dd4n24b38f90fe5c0ccf@mail.gmail.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0709281139460.25099@fbirervta.pbzchgretzou.qr>

On 9/28/07, Jan Engelhardt <jengelh@computergmbh.de> wrote:
>
> On Sep 27 2007 23:18, Vegard Nossum wrote:
> >  * kprint_<level>() is better than kprint(<level), ) because in the
> >    case of the default log-level, the argument can be omitted.
> >  * Memory allocated for entries and arguments is done in a ring-buffer
> >    with variable-sized chunks. Arguments are chained with a singly-
> >    linked list.
> >  * Use SUBSYSTEM and KBUILD_MODNAME
> >  * Rename kprint buffers to kprint blocks
>
> This is overall, quite a lot. I suggest one-thing-at-a-time,
> starting with kprint_<level>() that is compiled out if desired
> and no fancy block or newline stuff.
>
> _Then_, will see how it flies. All of this smells like a bit of
> overdesigning, aka http://en.wikipedia.org/wiki/YAGNI

Well, that's why I'm writing it down before actually coding it.
Designing is also trying to make things fit together without actually
having the physical parts at hand. So I'm trying to make an interface
that CAN support multi-line blocks in the future, since it's obviously
a desired (and currently missing) feature.

But I agree; It *is* hard to see how multi-line blocks can be
implemented without actually spelling it out in code. I've tried to do
it, and failed. Until a brilliant solution comes up, I'll skip it.

Vegard

  reply	other threads:[~2007-09-28 11:59 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-27 21:18 Vegard Nossum
2007-09-28  1:38 ` linux
2007-09-28  7:31 ` Geert Uytterhoeven
2007-09-28  7:44   ` Kyle Moffett
2007-09-28 11:49     ` Vegard Nossum
2007-09-28 11:46   ` Vegard Nossum
2007-09-28  8:22 ` Dick Streefland
2007-09-28  9:45 ` Jan Engelhardt
2007-09-28 11:59   ` Vegard Nossum [this message]
2007-09-28 12:11 ` Vegard Nossum
2007-09-28 13:30   ` Miguel Ojeda
2007-09-28 13:55     ` Vegard Nossum
2007-09-28 14:00       ` Miguel Ojeda
2007-09-28 16:30   ` Rob Landley
2007-09-28 16:41     ` Vegard Nossum

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=19f34abd0709280459w13266dd4n24b38f90fe5c0ccf@mail.gmail.com \
    --to=vegard.nossum@gmail.com \
    --cc=Emilian.Medve@freescale.com \
    --cc=Geert.Uytterhoeven@sonycom.com \
    --cc=arnd@arndb.de \
    --cc=dick.streefland@altium.nl \
    --cc=holzheu@linux.vnet.ibm.com \
    --cc=jengelh@computergmbh.de \
    --cc=jesse.barnes@intel.com \
    --cc=joe@perches.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@horizon.com \
    --cc=mrmacman_g4@mac.com \
    --cc=rob@landley.net \
    --cc=shemminger@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

all inboxes | Powered by JetHome®