[Ocfs2-devel] [PATCH] Remove filesize checks for sync I/O journal commit
Goldwyn Rodrigues
rgoldwyn at suse.de
Wed Nov 5 07:26:28 PST 2014
On 11/04/2014 04:45 PM, Andrew Morton wrote:
> On Tue, 4 Nov 2014 09:52:22 -0600 Goldwyn Rodrigues <rgoldwyn at suse.de> wrote:
>
>> Filesize is not a good indication that the file needs to be synced.
>> An example where this breaks is:
>> 1. Open the file in O_SYNC|O_RDWR
>> 2. Read a small portion of the file (say 64 bytes)
>> 3. Lseek to starting of the file
>> 4. Write 64 bytes
>>
>> If the node crashes, it is not written out to disk because this
>> was not committed in the journal and the other node which reads
>> the file after recovery reads stale data (even if the write on
>> the other node was successful)
>>
>> ...
>>
>> --- a/fs/ocfs2/file.c
>> +++ b/fs/ocfs2/file.c
>> @@ -2381,9 +2381,7 @@ out_dio:
>> if (ret < 0)
>> written = ret;
>>
>> - if (!ret && ((old_size != i_size_read(inode)) ||
>> - (old_clusters != OCFS2_I(inode)->ip_clusters) ||
>> - has_refcount)) {
>> + if (!ret) {
>> ret = jbd2_journal_force_commit(osb->journal->j_journal);
>> if (ret < 0)
>> written = ret;
>
> Can we have a signed-off-by for this, please?
>
Oops. Missed that.
Signed-off-by: Goldwyn Rodrigues <rgoldwyn at suse.de>
--
Goldwyn
More information about the Ocfs2-devel
mailing list