Re: [PATCH 12/34] docs: filesystems: convert dax.txt to ReST

From: Mauro Carvalho Chehab
Date: Tue Apr 21 2020 - 15:53:23 EST


Em Tue, 21 Apr 2020 11:31:21 -0700
"Darrick J. Wong" <darrick.wong@xxxxxxxxxx> escreveu:

> On Tue, Apr 21, 2020 at 06:31:17PM +0200, Mauro Carvalho Chehab wrote:
> > Em Wed, 15 Apr 2020 08:41:44 -0700
> > "Darrick J. Wong" <darrick.wong@xxxxxxxxxx> escreveu:
> >
> > > [add ira weiny to cc]
> > >
> > > On Wed, Apr 15, 2020 at 04:32:25PM +0200, Mauro Carvalho Chehab wrote:
> > > > - Add a SPDX header;
> > > > - Adjust document title;
> > > > - Some whitespace fixes and new line breaks;
> > > > - Add it to filesystems/index.rst.
> > > >
> > > > Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx>
> > > > ---
> > > > Documentation/admin-guide/ext4.rst | 2 +-
> > > > Documentation/filesystems/{dax.txt => dax.rst} | 9 +++++++--
> > > > Documentation/filesystems/ext2.rst | 2 +-
> > > > Documentation/filesystems/index.rst | 1 +
> > > > 4 files changed, 10 insertions(+), 4 deletions(-)
> > > > rename Documentation/filesystems/{dax.txt => dax.rst} (97%)
> > > >
> > > > diff --git a/Documentation/admin-guide/ext4.rst b/Documentation/admin-guide/ext4.rst
> > > > index 9443fcef1876..103bcc345bad 100644
> > > > --- a/Documentation/admin-guide/ext4.rst
> > > > +++ b/Documentation/admin-guide/ext4.rst
> > > > @@ -392,7 +392,7 @@ When mounting an ext4 filesystem, the following option are accepted:
> > > >
> > > > dax
> > > > Use direct access (no page cache). See
> > > > - Documentation/filesystems/dax.txt. Note that this option is
> > > > + Documentation/filesystems/dax.rst. Note that this option is
> > > > incompatible with data=journal.
> > > >
> > > > Data Mode
> > > > diff --git a/Documentation/filesystems/dax.txt b/Documentation/filesystems/dax.rst
> > > > similarity index 97%
> > > > rename from Documentation/filesystems/dax.txt
> > > > rename to Documentation/filesystems/dax.rst
> > > > index 735f3859b19f..5838144f80f0 100644
> > > > --- a/Documentation/filesystems/dax.txt
> > > > +++ b/Documentation/filesystems/dax.rst
> > >
> > > Err, this will collide with the work that Ira's doing on DAX for 5.8[1].
> > > Can the dax.txt conversion wait?
> >
> > Well, I can re-schedule it to 5.9. Or, if you merge the dax changes
> > at linux-next, I can rebase my patch on the top of it.
>
> That depends on how quick Ira can get the series merged. :)
>
> I personally think (hope) everyone's tired of arguing and we can just
> get it done for 5.8, but past experience tells me that rescheduling the
> rst conversion to 5.90 is at least a safer bet.

I can't tell much about Ira's patches, but, in the case of the ReST
conversion, there are not much left to convert anymore. The vast majority
of files under Documentation/*.txt are already in ReST format. They just
need to be renamed/moved to a better place. Besides that, there are
the fs patches on this series, network and a few misc text files.

I have patches for the remaining stuff already.

>
> --D
>
> > >
> > > --D
> > >
> > > [1] https://lore.kernel.org/linux-xfs/20200415152942.GS6742@magnolia/T/#m804562299416d865d8829caa82589a522b2080a5
> > >
> > > > @@ -1,5 +1,8 @@
> > > > +.. SPDX-License-Identifier: GPL-2.0
> > > > +
> > > > +=======================
> > > > Direct Access for files
> > > > ------------------------
> > > > +=======================
> > > >
> > > > Motivation
> > > > ----------
> > > > @@ -46,6 +49,7 @@ stall the CPU for an extended period, you should also not attempt to
> > > > implement direct_access.
> > > >
> > > > These block devices may be used for inspiration:
> > > > +
> > > > - brd: RAM backed block device driver
> > > > - dcssblk: s390 dcss block device driver
> > > > - pmem: NVDIMM persistent memory driver
> > > > @@ -55,6 +59,7 @@ Implementation Tips for Filesystem Writers
> > > > ------------------------------------------
> > > >
> > > > Filesystem support consists of
> > > > +
> > > > - adding support to mark inodes as being DAX by setting the S_DAX flag in
> > > > i_flags
> > > > - implementing ->read_iter and ->write_iter operations which use dax_iomap_rw()
> > > > @@ -127,6 +132,6 @@ by adding optional struct page support for pages under the control of
> > > > the driver (see CONFIG_NVDIMM_PFN in drivers/nvdimm for an example of
> > > > how to do this). In the non struct page cases O_DIRECT reads/writes to
> > > > those memory ranges from a non-DAX file will fail (note that O_DIRECT
> > > > -reads/writes _of a DAX file_ do work, it is the memory that is being
> > > > +reads/writes _of a DAX ``file_`` do work, it is the memory that is being
> > > > accessed that is key here). Other things that will not work in the
> > > > non struct page case include RDMA, sendfile() and splice().
> > > > diff --git a/Documentation/filesystems/ext2.rst b/Documentation/filesystems/ext2.rst
> > > > index d83dbbb162e2..fa416b7a5802 100644
> > > > --- a/Documentation/filesystems/ext2.rst
> > > > +++ b/Documentation/filesystems/ext2.rst
> > > > @@ -24,7 +24,7 @@ check=none, nocheck (*) Don't do extra checking of bitmaps on mount
> > > > (check=normal and check=strict options removed)
> > > >
> > > > dax Use direct access (no page cache). See
> > > > - Documentation/filesystems/dax.txt.
> > > > + Documentation/filesystems/dax.rst.
> > > >
> > > > debug Extra debugging information is sent to the
> > > > kernel syslog. Useful for developers.
> > > > diff --git a/Documentation/filesystems/index.rst b/Documentation/filesystems/index.rst
> > > > index c4f95f76ba6a..8e3ccb4ed483 100644
> > > > --- a/Documentation/filesystems/index.rst
> > > > +++ b/Documentation/filesystems/index.rst
> > > > @@ -24,6 +24,7 @@ algorithms work.
> > > > splice
> > > > locking
> > > > directory-locking
> > > > + dax
> > > >
> > > > automount-support
> > > >
> > > > --
> > > > 2.25.2
> > > >
> >
> >
> >
> > Thanks,
> > Mauro



Thanks,
Mauro