linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: John Hubbard <jhubbard@nvidia.com>
To: Mike Rapoport <rppt@linux.ibm.com>, <lsf-pc@lists.linux-foundation.org>
Cc: <linux-mm@kvack.org>
Subject: Re: [LSF/MM/BPF TOPIC]: MM documentation
Date: Fri, 21 Feb 2020 18:15:49 -0800	[thread overview]
Message-ID: <9556c040-7e5e-f23f-0137-b167e90ef1ef@nvidia.com> (raw)
In-Reply-To: <20200206165305.GB17499@linux.ibm.com>

On 2/6/20 8:53 AM, Mike Rapoport wrote:
> The mm documentation is, well, not entirely up to date. We can opt for
> dropping the outdated parts, which would generate a nice negative
> diffstat, but identifying the outdated documentation requires nearly
> as much effort as updating it, so I think that making and keeping
> the docs up to date would be a better option.
> 
> I'd like to discuss what can be done process-wise to improve the
> situation.
> 
> Some points I had in mind:
> 
> * Pay more attention to docs during review
> * Set an expectation level for docs accompanying a changeset
> * Spend some cycles to review and update the existing docs
> * Spend some more cycles to add new documentation
> 
> I'd appreciate a discussion about how we can get to the second edition
> of "Understanding the Linux Virtual Memory Manager", what are the gaps
> (although they are too many), and what would be the best way to close
> these gaps.
> 

It's a good topic to have.

You know, I've noticed something interesting about the documentation: the
*.rst documents tend to provide critical insight (even if they're imperfect
for various reaons), and furthermore, the generated HTML from the .rst docs
is very nicely readable (visually) as well.

The kerneldoc documentation is different: it's also got absolutely critical
content, and the native source (in .c/.h files) is also very readable. 
However, the generated HTML is *significantly* less readable than any of the
other 3 items I've listed here. That's mainly because the system picks up 
all kerneldoc items, even the short and less informative ones.

Yes, we can keep trying to make the kerneldoc documentation cleaner. But
it's also reasonable to claim that their are "tiers" of quality, and it's
OK for kerneldoc comments to be at a lower tier than the nice .rst docs.
In fact, we sort of implicitly do so. 

When browsing at the HTML level, I wish there were maybe some slight visual
indicator of these tiers. Because honestly, it seems reasonable for kernel
developers to do something like this:

* If reading at the HTML level, read the "docs" (.rst --> .html docs).

* Once the docs move to the kerneldoc level, you really want the source code
right next to the (often cryptic) kerneldoc documentation. So at that point,
I want out of the HTML, but the HTMl docs are all intermingled.


Maybe it's wrong to want this, but I'm gonna say it anyway. :) 


thanks,
-- 
John Hubbard
NVIDIA


  parent reply	other threads:[~2020-02-22  2:15 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-06 16:53 Mike Rapoport
2020-02-17  1:10 ` Ira Weiny
2020-02-22  2:15 ` John Hubbard [this message]
2022-02-01 19:03 [LSF/MM/BPF TOPIC] mm documentation Mike Rapoport
2023-05-12  1:39 ` Lorenzo Stoakes
2023-05-12 17:37   ` Mike Rapoport
2023-05-13 11:21     ` Lorenzo Stoakes
2023-05-15  6:07       ` Mike Rapoport
2023-05-16 15:10         ` Jonathan Corbet
2023-05-21 23:50           ` David Rientjes
2023-05-22  7:47             ` Lorenzo Stoakes
2023-05-23  8:50             ` Mike Rapoport

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=9556c040-7e5e-f23f-0137-b167e90ef1ef@nvidia.com \
    --to=jhubbard@nvidia.com \
    --cc=linux-mm@kvack.org \
    --cc=lsf-pc@lists.linux-foundation.org \
    --cc=rppt@linux.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