From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id CEE37CA90AF for ; Wed, 13 May 2020 11:25:22 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id ABE2F2078C for ; Wed, 13 May 2020 11:25:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729299AbgEMLZV (ORCPT ); Wed, 13 May 2020 07:25:21 -0400 Received: from mx2.suse.de ([195.135.220.15]:37952 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726020AbgEMLZV (ORCPT ); Wed, 13 May 2020 07:25:21 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 3A21AAF36; Wed, 13 May 2020 11:25:22 +0000 (UTC) Received: by quack2.suse.cz (Postfix, from userid 1000) id BD4A51E12AE; Wed, 13 May 2020 13:25:18 +0200 (CEST) Date: Wed, 13 May 2020 13:25:18 +0200 From: Jan Kara To: ira.weiny@intel.com Cc: linux-ext4@vger.kernel.org, Andreas Dilger , "Theodore Y. Ts'o" , Jan Kara , Al Viro , Dan Williams , Dave Chinner , Christoph Hellwig , Jeff Moyer , "Darrick J. Wong" , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 4/9] fs/ext4: Change EXT4_MOUNT_DAX to EXT4_MOUNT_DAX_ALWAYS Message-ID: <20200513112518.GD27709@quack2.suse.cz> References: <20200513054324.2138483-1-ira.weiny@intel.com> <20200513054324.2138483-5-ira.weiny@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200513054324.2138483-5-ira.weiny@intel.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue 12-05-20 22:43:19, ira.weiny@intel.com wrote: > From: Ira Weiny > > In prep for the new tri-state mount option which then introduces > EXT4_MOUNT_DAX_NEVER. > > Signed-off-by: Ira Weiny Looks good to me. You can add: Reviewed-by: Jan Kara Honza > > --- > Changes: > New patch > --- > fs/ext4/ext4.h | 4 ++-- > fs/ext4/inode.c | 2 +- > fs/ext4/super.c | 12 ++++++------ > 3 files changed, 9 insertions(+), 9 deletions(-) > > diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h > index 91eb4381cae5..1a3daf2d18ef 100644 > --- a/fs/ext4/ext4.h > +++ b/fs/ext4/ext4.h > @@ -1123,9 +1123,9 @@ struct ext4_inode_info { > #define EXT4_MOUNT_MINIX_DF 0x00080 /* Mimics the Minix statfs */ > #define EXT4_MOUNT_NOLOAD 0x00100 /* Don't use existing journal*/ > #ifdef CONFIG_FS_DAX > -#define EXT4_MOUNT_DAX 0x00200 /* Direct Access */ > +#define EXT4_MOUNT_DAX_ALWAYS 0x00200 /* Direct Access */ > #else > -#define EXT4_MOUNT_DAX 0 > +#define EXT4_MOUNT_DAX_ALWAYS 0 > #endif > #define EXT4_MOUNT_DATA_FLAGS 0x00C00 /* Mode for data writes: */ > #define EXT4_MOUNT_JOURNAL_DATA 0x00400 /* Write data to journal */ > diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c > index 2a4aae6acdcb..a10ff12194db 100644 > --- a/fs/ext4/inode.c > +++ b/fs/ext4/inode.c > @@ -4400,7 +4400,7 @@ int ext4_get_inode_loc(struct inode *inode, struct ext4_iloc *iloc) > > static bool ext4_should_use_dax(struct inode *inode) > { > - if (!test_opt(inode->i_sb, DAX)) > + if (!test_opt(inode->i_sb, DAX_ALWAYS)) > return false; > if (!S_ISREG(inode->i_mode)) > return false; > diff --git a/fs/ext4/super.c b/fs/ext4/super.c > index 9873ab27e3fa..d0434b513919 100644 > --- a/fs/ext4/super.c > +++ b/fs/ext4/super.c > @@ -1767,7 +1767,7 @@ static const struct mount_opts { > {Opt_min_batch_time, 0, MOPT_GTE0}, > {Opt_inode_readahead_blks, 0, MOPT_GTE0}, > {Opt_init_itable, 0, MOPT_GTE0}, > - {Opt_dax, EXT4_MOUNT_DAX, MOPT_SET}, > + {Opt_dax, EXT4_MOUNT_DAX_ALWAYS, MOPT_SET}, > {Opt_stripe, 0, MOPT_GTE0}, > {Opt_resuid, 0, MOPT_GTE0}, > {Opt_resgid, 0, MOPT_GTE0}, > @@ -3974,7 +3974,7 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent) > "both data=journal and dioread_nolock"); > goto failed_mount; > } > - if (test_opt(sb, DAX)) { > + if (test_opt(sb, DAX_ALWAYS)) { > ext4_msg(sb, KERN_ERR, "can't mount with " > "both data=journal and dax"); > goto failed_mount; > @@ -4084,7 +4084,7 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent) > goto failed_mount; > } > > - if (sbi->s_mount_opt & EXT4_MOUNT_DAX) { > + if (sbi->s_mount_opt & EXT4_MOUNT_DAX_ALWAYS) { > if (ext4_has_feature_inline_data(sb)) { > ext4_msg(sb, KERN_ERR, "Cannot use DAX on a filesystem" > " that may contain inline data"); > @@ -5404,7 +5404,7 @@ static int ext4_remount(struct super_block *sb, int *flags, char *data) > err = -EINVAL; > goto restore_opts; > } > - if (test_opt(sb, DAX)) { > + if (test_opt(sb, DAX_ALWAYS)) { > ext4_msg(sb, KERN_ERR, "can't mount with " > "both data=journal and dax"); > err = -EINVAL; > @@ -5425,10 +5425,10 @@ static int ext4_remount(struct super_block *sb, int *flags, char *data) > goto restore_opts; > } > > - if ((sbi->s_mount_opt ^ old_opts.s_mount_opt) & EXT4_MOUNT_DAX) { > + if ((sbi->s_mount_opt ^ old_opts.s_mount_opt) & EXT4_MOUNT_DAX_ALWAYS) { > ext4_msg(sb, KERN_WARNING, "warning: refusing change of " > "dax flag with busy inodes while remounting"); > - sbi->s_mount_opt ^= EXT4_MOUNT_DAX; > + sbi->s_mount_opt ^= EXT4_MOUNT_DAX_ALWAYS; > } > > if (sbi->s_mount_flags & EXT4_MF_FS_ABORTED) > -- > 2.25.1 > -- Jan Kara SUSE Labs, CR