ksummit.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
From: "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com>
To: Mark Brown <broonie@kernel.org>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: ksummit-discuss@lists.linuxfoundation.org
Subject: Re: [Ksummit-discuss] [CORE TOPIC] Documentation
Date: Wed, 05 Aug 2015 19:07:41 +0200	[thread overview]
Message-ID: <55C242DD.6020304@gmail.com> (raw)
In-Reply-To: <20150804153559.GO20873@sirena.org.uk>

On 08/04/2015 05:35 PM, Mark Brown wrote:
> On Tue, Aug 04, 2015 at 03:50:33PM +0300, Laurent Pinchart wrote:
>> On Monday 03 August 2015 08:33:11 Jonathan Corbet wrote:
> 
>>> The nice thing about formats like Markdown or ReST is that they *are*
>>> plain text for all practical purposes.  Much better than DocBook in that
>>> regard.
> 
>> My problem with inline documentation is that it makes easier for developers to 
>> write crappy doc and believe they've done their duty. It's not a technical 
>> issue, and I believe inline documentation has merits, especially for API 
>> documentation, but I've seen too many kerneldoc comments written as
> 
> That's not really anything to do with the fact that the documentation is
> inline, it's more to do with documentation that people are forced to
> write without any obvious interest being shown in the results.  Anything
> else that's done as a pure checkbox effort will have similar effects.
> 
>> That's just useless. Worse, it can give the impression to reviewers that the 
>> function has been documented while it clearly hasn't. Maybe we just need to 
>> tighten the review process and push harder for documentation, at the risk of 
>> rejecting useful contributions. That's not a new problem though.
> 
> I'm not sure that more requirements for documentation will help that
> much, there's a definite skill in writing and review documentation which
> isn't all that common.  Insisting on people writing more without doing
> something to address that may just result in more bad documentation, but
> of course it's a really hard problem to address :(

While it's clear that, like skill at coding, skill at documentation
is wildly variable, almost everyone gets better at it with practice! Like
most problems, the problem would be more tractable if people were paid to
work on this. For a short while, there was an LF documentation fellowship
that had a goal to help things here. It had variable success. For a some
months it supported me (as the second holder of the fellowship) to work
full time on man pages, but there was so much man pages work, that kernel
docs didn't get too much of a look in. And then a few months later
(2008 crash), the funds went away. I raised the topic of reanimating
that fellowship a couple of times subsequently, but nothing came of it. 
That was however quite a few years ago. Given recent initiatives
from LF (such as the Critical Infrastructure Initiative), I wonder
whether it's worth someone checking in with LF about the possibility
of getting this going again. One thing I'd caution against though is
the idea that someone can just be paid to write kernel documentation.
Many parts of the kernel require significant time just to grok what's
going on, let alone document it. A much more efficient route to 
producing good documentation would I think be to have someone
with solid writing skills (as well as coding skills) support others
(i.e., subsystem developers and maintainers) to produce documentation,
by providing writing assistance, technical review, editing, and so on.
Cheers,

Michael

-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/

  reply	other threads:[~2015-08-05 17:07 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-01 14:41 Jonathan Corbet
2015-08-02  7:07 ` Davidlohr Bueso
2015-08-03 13:35   ` Rafael J. Wysocki
2015-08-03 13:27     ` Konstantin Ryabitsev
2015-08-03 14:33     ` Jonathan Corbet
2015-08-03 20:45       ` Dmitry Torokhov
2015-08-04 10:59         ` Daniel Vetter
2015-08-04  0:52       ` Rafael J. Wysocki
2015-08-04 12:50       ` Laurent Pinchart
2015-08-04 13:03         ` Daniel Vetter
2015-08-04 14:28           ` Laurent Pinchart
2015-08-04 14:30             ` Daniel Vetter
2015-08-04 13:50         ` Dan Carpenter
2015-08-04 14:05           ` Geert Uytterhoeven
2015-08-04 14:29             ` Daniel Vetter
2015-08-04 14:30             ` Laurent Pinchart
2015-08-04 17:10               ` Geert Uytterhoeven
2015-08-04 14:42           ` Konstantin Ryabitsev
2015-08-04 18:21             ` Tim Bird
2015-08-04 21:00               ` Laurent Pinchart
2015-08-04 15:35         ` Mark Brown
2015-08-05 17:07           ` Michael Kerrisk (man-pages) [this message]
2015-08-04 17:24         ` Jonathan Corbet
2015-08-04  7:12 ` Michael Ellerman
2015-08-04  7:42   ` Marcel Holtmann
2015-08-04  8:33     ` Peter Huewe
2015-08-05 17:08       ` Michael Kerrisk (man-pages)
2015-08-05 17:19         ` josh
2015-08-05 17:21         ` Konstantin Ryabitsev
2015-08-04 12:54   ` Laurent Pinchart
2015-08-04 13:07     ` Daniel Vetter
2015-08-04 11:09 ` Daniel Vetter

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=55C242DD.6020304@gmail.com \
    --to=mtk.manpages@gmail.com \
    --cc=broonie@kernel.org \
    --cc=ksummit-discuss@lists.linuxfoundation.org \
    --cc=laurent.pinchart@ideasonboard.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