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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20200514111544.GG9569@quack2.suse.cz>
Date:   Thu, 14 May 2020 13:15:44 +0200
From:   Jan Kara <jack@...e.cz>
To:     ira.weiny@...el.com
Cc:     linux-ext4@...r.kernel.org,
        Andreas Dilger <adilger.kernel@...ger.ca>,
        "Theodore Y. Ts'o" <tytso@....edu>, Jan Kara <jack@...e.cz>,
        Al Viro <viro@...iv.linux.org.uk>,
        Dan Williams <dan.j.williams@...el.com>,
        Dave Chinner <david@...morbit.com>,
        Christoph Hellwig <hch@....de>, Jeff Moyer <jmoyer@...hat.com>,
        "Darrick J. Wong" <darrick.wong@...cle.com>,
        linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH V1 9/9] Documentation/dax: Update DAX enablement for ext4

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

Looks good to me. You can add:

Reviewed-by: Jan Kara <jack@...e.cz>

								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@...e.com>
SUSE Labs, CR