int ret, needs_barrier = 0;
        tid_t commit_tid;
  
 -      J_ASSERT(ext3_journal_current_handle() == NULL);
 -
+       trace_ext3_sync_file_enter(file, datasync);
+ 
        if (inode->i_sb->s_flags & MS_RDONLY)
                return 0;
  
-               return ret;
 +      ret = filemap_write_and_wait_range(inode->i_mapping, start, end);
 +      if (ret)
++              goto out;
 +
 +      /*
 +       * Taking the mutex here just to keep consistent with how fsync was
 +       * called previously, however it looks like we don't need to take
 +       * i_mutex at all.
 +       */
 +      mutex_lock(&inode->i_mutex);
 +
 +      J_ASSERT(ext3_journal_current_handle() == NULL);
  
        /*
         * data=writeback,ordered:
         *  safe in-journal, which is all fsync() needs to ensure.
         */
        if (ext3_should_journal_data(inode)) {
-               return ext3_force_commit(inode->i_sb);
 +              mutex_unlock(&inode->i_mutex);
+               ret = ext3_force_commit(inode->i_sb);
+               goto out;
        }
  
        if (datasync)
         */
        if (needs_barrier)
                blkdev_issue_flush(inode->i_sb->s_bdev, GFP_KERNEL, NULL);
+ 
 +      mutex_unlock(&inode->i_mutex);
+ out:
+       trace_ext3_sync_file_exit(inode, ret);
        return ret;
  }