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 f3fba84e4..6ae0426ac 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 6b3c2113d..f0bdc86f0 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 */ |