linux-xfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Darrick J. Wong" <darrick.wong@oracle.com>
To: Jonathan Corbet <corbet@lwn.net>
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 10:32:41 -0700	[thread overview]
Message-ID: <20180927173241.GB20086@magnolia> (raw)
In-Reply-To: <20180927085005.21f7e34a@lwn.net>

On Thu, Sep 27, 2018 at 08:50:05AM -0600, Jonathan Corbet wrote:
> 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 :)

Oops.  Sorry about that. :/

>  - 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.

If the xfs.txt file moves to the administration guide, how difficult is
it to have it link to the data structures & algorithms book, and vice
versa?

There's probably more administrative stuff from the old SGI guides that
could be cleaned up and added to the administration guide, but that's
something for later.

>  - 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?

Well... as discussed in a different part of the thread we can move the
documentation to CC-BY-SA 4.0 trivially, since the old docs were
CC-BY-SA 3.0+.

Relicensing the DS&A book to GPLv2 would be a /much/ bigger lift, one
which would require figuring out exactly which pieces of SGI have gone
where since 2006 and asking those organization(s) to allow relicensing.
I think that's more work than I'm willing to put in on this, since the
alternative would be to keep xfs-documentation.git running.

To address 'pull[ing] a lot of text from the code itself', AFAICT the
code snippets can be classified into three(?) categories:

 1. Structure definitions that were put in the book by someone at SGI
    which roughly speaking resemble what's in the kernel now.  They're
    not directly pulled from fs/xfs/; rather, someone at SGI put them
    in the book and they've been updated to match the kernel irregularly
    and not necessarily concurrently with feature development.

 2. Structure definitions and code put in the book by me; I'm fine with
    licensing those pieces GPLv2+ or CC-BY-SA 3.0+.

 3. Structure definitions and code put in the book by Dave Chinner (and
    anyone else).

Note that the only /code/ in the DS&A book were put there by me
(directory hash function) and Dave (sample metadata verifier functions),
and it probably isn't difficult to get either of us to agree to
relicense.

So... do separate structure definitions that just happen to have the
same layout as the kernel code create a derived product which would
prohibit us from simply having CC-SA 4.0 licensed documentation in the
kernel?

>    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.

<nod>

--D

> Thanks,
> 
> jon

      reply	other threads:[~2018-09-27 23:52 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
2018-09-27 17:32   ` Darrick J. Wong [this message]

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=20180927173241.GB20086@magnolia \
    --to=darrick.wong@oracle.com \
    --cc=corbet@lwn.net \
    --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).