From patchwork Wed Jul 5 19:00:50 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jeff Layton X-Patchwork-Id: 13302758 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id D77AEEB64DD for ; Wed, 5 Jul 2023 19:03:51 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 731578D0006; Wed, 5 Jul 2023 15:03:51 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 6BA648D0001; Wed, 5 Jul 2023 15:03:51 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 534B38D0006; Wed, 5 Jul 2023 15:03:51 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 43B1D8D0001 for ; Wed, 5 Jul 2023 15:03:51 -0400 (EDT) Received: from smtpin07.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id 08922AFF4F for ; Wed, 5 Jul 2023 19:03:51 +0000 (UTC) X-FDA: 80978482662.07.D737E93 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf14.hostedemail.com (Postfix) with ESMTP id B4C73100028 for ; Wed, 5 Jul 2023 19:03:48 +0000 (UTC) Authentication-Results: imf14.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b="Djy35r/J"; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf14.hostedemail.com: domain of jlayton@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=jlayton@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1688583828; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=W1LJ1Ev958+v2cYpgmJ52/5bNelskmJS3SGOJ4WTHtA=; b=SfImQRW16hDB5qT9Mggx26AAaRh5bKIqvtgWbWFlWujZnTOtHep0SuiT8s79wJQtUHxwKW hAf/64/M2FKjsYTzLTxFflFhCxzsC0HUm7JruAC06RBX4ytJXkbP1ntKY2Mtw1fFWn5D6B HZdEol+LR8v2B2fOUYzDwx3GRRMPJwY= ARC-Authentication-Results: i=1; imf14.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b="Djy35r/J"; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf14.hostedemail.com: domain of jlayton@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=jlayton@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1688583828; a=rsa-sha256; cv=none; b=F4+wPeKXNBmh3OXpKkXdDBhDZHlq49qpDmOeBHo+e+0zX/glbm/SAFPg3R5lrH45UvC7Dm SVzo7LPRlCigy0YHwyUJd/yUrmSw8o3hawfdSbLVqKECVrUtViWgGs032dkXGl/QPHg9DX WeTucL3DZ4faI95oelHdK00Kq59MmMU= Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id DEC99616FB; Wed, 5 Jul 2023 19:03:47 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8C394C433C8; Wed, 5 Jul 2023 19:03:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1688583827; bh=7amT9DrYu70ALB4f/8ioI9ChjNtEorP9RC8mHDJNfS0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=Djy35r/JI60Pfe0ZiI+Z9tvfNL61rTokHFk5MxL+luQelNZiTKVK6XlUnKTqVOt7h 10wc/pvErEgXyHf2RIUkXE4zE0UH3TK539DVcK0STS8UwkLmUkQlS66S3CcESJUIjC AC8Hr7c/iVK6xSwF3K4jPkjuBHYKz6BBOczAdB0F23f+G6uCV10RTaUgr2OaFK+Esr snFQPsZmnj2p9zcyrG0UVBJOmgvGgCgXp07gJ6a38nxOzeW5Fo4L/+ar/BjgcCow6L wwYrrso5LR4MSzSjySjM4N3yXTZyJ2qAk7DRfIiMn2vE3HwCBbk1wo4IFo95+y6M58 6XAVG3Zw534ow== From: Jeff Layton To: Christian Brauner , Alexander Viro , Eric Biederman , Kees Cook Cc: Jan Kara , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: [PATCH v2 25/92] fs: convert to ctime accessor functions Date: Wed, 5 Jul 2023 15:00:50 -0400 Message-ID: <20230705190309.579783-23-jlayton@kernel.org> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20230705190309.579783-1-jlayton@kernel.org> References: <20230705185755.579053-1-jlayton@kernel.org> <20230705190309.579783-1-jlayton@kernel.org> MIME-Version: 1.0 X-Rspam-User: X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: B4C73100028 X-Stat-Signature: 7145dotcu75s4z7ctteob9fyfmzp4p9g X-HE-Tag: 1688583828-497941 X-HE-Meta: U2FsdGVkX1/Req4o/PL6S/jL13R87JwbLuPduo4zM+/GYD8KvWGud85nScrNn8ZUYlHtzqCd6M62IwG9CEnb7YcxneDxMoVsLFn/KZT5WydRU9ZeTTd4fzf6J6kiZwxFjZVScWZHTIFp0VMNJVmGjd7oIXWVZzEuyK3Dqx+axIIHtRsKRQTN6yrsRfCCwmETXSyw4JZm5XVhBIJd8f9s1R531t2ZkkwJS8yBfb/lK77xMFpSqBWgHsxbcPBg3WbO5nShWV5m0zzTezwrU/NYHK72AcaBEuzkhnItcDFH4XdNmQbN9YX4qvNYFbj+pUuPZ5tu3t0xTEr91SN/C+C7l19AGp9VV/7G2t7xY1eh7BWF8J3W7mFWmsg2aHhN7hXjgbwHbxZOlGuQy9oiWBfNzbcYS1TeGN/pe/qjNe8CJ0hMad2MOnZM1kFFBBaNiQcNxn84tHzHFylAStp5MhhEG3RYmRsH7yUjN85Kj4c8L4gHZubh0y8DMxlthVuRt5s8OKu6buiRbt5NAPIZ1MvO9uEFkUPx3gLaAeUR5O8GugFN/c34RWbyjgAc1XvAPgQpWdb8+PXLmchVHYHljdm8POLsJTRdip9rNHXH+VKj9PL/rHbC0ejA3N6nk/G6U1LSnTmLFRctI4b35gofcc9CZDaET9IRieRuomeDcpzVpjHTch+z9/vsGljzPSKA/VYws/bzppSyNZfit+xEm9EZ7OaM/0cmYU1GW/GzU63TqJ9go1uu3IHv1aMHFQSll735X/PmAl7O3mg4RSX/DYl5IZMBj3Yq3dV49PzzcVVI4cYqPCs7a0V2Z64fbAmMZtpN0/iLHNR1eritwnkfRGrBXQAm/1rOeGvvvD/Wd0gXrndrM7lW9GA2waddbIuLiKvHq7ENTI5pC1NRAY2qIMiq3va5Wlglxne8v11CdkjFaV2FCk0LiVAVqt9RdyA2GNcs/79Ewl4uKGDS7WcC2db ju6Z95AD L+sY4m+Pk6Pa1egtIq7hcPAvOsDouclY/iCuKo7lNPpH0n4j+p+QUUc7VLeELsHKal1KEQW5LjMSlHRbpCPHtobm/N+Do5TF4Dbw2WHERDHleM0+AUgEJS/EkJ5anDQYkDec0QggYFIX4z+2eVNPfSBwkhvUC3Vsns1S3lavZmXdmA8nI10CjWdtBJAjAVTRXknLVojM7iXnPzyOnQx+7L/quxkG5PnHIe5Hri9+h23hwD88DGnbUFUcW7A32LtsCyYft2/pbrWV8qBer2pc2QC8cM9eyvPVksrLdCkkceQoLNqccqigBZyE09InkmkuXn8bEx5fIe4yu3/69DZ5Tp1RvCQ== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: In later patches, we're going to change how the inode's ctime field is used. Switch to using accessor functions instead of raw accesses of inode->i_ctime. Reviewed-by: Jan Kara Signed-off-by: Jeff Layton --- fs/attr.c | 2 +- fs/bad_inode.c | 3 +-- fs/binfmt_misc.c | 3 +-- fs/inode.c | 10 +++++++--- fs/nsfs.c | 2 +- fs/pipe.c | 2 +- fs/posix_acl.c | 2 +- fs/stack.c | 2 +- fs/stat.c | 2 +- 9 files changed, 15 insertions(+), 13 deletions(-) diff --git a/fs/attr.c b/fs/attr.c index d60dc1edb526..599f6d14c0ed 100644 --- a/fs/attr.c +++ b/fs/attr.c @@ -312,7 +312,7 @@ void setattr_copy(struct mnt_idmap *idmap, struct inode *inode, if (ia_valid & ATTR_MTIME) inode->i_mtime = attr->ia_mtime; if (ia_valid & ATTR_CTIME) - inode->i_ctime = attr->ia_ctime; + inode_set_ctime_to_ts(inode, attr->ia_ctime); if (ia_valid & ATTR_MODE) { umode_t mode = attr->ia_mode; if (!in_group_or_capable(idmap, inode, diff --git a/fs/bad_inode.c b/fs/bad_inode.c index db649487d58c..6e21f7412a85 100644 --- a/fs/bad_inode.c +++ b/fs/bad_inode.c @@ -209,8 +209,7 @@ void make_bad_inode(struct inode *inode) remove_inode_hash(inode); inode->i_mode = S_IFREG; - inode->i_atime = inode->i_mtime = inode->i_ctime = - current_time(inode); + inode->i_atime = inode->i_mtime = inode_set_ctime_current(inode); inode->i_op = &bad_inode_ops; inode->i_opflags &= ~IOP_XATTR; inode->i_fop = &bad_file_ops; diff --git a/fs/binfmt_misc.c b/fs/binfmt_misc.c index bb202ad369d5..e0108d17b085 100644 --- a/fs/binfmt_misc.c +++ b/fs/binfmt_misc.c @@ -547,8 +547,7 @@ static struct inode *bm_get_inode(struct super_block *sb, int mode) if (inode) { inode->i_ino = get_next_ino(); inode->i_mode = mode; - inode->i_atime = inode->i_mtime = inode->i_ctime = - current_time(inode); + inode->i_atime = inode->i_mtime = inode_set_ctime_current(inode); } return inode; } diff --git a/fs/inode.c b/fs/inode.c index 21b026d95b51..32e08c3fa9ff 100644 --- a/fs/inode.c +++ b/fs/inode.c @@ -1851,6 +1851,7 @@ EXPORT_SYMBOL(bmap); static int relatime_need_update(struct vfsmount *mnt, struct inode *inode, struct timespec64 now) { + struct timespec64 ctime; if (!(mnt->mnt_flags & MNT_RELATIME)) return 1; @@ -1862,7 +1863,8 @@ static int relatime_need_update(struct vfsmount *mnt, struct inode *inode, /* * Is ctime younger than or equal to atime? If yes, update atime: */ - if (timespec64_compare(&inode->i_ctime, &inode->i_atime) >= 0) + ctime = inode_get_ctime(inode); + if (timespec64_compare(&ctime, &inode->i_atime) >= 0) return 1; /* @@ -1885,7 +1887,7 @@ int generic_update_time(struct inode *inode, struct timespec64 *time, int flags) if (flags & S_ATIME) inode->i_atime = *time; if (flags & S_CTIME) - inode->i_ctime = *time; + inode_set_ctime_to_ts(inode, *time); if (flags & S_MTIME) inode->i_mtime = *time; @@ -2071,6 +2073,7 @@ EXPORT_SYMBOL(file_remove_privs); static int inode_needs_update_time(struct inode *inode, struct timespec64 *now) { int sync_it = 0; + struct timespec64 ctime; /* First try to exhaust all avenues to not sync */ if (IS_NOCMTIME(inode)) @@ -2079,7 +2082,8 @@ static int inode_needs_update_time(struct inode *inode, struct timespec64 *now) if (!timespec64_equal(&inode->i_mtime, now)) sync_it = S_MTIME; - if (!timespec64_equal(&inode->i_ctime, now)) + ctime = inode_get_ctime(inode); + if (!timespec64_equal(&ctime, now)) sync_it |= S_CTIME; if (IS_I_VERSION(inode) && inode_iversion_need_inc(inode)) diff --git a/fs/nsfs.c b/fs/nsfs.c index f602a96a1afe..647a22433bd8 100644 --- a/fs/nsfs.c +++ b/fs/nsfs.c @@ -84,7 +84,7 @@ static int __ns_get_path(struct path *path, struct ns_common *ns) return -ENOMEM; } inode->i_ino = ns->inum; - inode->i_mtime = inode->i_atime = inode->i_ctime = current_time(inode); + inode->i_mtime = inode->i_atime = inode_set_ctime_current(inode); inode->i_flags |= S_IMMUTABLE; inode->i_mode = S_IFREG | S_IRUGO; inode->i_fop = &ns_file_operations; diff --git a/fs/pipe.c b/fs/pipe.c index 2d88f73f585a..174682103669 100644 --- a/fs/pipe.c +++ b/fs/pipe.c @@ -899,7 +899,7 @@ static struct inode * get_pipe_inode(void) inode->i_mode = S_IFIFO | S_IRUSR | S_IWUSR; inode->i_uid = current_fsuid(); inode->i_gid = current_fsgid(); - inode->i_atime = inode->i_mtime = inode->i_ctime = current_time(inode); + inode->i_atime = inode->i_mtime = inode_set_ctime_current(inode); return inode; diff --git a/fs/posix_acl.c b/fs/posix_acl.c index 7fa1b738bbab..a05fe94970ce 100644 --- a/fs/posix_acl.c +++ b/fs/posix_acl.c @@ -1027,7 +1027,7 @@ int simple_set_acl(struct mnt_idmap *idmap, struct dentry *dentry, return error; } - inode->i_ctime = current_time(inode); + inode_set_ctime_current(inode); if (IS_I_VERSION(inode)) inode_inc_iversion(inode); set_cached_acl(inode, type, acl); diff --git a/fs/stack.c b/fs/stack.c index c9830924eb12..b5e01bdb5f5f 100644 --- a/fs/stack.c +++ b/fs/stack.c @@ -68,7 +68,7 @@ void fsstack_copy_attr_all(struct inode *dest, const struct inode *src) dest->i_rdev = src->i_rdev; dest->i_atime = src->i_atime; dest->i_mtime = src->i_mtime; - dest->i_ctime = src->i_ctime; + inode_set_ctime_to_ts(dest, inode_get_ctime(src)); dest->i_blkbits = src->i_blkbits; dest->i_flags = src->i_flags; set_nlink(dest, src->i_nlink); diff --git a/fs/stat.c b/fs/stat.c index 7c238da22ef0..8c2b30af19f5 100644 --- a/fs/stat.c +++ b/fs/stat.c @@ -58,7 +58,7 @@ void generic_fillattr(struct mnt_idmap *idmap, struct inode *inode, stat->size = i_size_read(inode); stat->atime = inode->i_atime; stat->mtime = inode->i_mtime; - stat->ctime = inode->i_ctime; + stat->ctime = inode_get_ctime(inode); stat->blksize = i_blocksize(inode); stat->blocks = inode->i_blocks; }