diff options
| author | Dave Chinner <dchinner@redhat.com> | 2011-01-04 04:49:29 +0000 |
|---|---|---|
| committer | Alex Elder <aelder@sgi.com> | 2011-01-11 20:28:17 -0600 |
| commit | c58efdb442bb49dea1d148f207560c41918c1bf4 (patch) | |
| tree | d6e117ae65ff7ff921600ef936a7bbb4e3130010 /fs/xfs/linux-2.6/xfs_super.c | |
| parent | eda77982729b7170bdc9e8855f0682edf322d277 (diff) | |
| download | op-kernel-dev-c58efdb442bb49dea1d148f207560c41918c1bf4.zip op-kernel-dev-c58efdb442bb49dea1d148f207560c41918c1bf4.tar.gz | |
xfs: ensure log covering transactions are synchronous
To ensure the log is covered and the filesystem idles correctly, we
need to ensure that dummy transactions hit the disk and do not stay
pinned in memory. If the superblock is pinned in memory, it can't
be flushed so the log covering cannot make progress. The result is
dependent on timing - more oftent han not we continue to issues a
log covering transaction every 36s rather than idling after ~90s.
Fix this by making the log covering transaction synchronous. To
avoid additional log force from xfssyncd, make the log covering
transaction take the place of the existing log force in the xfssyncd
background sync process.
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Alex Elder <aelder@sgi.com>
Diffstat (limited to 'fs/xfs/linux-2.6/xfs_super.c')
| -rw-r--r-- | fs/xfs/linux-2.6/xfs_super.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/linux-2.6/xfs_super.c b/fs/xfs/linux-2.6/xfs_super.c index c51faaa..af32f37 100644 --- a/fs/xfs/linux-2.6/xfs_super.c +++ b/fs/xfs/linux-2.6/xfs_super.c @@ -1413,7 +1413,7 @@ xfs_fs_freeze( xfs_save_resvblks(mp); xfs_quiesce_attr(mp); - return -xfs_fs_log_dummy(mp, SYNC_WAIT); + return -xfs_fs_log_dummy(mp); } STATIC int |
