Re: [Cluster-devel] [PATCH v7 08/13] fs: drop the timespec64 argument from update_time
Mike Marshall <[email protected]> Wed, 9 Aug 2023 14:38:07 -0400
| Newsgroups | com.redhat.cluster-devel,dev.linux.lists.ntfs3,dev.linux.lists.ocfs2-devel,dev.linux.lists.v9fs,net.sourceforge.lists.linux-f2fs-devel,org.infradead.lists.linux-mtd,org.kernel.vger.ceph-devel,org.kernel.vger.ecryptfs,org.kernel.vger.linux-btrfs,org.kernel.vger.linux-cifs,org.kernel.vger.linux-ext4,org.kernel.vger.linux-fsdevel,org.kernel.vger.linux-kernel,org.kernel.vger.linux-nfs,org.kernel.vger.linux-unionfs,org.kernel.vger.linux-xfs,org.kvack.linux-mm,org.ozlabs.lists.linux-erofs |
|---|---|
| Message-ID | <CAOg9mST=WFAjEwS9eNi_huoUpBvPy3R3fbFVTLUeFZAv6BJEEQ@mail.gmail.com> |
I've been following this patch on fsdevel... is there a remote I could fetch with a branch that has this in it? -Mike On Wed, Aug 9, 2023 at 8:32 AM Christian Brauner <[email protected]> wrote: > > On Mon, Aug 07, 2023 at 03:38:39PM -0400, Jeff Layton wrote: > > Now that all of the update_time operations are prepared for it, we can > > drop the timespec64 argument from the update_time operation. Do that and > > remove it from some associated functions like inode_update_time and > > inode_needs_update_time. > > > > Signed-off-by: Jeff Layton <[email protected]> > > --- > > fs/bad_inode.c | 3 +-- > > fs/btrfs/inode.c | 3 +-- > > fs/btrfs/volumes.c | 4 +--- > > fs/fat/fat.h | 3 +-- > > fs/fat/misc.c | 2 +- > > fs/gfs2/inode.c | 3 +-- > > fs/inode.c | 30 +++++++++++++----------------- > > fs/overlayfs/inode.c | 2 +- > > fs/overlayfs/overlayfs.h | 2 +- > > fs/ubifs/file.c | 3 +-- > > fs/ubifs/ubifs.h | 2 +- > > fs/xfs/xfs_iops.c | 1 - > > include/linux/fs.h | 4 ++-- > > This was missing the conversion of fs/orangefs orangefs_update_time() > causing the build to fail. So at some point kbuild will yell here. > Fwiw, I've fixed that up in-tree.