diff options
author | Steven Whitehouse <swhiteho@redhat.com> | 2008-09-18 08:53:59 -0400 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2008-09-18 08:53:59 -0400 |
commit | 719ee344675c2efed9115934f19aa66a526b6e5b (patch) | |
tree | 7f9502d696a09eaf5eba0a0d3fa35557ffd3c503 /fs/gfs2/incore.h | |
parent | 37ec89e83c4ca98323fe74f139301ff3949cfdb6 (diff) |
GFS2: high time to take some time over atime
Until now, we've used the same scheme as GFS1 for atime. This has failed
since atime is a per vfsmnt flag, not a per fs flag and as such the
"noatime" flag was not getting passed down to the filesystems. This
patch removes all the "special casing" around atime updates and we
simply use the VFS's atime code.
The net result is that GFS2 will now support all the same atime related
mount options of any other filesystem on a per-vfsmnt basis. We do lose
the "lazy atime" updates, but we gain "relatime". We could add lazy
atime to the VFS at a later date, if there is a requirement for that
variant still - I suspect relatime will be enough.
Also we lose about 100 lines of code after this patch has been applied,
and I have a suspicion that it will speed things up a bit, even when
atime is "on". So it seems like a nice clean up as well.
From a user perspective, everything stays the same except the loss of
the per-fs atime quantum tweekable (ought to be per-vfsmnt at the very
least, and to be honest I don't think anybody ever used it) and that a
number of options which were ignored before now work correctly.
Please let me know if you've got any comments. I'm pushing this out
early so that you can all see what my plans are.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/incore.h')
-rw-r--r-- | fs/gfs2/incore.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/gfs2/incore.h b/fs/gfs2/incore.h index b2c5784092cf..f1ed3a1bf8aa 100644 --- a/fs/gfs2/incore.h +++ b/fs/gfs2/incore.h | |||
@@ -420,7 +420,6 @@ struct gfs2_tune { | |||
420 | unsigned int gt_quota_scale_den; /* Denominator */ | 420 | unsigned int gt_quota_scale_den; /* Denominator */ |
421 | unsigned int gt_quota_cache_secs; | 421 | unsigned int gt_quota_cache_secs; |
422 | unsigned int gt_quota_quantum; /* Secs between syncs to quota file */ | 422 | unsigned int gt_quota_quantum; /* Secs between syncs to quota file */ |
423 | unsigned int gt_atime_quantum; /* Min secs between atime updates */ | ||
424 | unsigned int gt_new_files_jdata; | 423 | unsigned int gt_new_files_jdata; |
425 | unsigned int gt_max_readahead; /* Max bytes to read-ahead from disk */ | 424 | unsigned int gt_max_readahead; /* Max bytes to read-ahead from disk */ |
426 | unsigned int gt_stall_secs; /* Detects trouble! */ | 425 | unsigned int gt_stall_secs; /* Detects trouble! */ |
@@ -433,7 +432,6 @@ enum { | |||
433 | SDF_JOURNAL_CHECKED = 0, | 432 | SDF_JOURNAL_CHECKED = 0, |
434 | SDF_JOURNAL_LIVE = 1, | 433 | SDF_JOURNAL_LIVE = 1, |
435 | SDF_SHUTDOWN = 2, | 434 | SDF_SHUTDOWN = 2, |
436 | SDF_NOATIME = 3, | ||
437 | }; | 435 | }; |
438 | 436 | ||
439 | #define GFS2_FSNAME_LEN 256 | 437 | #define GFS2_FSNAME_LEN 256 |