mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: holzheu <holzheu@linux.vnet.ibm.com>
To: Gerrit Huizenga <gh@us.ibm.com>
Cc: Jan Kara <jack@suse.cz>,
	Andrew Morton <akpm@linux-foundation.org>,
	linux-kernel@vger.kernel.org, randy.dunlap@oracle.com,
	gregkh@suse.de, mtk-manpages@gmx.net, schwidefsky@de.ibm.com,
	heiko.carstens@de.ibm.com,
	"lf_kernel_messages@linux-foundation.org" 
	<lf_kernel_messages@linux-foundation.org>
Subject: Re: [RFC/PATCH] Documentation of kernel messages
Date: Mon, 18 Jun 2007 14:55:54 +0200	[thread overview]
Message-ID: <1182171354.11400.40.camel@localhost.localdomain> (raw)
In-Reply-To: <E1HzGkR-0008DO-Un@w-gerrit.beaverton.ibm.com>

Hi Gerrit,

The common thing of your and our approach is, that we need an ID to
identify a message either by:

* Using hashes + component name (maybe)
* Or using hand-made message numbers + component name. Numbers have
  to be unique within the kernel component

I think the main difference of your approach is, that documentation will
be maintained outside the kernel. This has advantages, but also
disadvantages:

(+) Developers have less work to do :-)
(+) Kernel sources and patches have less lines of code, since message
    descriptions are not contained in the kernel sources.
(+) You can support multiple languages

(-) Hard to keep printks and descriptions up-to-date. If using hashes,
    each typo fix in a printk breaks the link to your documentation.
(-) Since the developer knows the meaning of a printk best, he probably
    would be the right person to write the description.
(-) For every Distribution or vanilla kernel you probably need separate
    external message databases.

I like the fact, that every printk gets an ID in your approach and you
do not need additional printk macros.

Maybe both approaches are not mutually exclusive. If developers would
like to document their messages within the kernel sources, they still
could use kernel-doc and our KMSG_DOC macro do do that.

I admit, that I am not sure about the the best solution here.

Michael



  parent reply	other threads:[~2007-06-18 12:59 UTC|newest]

Thread overview: 133+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-13 15:06 holzheu
2007-06-13 16:37 ` Dave Hansen
2007-06-13 17:11   ` Sam Ravnborg
2007-06-13 17:42   ` holzheu
2007-06-13 16:50 ` Valdis.Kletnieks
2007-06-13 18:09   ` Rob Landley
2007-06-13 18:11   ` holzheu
2007-06-14  8:47     ` Krzysztof Halasa
2007-06-15  8:49   ` Jan Engelhardt
2007-06-13 17:16 ` Alexey Dobriyan
2007-06-13 17:46   ` holzheu
2007-06-13 17:51 ` Greg KH
2007-06-13 18:18   ` holzheu
2007-06-13 18:32     ` Greg KH
2007-06-13 18:49       ` Dave Hansen
2007-06-13 18:49       ` Kok, Auke
2007-06-14  8:17       ` Martin Schwidefsky
2007-06-13 19:04     ` Joe Perches
2007-06-14  2:54       ` Valdis.Kletnieks
2007-06-14  7:05         ` H. Peter Anvin
2007-06-15  8:51       ` Jan Engelhardt
2007-06-13 18:15 ` Andrew Morton
2007-06-14  8:31   ` Martin Schwidefsky
2007-06-14  9:41   ` Jan Kara
2007-06-14 10:38     ` holzheu
2007-06-14 11:47       ` holzheu
2007-06-14 12:26         ` Jan Kara
2007-06-14 15:22           ` holzheu
2007-06-15 18:42       ` Gerrit Huizenga
2007-06-15 18:51         ` Randy Dunlap
2007-06-15 19:27           ` Gerrit Huizenga
2007-06-15 20:01           ` Greg KH
2007-06-18 12:55         ` holzheu [this message]
2007-06-18 13:12           ` Arjan van de Ven
2007-06-18 13:33             ` Jan Kara
2007-06-18 13:53             ` holzheu
2007-06-19  1:36               ` Arjan van de Ven
2007-06-19  8:51                 ` holzheu
2007-06-19 19:24                   ` Arjan van de Ven
2007-06-19 11:31                 ` holzheu
2007-06-19  5:41           ` [Lf_kernel_messages] " Kunai, Takashi
2007-06-18 13:11         ` Sam Ravnborg
2007-06-18 15:35           ` Randy Dunlap
2007-06-19  0:13         ` Tim Bird
2007-06-19  3:52           ` Gerrit Huizenga
2007-06-15 12:40     ` Pavel Machek
2007-06-18 13:42       ` holzheu
2007-06-18 14:02         ` Pavel Machek
2007-06-25 13:48   ` Documentation of kernel messages (Summary) Michael Holzheu
2007-06-25 15:44     ` Rob Landley
2007-06-25 18:05       ` Sam Ravnborg
2007-06-27 15:11       ` Michael Holzheu
2007-07-09  5:15         ` Kunai, Takashi
2007-07-09  5:36           ` H. Peter Anvin
2007-07-09 16:48             ` Rob Landley
2007-07-09 17:18               ` Gerrit Huizenga
2007-07-10 16:12                 ` Rob Landley
2007-07-11 12:15                   ` Michael Holzheu
2007-07-11 14:26                   ` Li Yang
2007-07-11 18:13                     ` Rob Landley
2007-07-11 21:32                       ` Theodore Tso
2007-07-13 10:53                         ` Alan Cox
2007-07-13 12:49                           ` Li Yang
2007-07-13 13:43                             ` Theodore Tso
2007-07-13 13:25                           ` Stefan Richter
2007-07-13 21:10                             ` Valdis.Kletnieks
2007-07-13 18:05                           ` Rob Landley
2007-07-14  3:54                             ` Randy Dunlap
2007-07-15  1:56                               ` Rob Landley
2007-07-15 16:28                                 ` Randy Dunlap
2007-07-16 19:53                                   ` Rob Landley
2007-08-03 18:11                                     ` Randy Dunlap
2007-08-03 19:32                                       ` Rob Landley
2007-08-04  3:50                                         ` Greg KH
2007-08-04 19:02                                           ` Rob Landley
2007-08-04  3:52                                         ` Greg KH
2007-08-04 18:54                                           ` Rob Landley
2007-08-04 20:04                                             ` Stefan Richter
2007-08-06 15:50                                               ` Rob Landley
2007-07-11 21:53                       ` Valdis.Kletnieks
2007-07-12 16:56                         ` Rob Landley
2007-07-12 13:53                       ` Li Yang-r58472
2007-07-12 16:05                         ` [PATCH] Chinese Language Maintainer Rob Landley
2007-07-12 19:52                           ` Geert Uytterhoeven
2007-07-12 20:02                           ` Pavel Machek
2007-07-13  7:42                             ` Geert Uytterhoeven
2007-07-13 16:06                             ` Rob Landley
2007-07-13 12:43                           ` Li Yang
2007-07-13 17:52                             ` Rob Landley
2007-07-15 14:42                               ` Li Yang
2007-07-15 18:12                                 ` H. Peter Anvin
2007-07-15 18:50                                   ` Alan Cox
2007-07-15 19:11                                     ` H. Peter Anvin
2007-07-15 20:25                                       ` Rik van Riel
2007-07-16  4:40                                     ` Ganesan Rajagopal
2007-07-16 21:43                                       ` Alan Cox
2007-07-17  7:19                                         ` Ganesan Rajagopal
2007-07-15 18:53                                   ` Li Yang
2007-07-15 20:25                                   ` Rene Herman
2007-07-16  9:17                                   ` Dr. Keith G. Bowden
2007-07-16  2:49                                 ` Tsugikazu Shibata
2007-07-16  3:03                                   ` H. Peter Anvin
2007-07-17 16:24                                   ` Li Yang
2007-07-17 21:06                                     ` Rob Landley
2007-07-12 16:41                         ` Documentation of kernel messages (Summary) Tsugikazu Shibata
2007-07-12 17:35                         ` Rob Landley
2007-07-13  2:54                           ` Tsugikazu Shibata
2007-07-13 17:12                             ` Rob Landley
2007-07-14  1:46                               ` Tsugikazu Shibata
2007-07-15  2:12                                 ` Rob Landley
2007-07-15 16:46                                   ` Tsugikazu Shibata
2007-07-17 16:06                                     ` Rob Landley
2007-07-17 23:30                                       ` Tsugikazu Shibata
2007-07-13 11:54                           ` Li Yang
2007-07-15 20:30                             ` Rik van Riel
2007-07-09 18:33               ` Adrian Bunk
2007-07-10 16:25                 ` Rob Landley
2007-07-10 18:54                   ` Adrian Bunk
2007-07-10 19:53                     ` Rob Landley
2007-07-17  0:31                   ` Tim Bird
2007-07-17  1:17                     ` H. Peter Anvin
2007-07-17 16:42                       ` Rob Landley
2007-07-17 16:29                     ` Rob Landley
2007-07-10  7:59               ` Li Yang-r58472
2007-07-10  6:38             ` Dave Young
2007-07-09  7:01           ` Oliver Neukum
2007-07-09 22:59             ` Satyam Sharma
2007-07-10  6:15               ` Oliver Neukum
2007-07-09 18:10       ` [Lf_kernel_messages] " Theodore Tso
2007-07-09 22:12         ` Alan Cox
2007-07-10 16:50         ` Rob Landley
2007-06-13 18:23 ` [RFC/PATCH] Documentation of kernel messages David Miller
2007-06-13 18:27   ` holzheu

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=1182171354.11400.40.camel@localhost.localdomain \
    --to=holzheu@linux.vnet.ibm.com \
    --cc=akpm@linux-foundation.org \
    --cc=gh@us.ibm.com \
    --cc=gregkh@suse.de \
    --cc=heiko.carstens@de.ibm.com \
    --cc=jack@suse.cz \
    --cc=lf_kernel_messages@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mtk-manpages@gmx.net \
    --cc=randy.dunlap@oracle.com \
    --cc=schwidefsky@de.ibm.com \
    /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®