[Cluster-devel] [GFS2 PATCH][TRY #2] GFS2: eliminate redundant log flush

[Bob Peterson <rpeterso@redhat.com>]

----- Original Message -----
| ----- Original Message -----
| | What ensures that the revokes are on disk?
| (snip)
| | This one looks like it is probably correct, however there is no
| | point
| | in
| | adding a comment to refer to code that has just been removed. Just
| | remove that call and put the explanation in the patch description,
| | or
| | maybe include just a short comment so say that there is a log flush
| | implicit in the write_inode_now() call,
| | 
| | Steve.
| 
| You're right about the revokes; my bad. Here is a respin, with just
| a comment in the patch rather than in the code:
| 
| Hi,
| 
| In function gfs2_evict_inode, there was a call to gfs2_log_flush
| that is unnecessary because it immediately calls write_inode_now
| which makes another call to gfs2_log_flush. This patch eliminates
| the redundant log flush.
| 
| Regards,
| 
| Bob Peterson
| Red Hat GFS
| 
| Signed-off-by: Bob Peterson <rpeterso@redhat.com>
| ---
| diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c
| index 3cbac68..ba4fde5 100644
| --- a/fs/gfs2/super.c
| +++ b/fs/gfs2/super.c
| @@ -1542,7 +1542,6 @@ static void gfs2_evict_inode(struct inode
| *inode)
|  	goto out_unlock;
|  
|  out_truncate:
| -	gfs2_log_flush(sdp, ip->i_gl);
|  	write_inode_now(inode, 1);
|  	gfs2_ail_flush(ip->i_gl, 0);
|  
| 

Hi,

I've still not heard back on this revised patch from 09 Aug, and it
has not gone into -nmw. Perhaps it was missed?

Regards,

Bob Peterson
Red Hat File Systems