Re: [PATCH V1 9/9] Documentation/dax: Update DAX enablement for ext4

From: Jan Kara
Date: Thu May 14 2020 - 07:15:48 EST


On Wed 13-05-20 23:53:15, ira.weiny@xxxxxxxxx wrote:
> From: Ira Weiny <ira.weiny@xxxxxxxxx>
>
> Update the document to reflect ext4 and xfs now behave the same.
>
> Signed-off-by: Ira Weiny <ira.weiny@xxxxxxxxx>

Looks good to me. You can add:

Reviewed-by: Jan Kara <jack@xxxxxxx>

Honza

>
> ---
> Changes from RFC:
> Update with ext2 text...
> ---
> Documentation/filesystems/dax.txt | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/filesystems/dax.txt b/Documentation/filesystems/dax.txt
> index 735fb4b54117..265c4f808dbf 100644
> --- a/Documentation/filesystems/dax.txt
> +++ b/Documentation/filesystems/dax.txt
> @@ -25,7 +25,7 @@ size when creating the filesystem.
> Currently 3 filesystems support DAX: ext2, ext4 and xfs. Enabling DAX on them
> is different.
>
> -Enabling DAX on ext4 and ext2
> +Enabling DAX on ext2
> -----------------------------
>
> When mounting the filesystem, use the "-o dax" option on the command line or
> @@ -33,8 +33,8 @@ add 'dax' to the options in /etc/fstab. This works to enable DAX on all files
> within the filesystem. It is equivalent to the '-o dax=always' behavior below.
>
>
> -Enabling DAX on xfs
> --------------------
> +Enabling DAX on xfs and ext4
> +----------------------------
>
> Summary
> -------
> --
> 2.25.1
>
--
Jan Kara <jack@xxxxxxxx>
SUSE Labs, CR