On Feb 27, 2015, at 11:17 PM, Trond Myklebust <[email protected]> wrote:
> Signed-off-by: Trond Myklebust <[email protected]>
Tested-by: Chuck Lever <[email protected]>
I applied this series and the patch that addresses the generic/133
O_DIRECT regression to stock 3.19.0.
I tested using xfstests with NFSv3 and NFSv4.0, Solaris 11 update 2
and Linux 3.19 servers. No new test failures (between 2 and 4
failures out of 55 tests run, which is typical).
I also used dbench and fio to look for performance regressions.
No regression found so far.
> ---
> fs/nfs/inode.c | 16 ++++++++++++++++
> include/linux/nfs_fs.h | 1 +
> 2 files changed, 17 insertions(+)
>
> diff --git a/fs/nfs/inode.c b/fs/nfs/inode.c
> index 83107be3dd01..b0cbc1ba82da 100644
> --- a/fs/nfs/inode.c
> +++ b/fs/nfs/inode.c
> @@ -1260,6 +1260,22 @@ void nfs_fattr_init(struct nfs_fattr *fattr)
> }
> EXPORT_SYMBOL_GPL(nfs_fattr_init);
>
> +/**
> + * nfs_fattr_set_barrier
> + * @fattr: attributes
> + *
> + * Used to set a barrier after an attribute was updated. This
> + * barrier ensures that older attributes from RPC calls that may
> + * have raced with our update cannot clobber these new values.
> + * Note that you are still responsible for ensuring that other
> + * operations which change the attribute on the server do not
> + * collide.
> + */
> +void nfs_fattr_set_barrier(struct nfs_fattr *fattr)
> +{
> + fattr->gencount = nfs_inc_attr_generation_counter();
> +}
> +
> struct nfs_fattr *nfs_alloc_fattr(void)
> {
> struct nfs_fattr *fattr;
> diff --git a/include/linux/nfs_fs.h b/include/linux/nfs_fs.h
> index 2f77e0c651c8..3a4ffb5856cd 100644
> --- a/include/linux/nfs_fs.h
> +++ b/include/linux/nfs_fs.h
> @@ -369,6 +369,7 @@ extern struct nfs_lock_context *nfs_get_lock_context(struct nfs_open_context *ct
> extern void nfs_put_lock_context(struct nfs_lock_context *l_ctx);
> extern u64 nfs_compat_user_ino64(u64 fileid);
> extern void nfs_fattr_init(struct nfs_fattr *fattr);
> +extern void nfs_fattr_set_barrier(struct nfs_fattr *fattr);
> extern unsigned long nfs_inc_attr_generation_counter(void);
>
> extern struct nfs_fattr *nfs_alloc_fattr(void);
> --
> 2.1.0
>
--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com
On Sun, Mar 1, 2015 at 9:35 PM, Chuck Lever <[email protected]> wrote:
>
> On Feb 27, 2015, at 11:17 PM, Trond Myklebust <[email protected]> wrote:
>
>> Signed-off-by: Trond Myklebust <[email protected]>
>
> Tested-by: Chuck Lever <[email protected]>
>
> I applied this series and the patch that addresses the generic/133
> O_DIRECT regression to stock 3.19.0.
>
> I tested using xfstests with NFSv3 and NFSv4.0, Solaris 11 update 2
> and Linux 3.19 servers. No new test failures (between 2 and 4
> failures out of 55 tests run, which is typical).
>
> I also used dbench and fio to look for performance regressions.
> No regression found so far.
>
Thanks!
--
Trond Myklebust
Linux NFS client maintainer, PrimaryData
[email protected]