diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2010-05-27 13:26:37 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-05-27 13:26:37 -0400 |
commit | e4ce30f3779c2ddaa7dfaa4042209e5dbacbada5 (patch) | |
tree | cc64c1dcd16b5dbf71ebc8338b339e6fb04abaee /fs/ext4/fsync.c | |
parent | b899ebeb05da4287ce845976727e3e83dadd25d5 (diff) | |
parent | 14ece1028b3ed53ffec1b1213ffc6acaf79ad77c (diff) |
Merge branch 'for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4
* 'for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4: (40 commits)
ext4: Make fsync sync new parent directories in no-journal mode
ext4: Drop whitespace at end of lines
ext4: Fix compat EXT4_IOC_ADD_GROUP
ext4: Conditionally define compat ioctl numbers
tracing: Convert more ext4 events to DEFINE_EVENT
ext4: Add new tracepoints to track mballoc's buddy bitmap loads
ext4: Add a missing trace hook
ext4: restart ext4_ext_remove_space() after transaction restart
ext4: Clear the EXT4_EOFBLOCKS_FL flag only when warranted
ext4: Avoid crashing on NULL ptr dereference on a filesystem error
ext4: Use bitops to read/modify i_flags in struct ext4_inode_info
ext4: Convert calls of ext4_error() to EXT4_ERROR_INODE()
ext4: Convert callers of ext4_get_blocks() to use ext4_map_blocks()
ext4: Add new abstraction ext4_map_blocks() underneath ext4_get_blocks()
ext4: Use our own write_cache_pages()
ext4: Show journal_checksum option
ext4: Fix for ext4_mb_collect_stats()
ext4: check for a good block group before loading buddy pages
ext4: Prevent creation of files larger than RLIMIT_FSIZE using fallocate
ext4: Remove extraneous newlines in ext4_msg() calls
...
Fixed up trivial conflict in fs/ext4/fsync.c
Diffstat (limited to 'fs/ext4/fsync.c')
-rw-r--r-- | fs/ext4/fsync.c | 35 |
1 files changed, 31 insertions, 4 deletions
diff --git a/fs/ext4/fsync.c b/fs/ext4/fsync.c index ef3d980e67cb..b6a74f991bf4 100644 --- a/fs/ext4/fsync.c +++ b/fs/ext4/fsync.c | |||
@@ -35,6 +35,29 @@ | |||
35 | #include <trace/events/ext4.h> | 35 | #include <trace/events/ext4.h> |
36 | 36 | ||
37 | /* | 37 | /* |
38 | * If we're not journaling and this is a just-created file, we have to | ||
39 | * sync our parent directory (if it was freshly created) since | ||
40 | * otherwise it will only be written by writeback, leaving a huge | ||
41 | * window during which a crash may lose the file. This may apply for | ||
42 | * the parent directory's parent as well, and so on recursively, if | ||
43 | * they are also freshly created. | ||
44 | */ | ||
45 | static void ext4_sync_parent(struct inode *inode) | ||
46 | { | ||
47 | struct dentry *dentry = NULL; | ||
48 | |||
49 | while (inode && ext4_test_inode_state(inode, EXT4_STATE_NEWENTRY)) { | ||
50 | ext4_clear_inode_state(inode, EXT4_STATE_NEWENTRY); | ||
51 | dentry = list_entry(inode->i_dentry.next, | ||
52 | struct dentry, d_alias); | ||
53 | if (!dentry || !dentry->d_parent || !dentry->d_parent->d_inode) | ||
54 | break; | ||
55 | inode = dentry->d_parent->d_inode; | ||
56 | sync_mapping_buffers(inode->i_mapping); | ||
57 | } | ||
58 | } | ||
59 | |||
60 | /* | ||
38 | * akpm: A new design for ext4_sync_file(). | 61 | * akpm: A new design for ext4_sync_file(). |
39 | * | 62 | * |
40 | * This is only called from sys_fsync(), sys_fdatasync() and sys_msync(). | 63 | * This is only called from sys_fsync(), sys_fdatasync() and sys_msync(). |
@@ -66,9 +89,13 @@ int ext4_sync_file(struct file *file, struct dentry *dentry, int datasync) | |||
66 | ret = flush_completed_IO(inode); | 89 | ret = flush_completed_IO(inode); |
67 | if (ret < 0) | 90 | if (ret < 0) |
68 | return ret; | 91 | return ret; |
69 | 92 | ||
70 | if (!journal) | 93 | if (!journal) { |
71 | return simple_fsync(file, dentry, datasync); | 94 | ret = simple_fsync(file, dentry, datasync); |
95 | if (!ret && !list_empty(&inode->i_dentry)) | ||
96 | ext4_sync_parent(inode); | ||
97 | return ret; | ||
98 | } | ||
72 | 99 | ||
73 | /* | 100 | /* |
74 | * data=writeback,ordered: | 101 | * data=writeback,ordered: |
@@ -102,7 +129,7 @@ int ext4_sync_file(struct file *file, struct dentry *dentry, int datasync) | |||
102 | (journal->j_flags & JBD2_BARRIER)) | 129 | (journal->j_flags & JBD2_BARRIER)) |
103 | blkdev_issue_flush(inode->i_sb->s_bdev, GFP_KERNEL, | 130 | blkdev_issue_flush(inode->i_sb->s_bdev, GFP_KERNEL, |
104 | NULL, BLKDEV_IFL_WAIT); | 131 | NULL, BLKDEV_IFL_WAIT); |
105 | jbd2_log_wait_commit(journal, commit_tid); | 132 | ret = jbd2_log_wait_commit(journal, commit_tid); |
106 | } else if (journal->j_flags & JBD2_BARRIER) | 133 | } else if (journal->j_flags & JBD2_BARRIER) |
107 | blkdev_issue_flush(inode->i_sb->s_bdev, GFP_KERNEL, NULL, | 134 | blkdev_issue_flush(inode->i_sb->s_bdev, GFP_KERNEL, NULL, |
108 | BLKDEV_IFL_WAIT); | 135 | BLKDEV_IFL_WAIT); |