Previous patch didn't export sync_inode_wodata(), that had a problem with modula ext2 fs. Bug-fixed version is attached here. Machida, Hiroyuki wrote: > > I revise a previous patch. Now checking dirty flag is done > at vfs layer, as OGAWA-san said. I realized ext2_sync_inode() > is good for syncing inode without it's data. I moved it to vfs layer > and rename it to sync_inode_wodata(). > > The first patch attached here is changes on vfs layer. > Second patch attached at the next mail is changes on ext2 fs. > > > OGAWA Hirofumi wrote: > >> "Machida, Hiroyuki" writes: >> >> >>> + if ( (!error) && IS_SYNC(inode)) { >>> + error = write_inode_now(inode, 1); >>> + } >> >> >> >> We don't need to sync the data pages at all here. And I think it is >> not right place for doing this. If we need this, since we need to see >> O_SYNC for fchxxx() VFS would be right place to do it. >> >> But, personally, I'd like to kill the "-o sync" stuff for these >> independent meta data rather. Then... > > > > ------------------------------------------------------------------------ > > > This patch adds inode-sync after attribute changes, if needed. > > * fs-sync-attr.patch for 2.6.13 > > fs/fs-writeback.c | 19 +++++++++++++++++++ > fs/open.c | 12 ++++++++++++ > include/linux/fs.h | 1 + > 4 files changed, 32 insertions(+) > > Signed-off-by: Hiroyuki Machida : --