Skip to content

RFC vfs 64 bit time transition proposal

Deepa Dinamani linux-fsdevel lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 RFC Latest12 patches 5 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/12vfs: Add vfs_time abstractionsinclude/linux/fs.h+19 -00 replies
2/12fs: cifs: Change cifs_fscache_inode_auxdata to use vfs_time data typefs/cifs/cache.c+2 -20 replies
3/12fs: cifs: Change cifs_fattr timestamps data type to vfs_timefs/cifs/cifsencrypt.c fs/cifs/cifsglob.h fs/cifs/cifsproto.h +3 more+15 -150 replies
4/12fs: cifs: Make cnvrtDosUnixTm() y2038 safefs/cifs/netmisc.c+3 -20 replies
5/12fs: cifs: Use vfs_time_get_real_* time functionsfs/cifs/cifsencrypt.c fs/cifs/cifssmb.c+4 -40 replies
6/12fs: btrfs: Change btrfs_inode.i_otime to use vfs_time data typefs/btrfs/btrfs_inode.h+1 -10 replies
7/12fs: btrfs: Use vfs_time data type for btrfs_update_time()fs/btrfs/inode.c+1 -10 replies
8/12fs: btrfs: Change timespec data types to use vfs_timefs/btrfs/file.c fs/btrfs/ioctl.c fs/btrfs/root-tree.c +1 more+7 -70 replies
9/12fs: ceph: Change encode and decode functions to use vfs_timeinclude/linux/ceph/decode.h+6 -50 replies
10/12fs: ceph: Replace timespec data type with vfs_timefs/ceph/cache.c fs/ceph/caps.c fs/ceph/dir.c +6 more+32 -320 replies
11/12net: ceph: use vfs_time data type instead of timespecinclude/linux/ceph/messenger.h include/linux/ceph/osd_client.h net/ceph/osd_client.c+4 -35 replies
12/12fs: xfs: change inode times to use vfs_time data typefs/xfs/xfs_inode.c fs/xfs/xfs_iops.c fs/xfs/xfs_trans_inode.c+5 -50 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [RFC v2a 00/12] vfs 64 bit time transition proposal

Introduction

The series is one of the proposals on how to transition VFS timestamps to use 64 bit time. This is the inode_timespec idea proposed in the original RFC series. The type name has been changed to vfs_time based on Dave Chinner’s suggestion.

Solution

This series defines a new type name for vfs timestamps: vfs_time. All the individual file systems will use struct vfs_time to access vfs timestamps. Current time APIs also return vfs times and are considered to be part of vfs as they are exclusively used for inode timestamps.

These are the steps involved:

1.Define vfs_time as an alias to timespec
2.Change individual filesystems to use only vfs_time data type
for timestamps. Make sure each filesystem will be safe when converted to use timespec64
3.Change vfs_time to be an alias for timespec64. 4.Change all filesystems to use timespec64 directly.

Note that the series only includes patches for steps 1 and 2.

Concerns

1.Before the vfs layer is done, it might be confusing to developers
as individual filesystems now will have two options for access:
timespec and vfs_time type.

Deepa Dinamani (12):
vfs: Add vfs_time abstractions
fs: cifs: Change cifs_fscache_inode_auxdata to use vfs_time data type
fs: cifs: Change cifs_fattr timestamps data type to vfs_time
fs: cifs: Make cnvrtDosUnixTm() y2038 safe
fs: cifs: Use vfs_time_get_real_* time functions
fs: btrfs: Change btrfs_inode.i_otime to use vfs_time data type
fs: btrfs: Use vfs_time data type for btrfs_update_time()
fs: btrfs: Change timespec data types to use vfs_time
fs: ceph: Change encode and decode functions to use vfs_time
fs: ceph: Replace timespec data type with vfs_time
net: ceph: use vfs_time data type instead of timespec
fs: xfs: change inode times to use vfs_time data type

 fs/btrfs/btrfs_inode.h          |  2 +-
 fs/btrfs/file.c                 |  6 +++---
 fs/btrfs/inode.c                |  2 +-
 fs/btrfs/ioctl.c                |  4 ++--
 fs/btrfs/root-tree.c            |  2 +-
 fs/btrfs/transaction.c          |  2 +-
 fs/ceph/cache.c                 |  2 +-
 fs/ceph/caps.c                  |  6 +++---
 fs/ceph/dir.c                   |  4 ++--
 fs/ceph/file.c                  |  6 +++---
 fs/ceph/inode.c                 | 32 ++++++++++++++++----------------
 fs/ceph/mds_client.c            |  2 +-
 fs/ceph/mds_client.h            |  2 +-
 fs/ceph/super.h                 |  8 ++++----
 fs/ceph/xattr.c                 |  2 +-
 fs/cifs/cache.c                 |  4 ++--
 fs/cifs/cifsencrypt.c           |  4 ++--
 fs/cifs/cifsglob.h              |  6 +++---
 fs/cifs/cifsproto.h             |  6 +++---
 fs/cifs/cifssmb.c               | 10 +++++-----
 fs/cifs/inode.c                 |  2 +-
 fs/cifs/netmisc.c               | 15 ++++++++-------
 fs/xfs/xfs_inode.c              |  2 +-
 fs/xfs/xfs_iops.c               |  2 +-
 fs/xfs/xfs_trans_inode.c        |  6 +++---
 include/linux/ceph/decode.h     | 11 ++++++-----
 include/linux/ceph/messenger.h  |  1 +
 include/linux/ceph/osd_client.h |  4 ++--
 include/linux/fs.h              | 19 +++++++++++++++++++
 net/ceph/osd_client.c           |  2 +-
 30 files changed, 99 insertions(+), 77 deletions(-)
Signature · 1 line
1.9.1