* [Cluster-devel] [PATCH] GFS2: bz 496716 - GFS2 ">>" will not update ctime, mtime after appending to the file
2009-05-12 16:16 [Cluster-devel] [PATCH] GFS2: bz 496716 - GFS2 ">>" will not update ctime, mtime after appending to the file Abhijith Das
@ 2009-05-12 16:21 ` Steven Whitehouse
0 siblings, 0 replies; 2+ messages in thread
From: Steven Whitehouse @ 2009-05-12 16:21 UTC (permalink / raw)
To: cluster-devel.redhat.com
Hi,
Now in the -nmw git tree. Thanks,
Steve.
On Tue, 2009-05-12 at 11:16 -0500, Abhijith Das wrote:
> This patch copies the timestamps from the vfs inode into gfs2 and syncs
> it to the disk inode during writes.
>
> Signed-off-by: Abhijith Das <adas@redhat.com>
> plain text document attachment (bz496716-4-git.patch)
> diff --git a/fs/gfs2/ops_address.c b/fs/gfs2/ops_address.c
> index a6dde17..e566421 100644
> --- a/fs/gfs2/ops_address.c
> +++ b/fs/gfs2/ops_address.c
> @@ -781,10 +781,12 @@ static int gfs2_stuffed_write_end(struct inode *inode, struct buffer_head *dibh,
> unlock_page(page);
> page_cache_release(page);
>
> - if (inode->i_size < to) {
> - i_size_write(inode, to);
> - ip->i_disksize = inode->i_size;
> - di->di_size = cpu_to_be64(inode->i_size);
> + if (copied) {
> + if (inode->i_size < to) {
> + i_size_write(inode, to);
> + ip->i_disksize = inode->i_size;
> + }
> + gfs2_dinode_out(ip, di);
> mark_inode_dirty(inode);
> }
>
> @@ -824,7 +826,6 @@ static int gfs2_write_end(struct file *file, struct address_space *mapping,
> struct gfs2_sbd *sdp = GFS2_SB(inode);
> struct buffer_head *dibh;
> struct gfs2_alloc *al = ip->i_alloc;
> - struct gfs2_dinode *di;
> unsigned int from = pos & (PAGE_CACHE_SIZE - 1);
> unsigned int to = from + len;
> int ret;
> @@ -847,11 +848,10 @@ static int gfs2_write_end(struct file *file, struct address_space *mapping,
> gfs2_page_add_databufs(ip, page, from, to);
>
> ret = generic_write_end(file, mapping, pos, len, copied, page, fsdata);
> -
> - if (likely(ret >= 0) && (inode->i_size > ip->i_disksize)) {
> - di = (struct gfs2_dinode *)dibh->b_data;
> - ip->i_disksize = inode->i_size;
> - di->di_size = cpu_to_be64(inode->i_size);
> + if (ret > 0) {
> + if (inode->i_size > ip->i_disksize)
> + ip->i_disksize = inode->i_size;
> + gfs2_dinode_out(ip, dibh->b_data);
> mark_inode_dirty(inode);
> }
>
^ permalink raw reply [flat|nested] 2+ messages in thread