[PATCH 5/6] leases: break leases on any attribute modification
J. Bruce Fields
bfields at fieldses.org
Wed Sep 21 09:35:30 MDT 2011
On Wed, Sep 21, 2011 at 10:58:16AM -0400, J. Bruce Fields wrote:
> NFSv4 uses leases to guarantee that clients can cash metadata as well as
^^^^
Um, you know what I meant. Corrected in my local repo.--b.
> data.
>
> (I suspect the same is true for Samba.)
>
> This covers chmod, chown, etc.
>
> Signed-off-by: J. Bruce Fields <bfields at redhat.com>
> ---
> fs/attr.c | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/fs/attr.c b/fs/attr.c
> index 538e279..4ce31cb 100644
> --- a/fs/attr.c
> +++ b/fs/attr.c
> @@ -231,6 +231,9 @@ int notify_change(struct dentry * dentry, struct iattr * attr)
> error = security_inode_setattr(dentry, attr);
> if (error)
> return error;
> + error = break_lease(inode, O_WRONLY);
> + if (error)
> + return error;
>
> if (inode->i_op->setattr)
> error = inode->i_op->setattr(dentry, attr);
> --
> 1.7.4.1
>
More information about the samba-technical
mailing list