>From Malahal <mala...@gmail.com>:
Malahal has uploaded this change for review. (
https://review.gerrithub.io/368729
Change subject: [FSAL_GPFS] remove fsync() from gpfs_write2()
......................................................................
[FSAL_GPFS] remove fsync() from gpfs_write2()
We don't check for EUNATCH error after fsync() system call, so we end up
propagating a failure to NFS client applications. GPFSFSAL_write
or gpfs_write_plus_fd take the given fsal_stable and sync the data if
needed. So there is no need to make a separate fsync() here.
Change-Id: I4ff7e14f3323bcc4f0dc3fc3a4e21435992ea308
Signed-off-by: Malahal Naineni <mala...@us.ibm.com>
---
M src/FSAL/FSAL_GPFS/file.c
1 file changed, 0 insertions(+), 14 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha
refs/changes/29/368729/1
--
To view, visit https://review.gerrithub.io/368729
To unsubscribe, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4ff7e14f3323bcc4f0dc3fc3a4e21435992ea308
Gerrit-Change-Number: 368729
Gerrit-PatchSet: 1
Gerrit-Owner: Malahal <mala...@gmail.com>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
