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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Sep 09, 2025 at 01:48:56PM -0600, Jonathan Corbet wrote:
> Bagas Sanjaya <bagasdotme@xxxxxxxxx> 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@xxxxxxxxxxxxx>
> > Signed-off-by: Bagas Sanjaya <bagasdotme@xxxxxxxxx>
> > ---
> > 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@xxxxxxxxx/
> >
> >  .../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: signature.asc
Description: PGP signature


[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux