Re: [PATCH 6/9] fuse: Push file_update_time() intofuse_page_mkwrite()

From: Jan Kara
Date: Thu Mar 01 2012 - 15:37:10 EST


On Thu 01-03-12 20:31:17, Miklos Szeredi wrote:
> Jan Kara <jack@xxxxxxx> writes:
>
> > CC: Miklos Szeredi <miklos@xxxxxxxxxx>
> > CC: fuse-devel@xxxxxxxxxxxxxxxxxxxxx
> > Signed-off-by: Jan Kara <jack@xxxxxxx>
> > ---
> > fs/fuse/file.c | 1 +
> > 1 files changed, 1 insertions(+), 0 deletions(-)
> >
> > diff --git a/fs/fuse/file.c b/fs/fuse/file.c
> > index 4a199fd..eade72e 100644
> > --- a/fs/fuse/file.c
> > +++ b/fs/fuse/file.c
> > @@ -1323,6 +1323,7 @@ static int fuse_page_mkwrite(struct vm_area_struct *vma, struct vm_fault *vmf)
> > */
> > struct inode *inode = vma->vm_file->f_mapping->host;
> >
> > + file_update_time(vma->vm_file);
>
> Fuse sets S_CMTIME in inode flags, so this is a no-op. IOW the patch is
> not needed.
I see. Thanks. Patch dropped.

Honza
--
Jan Kara <jack@xxxxxxx>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/