Re: [PATCH v3] Documentation: gfs2: Consolidate GFS2 docs into its own subdirectory

[Bagas Sanjaya <bagasdotme@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1901 bytes --]

On Tue, Sep 09, 2025 at 01:48:56PM -0600, Jonathan Corbet wrote:
> Bagas Sanjaya <bagasdotme@gmail.com> writes:
> 
> > Documentation for GFS2 is scattered in three docs that are in
> > Documentation/filesystems/ directory. As these docs are standing out as
> > a group, move them into separate gfs2/ subdirectory.
> >
> > Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
> > Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> > ---
> > Changes since v2 [1]:
> >
> >   * Do not use wildcard on MAINTAINERS file entry and add review tag (Randy)
> >
> > [1]: https://lore.kernel.org/linux-doc/20250827022122.12132-1-bagasdotme@gmail.com/
> >
> >  .../filesystems/{gfs2-glocks.rst => gfs2/glocks.rst} |  0
> >  Documentation/filesystems/gfs2/index.rst             | 12 ++++++++++++
> >  .../filesystems/{gfs2.rst => gfs2/overview.rst}      |  6 +++---
> >  .../{gfs2-uevents.rst => gfs2/uevents.rst}           |  0
> >  Documentation/filesystems/index.rst                  |  4 +---
> >  MAINTAINERS                                          |  2 +-
> >  6 files changed, 17 insertions(+), 7 deletions(-)
> >  rename Documentation/filesystems/{gfs2-glocks.rst => gfs2/glocks.rst} (100%)
> >  create mode 100644 Documentation/filesystems/gfs2/index.rst
> >  rename Documentation/filesystems/{gfs2.rst => gfs2/overview.rst} (96%)
> >  rename Documentation/filesystems/{gfs2-uevents.rst => gfs2/uevents.rst} (100%)
> 
> I'm not sure which tree this is meant to go through ...

There's linux-gfs2 tree (see MAINTAINERS)

> 
> I do have one suggestion: the changes seem good as far as I go, but I
> would have made gfs2.rst into the new index.rst and avoided creating an
> essentially no-content file there.

As gfs2 docs currently only has not many (only three), I'm fine with your
suggestion.

Thanks.

-- 
An old man doll... just what I always wanted! - Clara

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]