diff options
| author | Amar Tumballi <amarts@redhat.com> | 2012-08-17 14:21:07 +0530 | 
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-08-20 00:16:01 -0700 | 
| commit | 2f2e3bfb5ef89b5ba266a3df7496f95b11fb93e1 (patch) | |
| tree | cf4b12316f6fd241858614690dcdc35b284ea7f9 /api | |
| parent | 99f0daf2f46d56b78adfe1a3df1195e1388feaab (diff) | |
syncop: handle 'dataonly' flag in syncop_fsync()
* and also in syncop_readv(), don't look at _cbk args if op_ret
  is < 0.
Change-Id: I3ab2982bc6d186e75b6adb74c8981e4ff7058bbe
Signed-off-by: Amar Tumballi <amarts@redhat.com>
BUG: 839950
Reviewed-on: http://review.gluster.org/3828
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'api')
| -rw-r--r-- | api/src/glfs-fops.c | 6 | ||||
| -rw-r--r-- | api/src/glfs.c | 2 | 
2 files changed, 2 insertions, 6 deletions
diff --git a/api/src/glfs-fops.c b/api/src/glfs-fops.c index f3fba84e4e7..6ae0426ac8f 100644 --- a/api/src/glfs-fops.c +++ b/api/src/glfs-fops.c @@ -699,8 +699,7 @@ glfs_fsync (struct glfs_fd *glfd)  		goto out;  	} -	ret = syncop_fsync (subvol, glfd->fd); -//	ret = syncop_fsync (subvol, glfd->fd, 0); +	ret = syncop_fsync (subvol, glfd->fd, 0);  out:  	return ret;  } @@ -761,8 +760,7 @@ glfs_fdatasync (struct glfs_fd *glfd)  		goto out;  	} -	ret = syncop_fsync (subvol, glfd->fd); -//	ret = syncop_fsync (subvol, glfd->fd, 1); +	ret = syncop_fsync (subvol, glfd->fd, 1);  out:  	return ret;  } diff --git a/api/src/glfs.c b/api/src/glfs.c index 6b3c2113d53..f0bdc86f0c6 100644 --- a/api/src/glfs.c +++ b/api/src/glfs.c @@ -16,14 +16,12 @@      reach it via THIS.    - fd migration on graph switch.    - update syncop functions to accept/return xdata. ??? -  - syncop_readv to not touch params if args.op_ret < 0.    - protocol/client to reconnect immediately after portmap disconnect.    - handle SEEK_END failure in _lseek()    - handle umask (per filesystem?)    - implement glfs_set_xlator_option(), like --xlator-option    - make itables LRU based    - implement glfs_fini() -  - modify syncop_fsync() to accept 'dataonly' flag    - 0-copy for readv/writev    - reconcile the open/creat mess  */  | 
