From: Jonathan Corbet <corbet@lwn.net>
To: "Darrick J. Wong" <darrick.wong@oracle.com>
Cc: david@fromorbit.com, linux-xfs@vger.kernel.org
Subject: Re: [RFC PATCH 00/24] xfs-4.20: major documentation surgery
Date: Thu, 27 Sep 2018 08:50:05 -0600 [thread overview]
Message-ID: <20180927085005.21f7e34a@lwn.net> (raw)
In-Reply-To: <153799045443.31202.17537455000771265705.stgit@magnolia>
On Wed, 26 Sep 2018 12:34:14 -0700
"Darrick J. Wong" <darrick.wong@oracle.com> wrote:
> This series converts the existing in-kernel xfs documentation to rst
> format, links it in with the rest of the kernel's rst documetation, and
> then begins pulling in the contents of the Data Structures & Algorithms
> book from the xfs-documentation git tree. No changes are made to the
> text during the import process except to fix things that the conversion
> process (asciidoctor + pandoc) didn't do correctly. The goal of this
> series is to tie together the XFS code with the on-disk format
> documentation for the features supported by the code.
Some overall comments:
- I certainly approve of improving the documentation and bringing it into
the docs tree - even if you don't CC the docs maintainer :)
- When people do this work, I often end up asking them to think about who
the audience is for the documentation. Developers tend to want to
group all of their docs together, but readers - the people the docs are
for - tend to have different ideas.
So, for example, the xfs.txt document converted in part 1 really, IMO,
belongs in the admin guide - it's information for system
administrators. The data structures and algorithms stuff, instead, is
aimed at developers. I would really argue for separating the two.
It's more work, but it's friendlier to our readers in the long term,
and moves us away from our current pile of unorganized text.
- CC-SA is a great license for documentation; I would rather use it for
all kernel docs. But I think we head into dangerous territory if we
introduce non-GPL-compatible licenses into the kernel documentation.
The docs pull a lot of text from the code itself, to the point that it's
really hard to say that the processed docs are not a derived product of
the kernel code itself. Do we really want to create a situation where
the output of "make *docs" can't be legally distributed?
Hmm...it looks like we have exactly one document asserting CC-SA now,
slipped in by Willy for 4.16. If I'd noticed it, I would have
complained at the time; maybe I'll do so now. In any case, I think we
need to be careful about adding more.
Thanks,
jon
next prev parent reply other threads:[~2018-09-27 21:08 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-26 19:34 [RFC PATCH 00/24] xfs-4.20: major documentation surgery Darrick J. Wong
2018-09-26 19:34 ` [PATCH 01/24] docs: convert xfs.txt to xfs.rst Darrick J. Wong
2018-09-26 19:34 ` [PATCH 02/24] docs: carry a copy of the CC-by-SA 3.0 license Darrick J. Wong
2018-09-26 21:30 ` Christoph Hellwig
2018-09-26 23:16 ` Darrick J. Wong
2018-09-26 23:21 ` Dave Chinner
2018-09-26 23:41 ` Darrick J. Wong
2018-09-26 19:34 ` [PATCH 03/24] docs: add skeleton of XFS Data Structures and Algorithms book Darrick J. Wong
2018-09-26 19:34 ` [PATCH 05/24] docs: add XFS self-describing metadata integrity doc to DS&A book Darrick J. Wong
2018-09-26 19:34 ` [PATCH 06/24] docs: add XFS delayed logging design " Darrick J. Wong
2018-09-26 19:35 ` [PATCH 07/24] docs: add XFS shared data block chapter " Darrick J. Wong
2018-09-26 19:35 ` [PATCH 08/24] docs: add XFS online repair " Darrick J. Wong
2018-09-26 19:35 ` [PATCH 09/24] docs: add XFS common types and magic numbers " Darrick J. Wong
2018-09-26 19:35 ` [PATCH 10/24] docs: add XFS testing chapter to the " Darrick J. Wong
2018-09-26 19:35 ` [PATCH 11/24] docs: add XFS btrees " Darrick J. Wong
2018-09-26 19:35 ` [PATCH 12/24] docs: add XFS dir/attr btree structure " Darrick J. Wong
2018-09-26 19:35 ` [PATCH 13/24] docs: add XFS allocation group metadata " Darrick J. Wong
2018-09-26 19:35 ` [PATCH 14/24] docs: add XFS reverse mapping structures " Darrick J. Wong
2018-09-26 19:35 ` [PATCH 15/24] docs: add XFS refcount btree structure to " Darrick J. Wong
2018-09-26 19:36 ` [PATCH 16/24] docs: add XFS log to the " Darrick J. Wong
2018-09-26 19:36 ` [PATCH 17/24] docs: add XFS internal inodes " Darrick J. Wong
2018-09-26 19:36 ` [PATCH 18/24] docs: add preliminary XFS realtime rmapbt structures " Darrick J. Wong
2018-09-26 19:36 ` [PATCH 19/24] docs: add XFS inode format " Darrick J. Wong
2018-09-26 19:36 ` [PATCH 20/24] docs: add XFS data extent map doc " Darrick J. Wong
2018-09-26 19:36 ` [PATCH 21/24] docs: add XFS directory structure " Darrick J. Wong
2018-09-26 19:36 ` [PATCH 22/24] docs: add XFS extended attributes structures " Darrick J. Wong
2018-09-26 19:37 ` [PATCH 23/24] docs: add XFS symlink " Darrick J. Wong
2018-09-26 19:37 ` [PATCH 24/24] docs: add XFS metadump structure to " Darrick J. Wong
2018-09-27 9:09 ` [RFC PATCH 00/24] xfs-4.20: major documentation surgery Carlos Maiolino
2018-09-27 14:50 ` Jonathan Corbet [this message]
2018-09-27 17:32 ` Darrick J. Wong
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=20180927085005.21f7e34a@lwn.net \
--to=corbet@lwn.net \
--cc=darrick.wong@oracle.com \
--cc=david@fromorbit.com \
--cc=linux-xfs@vger.kernel.org \
/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;
as well as URLs for NNTP newsgroup(s).