Re: [PATCH] exofs: Avoid using file_fsync()

[Boaz Harrosh <bharrosh@panasas.com>]

On 06/15/2009 04:50 PM, Christoph Hellwig wrote:
> On Mon, Jun 15, 2009 at 04:31:01PM +0300, Boaz Harrosh wrote:
>> +	ret = write_inode_now(inode, 0);
> 
> You shouldn't need a write_inode_now, but rather just a similar
> sync_inode call as in ext2 or the new simple_fsync as data was
> already written by the VFS.
> 

I looked into it some more and need your advise please?

It looks simple_fsync, sync_inode, and write_inode_now all do
__writeback_single_inode() inside. But write_inode_now also
waits.

I think I want to wait here, it's my close-to-open barrier.

No?

Thanks
Boaz