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.
next prev parent 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