public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@osdl.org>
To: "Randy.Dunlap" <rdunlap@xenotime.net>
Cc: "Rafael J. Wysocki" <rjw@sisk.pl>, Greg KH <greg@kroah.com>,
	Pavel Machek <pavel@ucw.cz>, LKML <linux-kernel@vger.kernel.org>
Subject: Re: [RFC][PATCH -mm] PM: add /sys/power documentation to Documentation/ABI
Date: Wed, 30 Aug 2006 15:32:42 -0700	[thread overview]
Message-ID: <20060830153242.c5081692.akpm@osdl.org> (raw)
In-Reply-To: <20060830150358.55233204.rdunlap@xenotime.net>

On Wed, 30 Aug 2006 15:03:58 -0700
"Randy.Dunlap" <rdunlap@xenotime.net> wrote:

> On Wed, 30 Aug 2006 14:43:50 -0700 Andrew Morton wrote:
> 
> > On Wed, 30 Aug 2006 23:38:06 +0200
> > "Rafael J. Wysocki" <rjw@sisk.pl> wrote:
> > 
> > > Documentation/ABI/testing
> > 
> > psst, Greg, they're still sending too much stuff: need more paperwork!
> > 
> > Some words about this in Documentation/SubmitChecklist would be nice.
> 
> and people actually using it would be nice(r).

c'mon, it must be taped to 1,000 monitors and bathroom doors by now?

> Is this close to what you are looking for?

I guess so, thanks.  Pointing at the README was sneaky.

> 
> From: Randy Dunlap <rdunlap@xenotime.net>
> 
> Mention Documenation/ABI/ requirements in Documentation/SubmitChecklist.
> 
> Signed-off-by: Randy Dunlap <rdunlap@xenotime.net>
> ---
>  Documentation/SubmitChecklist |    3 +++
>  1 files changed, 3 insertions(+)
> 
> --- linux-2618-rc5all.orig/Documentation/SubmitChecklist
> +++ linux-2618-rc5all/Documentation/SubmitChecklist
> @@ -61,3 +61,6 @@ kernel patches.
>      Documentation/kernel-parameters.txt.
>  
>  18: All new module parameters are documented with MODULE_PARM_DESC()
> +
> +19: All new userspace interfaces are documented in Documentation/ABI/.
> +    See Documentation/ABI/README for more information.

This ABI/ thing rather snuck under my radar (I saw it go past, but a lot of
things go past).

It'll be good if it works, but it is going to take quite a lot of thought,
effort and maintainer vigilance to be successful and to avoid becoming
rotware.

I wonder how hard it would be to write a script which parses a diff, works
out if it touches ABI things, complain if it doesn't alter
Documentation/ABI/*?  Not very - it's just a matter of defining a suitable
regexp.

What _should_ be documented in there, anyway?

- syscalls, obviously.

- /proc?  If so, everything, or are there exceptions?

- /sys?  If so, everything, or are there exceptions?

- ioctl numbers and payloads?

- netlink messages?

- ethtool thingies?  netdev interface names?  /proc/iomem identifiers? 
  module names?  kernel-thread comm[] contents?  The ABI is pretty fat.

scary.

  reply	other threads:[~2006-08-30 22:32 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-30 21:38 [RFC][PATCH -mm] PM: add /sys/power documentation to Documentation/ABI Rafael J. Wysocki
2006-08-30 21:35 ` Pavel Machek
2006-08-30 21:43 ` Andrew Morton
2006-08-30 22:03   ` Randy.Dunlap
2006-08-30 22:32     ` Andrew Morton [this message]
2006-08-30 22:51       ` Greg KH
2006-08-30 22:04   ` Pavel Machek

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=20060830153242.c5081692.akpm@osdl.org \
    --to=akpm@osdl.org \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@ucw.cz \
    --cc=rdunlap@xenotime.net \
    --cc=rjw@sisk.pl \
    /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