ksummit.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rjw@rjwysocki.net>
To: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Cc: ksummit-discuss@lists.linux-foundation.org
Subject: Re: [Ksummit-discuss] [MAINTAINERS SUMMIT] Documentation issues
Date: Mon, 26 Jun 2017 23:16:34 +0200	[thread overview]
Message-ID: <6778198.2AiHWZiSUU@aspire.rjw.lan> (raw)
In-Reply-To: <20170625221546.49c4b997@vento.lan>

On Sunday, June 25, 2017 10:15:46 PM Mauro Carvalho Chehab wrote:
> Em Sun, 25 Jun 2017 23:42:20 +0200
> "Rafael J. Wysocki" <rjw@rjwysocki.net> escreveu:
> 
> > On Sunday, June 25, 2017 05:32:31 PM Matthew Wilcox wrote:
> > > I find documentation comes in the following bins:
> > > 
> > > 1. User documentation (possible sub-split into "how to use this from kernel
> > > space" and "how to use this from user space")
> > > 2. Information for someone who's interested in modifying the code. Possibly
> > > including architectural considerations (eg locking), performance, ideas for
> > > future improvement, etc.
> > > 3. Random swearing and abuse  
> > 
> > There also is information on various frameworks that driver writers are expected
> > to use and honestly various mixtures of that with 1. above.
> > 
> > Mutual exclusion mechanisms also need to be documented properly IMO and so on.
> > 
> > > I think the second and third categories of documentation should be kept out
> > > of the kernel books and left as plain comments by the code.
> > > 
> > > On 24 Jun 2017 9:41 am, "Mauro Carvalho Chehab" <mchehab@s-opensource.com>
> > > wrote:
> > > 
> > > Yeah, one of the problems with existing documentation is that,
> > > sometimes, the same document describes both userspace-relevant
> > > info and kernelspace APIs.  
> > 
> > Right.
> > 
> > But, alas, those things happen to be related, especially when framework-provided
> > sysfs attributes and similar come into play.  With the current layout of things
> > it sometimes is hard to avoid documenting the same thing in two different
> > places, risking that the two descriptions of the same thing will diverge in the
> > future eventually.
> > 
> 
> Well, sysfs attribute description is currently a little messy, IMHO.
> We have (or should have) all of them described under Documentation/ABI/,
> but it is not uncommon to have them described on some other random
> places. As things get bitrot, we end by having different descriptions
> on each place.
> 
> I also suspect that some sysfs descriptions are only outside
> Documentation/ABI/ (although I never actually tried to seek for such
> issue).

That would be my guess too, although I don't remember any specific
examples readily.

My point is that some frameworks expose things via sysfs and drivers are
expected to provide callbacks invoked from there or similar.  The information
on what is expected to be provided falls under the "driver API", but it is
strictly related to the "admin guide" part as well.

> > Moreover, does debugfs fall into "user documentation" or "developer documentation",
> > for example?
> > 
> > And generally documentation on how to diagnose problems for that matter?
> 
> The same thing for sysfs applies to debugfs.
> 
> IMHO, debugfs and information about how to diagnose problems
> belong to "user documentation" / "how to use this from user space"
> (using Matthew's classification), in the sense that an advanced
> admin may use them, in order to properly address some issue that
> will eventually generate a bug report (either for Kernel or to some
> application/library).

It also is part of driver developer's toolkit, however, like unit tests and such.

Thanks,
Rafael

  reply	other threads:[~2017-06-26 21:16 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-23 18:39 Jonathan Corbet
2017-06-23 23:09 ` Rafael J. Wysocki
2017-06-24 12:03   ` Rafael J. Wysocki
2017-06-24 10:46 ` Mauro Carvalho Chehab
2017-06-24 12:20   ` Rafael J. Wysocki
2017-06-24 13:41     ` Mauro Carvalho Chehab
2017-06-25 20:56       ` Jiri Kosina
2017-06-26  1:20         ` Mauro Carvalho Chehab
2017-06-26 21:18           ` Rafael J. Wysocki
2017-06-26  5:58         ` Takashi Iwai
2017-06-26 21:28           ` Rafael J. Wysocki
2017-06-27  8:41             ` Daniel Vetter
2017-06-27 10:31               ` Mauro Carvalho Chehab
2017-06-26 22:18         ` Jonathan Corbet
2017-06-27 18:42           ` Bird, Timothy
2017-06-28 19:48           ` Geert Uytterhoeven
     [not found]       ` <CAFhKne8ZttmqWYJToCw9EDywzeMdp=eiFpoZ+O5xrzmKnpA09Q@mail.gmail.com>
     [not found]         ` <CAFhKne_W-EbjUd_Cm4kyBHrVK6K9r8Ss3gY0ogO1nztbQZYBEg@mail.gmail.com>
2017-06-25 21:32           ` Matthew Wilcox
2017-06-25 21:42             ` Rafael J. Wysocki
2017-06-26  1:15               ` Mauro Carvalho Chehab
2017-06-26 21:16                 ` Rafael J. Wysocki [this message]
2017-06-26  0:58             ` Mauro Carvalho Chehab
2017-06-25 16:13     ` Jonathan Corbet
2017-06-26 12:19 ` Mark Brown
2017-06-27  8:38   ` Daniel Vetter
2017-06-27 15:33     ` Mark Brown

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=6778198.2AiHWZiSUU@aspire.rjw.lan \
    --to=rjw@rjwysocki.net \
    --cc=ksummit-discuss@lists.linux-foundation.org \
    --cc=mchehab@s-opensource.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