diff options
author | Cyril Plisko <[email protected]> | 2012-09-16 09:03:04 +0300 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2012-10-03 10:49:45 -0700 |
commit | 393b44c7112854fc11dc97885a87f666780cc9fe (patch) | |
tree | 1348aeb3607c55d66660c8424d5e5498bd328180 /module/zfs | |
parent | 23a61ccc1b89fa5b89b5d8383d57cf5152406cfc (diff) |
Implement .commit_metadata hook for NFS export
In order to implement synchronous NFS metadata semantics ZFS
needs to provide the .commit_metadata hook. All it takes there
is to make sure changes are committed to ZIL. Fortunately
zfs_fsync() does just that, so simply calling it from
zpl_commit_metadata() does the trick.
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #969
Diffstat (limited to 'module/zfs')
-rw-r--r-- | module/zfs/zpl_export.c | 22 |
1 files changed, 21 insertions, 1 deletions
diff --git a/module/zfs/zpl_export.c b/module/zfs/zpl_export.c index 395c5584c..94625e13c 100644 --- a/module/zfs/zpl_export.c +++ b/module/zfs/zpl_export.c @@ -20,6 +20,7 @@ */ /* * Copyright (c) 2011 Gunnar Beutner + * Copyright (c) 2012 Cyril Plisko. All rights reserved. */ @@ -121,8 +122,27 @@ zpl_get_parent(struct dentry *child) return zpl_dentry_obtain_alias(ip); } +#ifdef HAVE_COMMIT_METADATA +static int +zpl_commit_metadata(struct inode *inode) +{ + cred_t *cr = CRED(); + int error; + + crhold(cr); + error = -zfs_fsync(inode, 0, cr); + crfree(cr); + ASSERT3S(error, <=, 0); + + return error; +} +#endif /* HAVE_COMMIT_METADATA */ + const struct export_operations zpl_export_operations = { .encode_fh = zpl_encode_fh, .fh_to_dentry = zpl_fh_to_dentry, - .get_parent = zpl_get_parent + .get_parent = zpl_get_parent, +#ifdef HAVE_COMMIT_METADATA + .commit_metadata= zpl_commit_metadata, +#endif /* HAVE_COMMIT_METADATA */ }; |