Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 845802b1 authored by Andreas Gruenbacher's avatar Andreas Gruenbacher Committed by Bob Peterson
Browse files

gfs2: Remove ordered write mode handling from gfs2_trans_add_data 



In journaled data mode, we need to add each buffer head to the current
transaction.  In ordered write mode, we only need to add the inode to
the ordered inode list.  So far, both cases are handled in
gfs2_trans_add_data.  This makes the code look misleading and is
inefficient for small block sizes as well.  Handle both cases separately
instead.

Signed-off-by: default avatarAndreas Gruenbacher <agruenba@redhat.com>
Signed-off-by: default avatarBob Peterson <rpeterso@redhat.com>
parent d6382a35

fs/gfs2/aops.c

+4 −3
Original line number Diff line number Diff line
@@ -54,7 +54,6 @@ static void gfs2_page_add_databufs(struct gfs2_inode *ip, struct page *page, 
			continue;

		if (start >= to)

			break;

		if (gfs2_is_jdata(ip))

		set_buffer_uptodate(bh);

		gfs2_trans_add_data(ip->i_gl, bh);

	}
@@ -894,8 +893,10 @@ static int gfs2_write_end(struct file *file, struct address_space *mapping, 
		goto out2;

	}



	if (!gfs2_is_writeback(ip))

	if (gfs2_is_jdata(ip))

		gfs2_page_add_databufs(ip, page, pos & ~PAGE_MASK, len);

	else

		gfs2_ordered_add_inode(ip);



	ret = generic_write_end(file, mapping, pos, len, copied, page, fsdata);

	page = NULL;

fs/gfs2/bmap.c

+8 −4
Original line number Diff line number Diff line
@@ -89,10 +89,12 @@ static int gfs2_unstuffer_page(struct gfs2_inode *ip, struct buffer_head *dibh, 
		map_bh(bh, inode->i_sb, block);



	set_buffer_uptodate(bh);

	if (!gfs2_is_jdata(ip))

		mark_buffer_dirty(bh);

	if (!gfs2_is_writeback(ip))

	if (gfs2_is_jdata(ip))

		gfs2_trans_add_data(ip->i_gl, bh);

	else {

		mark_buffer_dirty(bh);

		gfs2_ordered_add_inode(ip);

	}



	if (release) {

		unlock_page(page);
@@ -1028,8 +1030,10 @@ static int gfs2_block_zero_range(struct inode *inode, loff_t from, 
		err = 0;

	}



	if (!gfs2_is_writeback(ip))

	if (gfs2_is_jdata(ip))

		gfs2_trans_add_data(ip->i_gl, bh);

	else

		gfs2_ordered_add_inode(ip);



	zero_user(page, offset, length);

	mark_buffer_dirty(bh);

fs/gfs2/log.h

+6 −1
Original line number Diff line number Diff line
@@ -14,6 +14,7 @@ 
#include <linux/spinlock.h>

#include <linux/writeback.h>

#include "incore.h"

#include "inode.h"



/**

 * gfs2_log_lock - acquire the right to mess with the log manager
@@ -50,8 +51,12 @@ static inline void gfs2_log_pointers_init(struct gfs2_sbd *sdp, 


static inline void gfs2_ordered_add_inode(struct gfs2_inode *ip)

{

	struct gfs2_sbd *sdp = GFS2_SB(&ip->i_inode);

	struct gfs2_sbd *sdp;



	if (!gfs2_is_ordered(ip))

		return;



	sdp = GFS2_SB(&ip->i_inode);

	if (!test_bit(GIF_ORDERED, &ip->i_flags)) {

		spin_lock(&sdp->sd_ordered_lock);

		if (!test_and_set_bit(GIF_ORDERED, &ip->i_flags))

fs/gfs2/quota.c

+4 −1
Original line number Diff line number Diff line
@@ -735,7 +735,10 @@ static int gfs2_write_buf_to_page(struct gfs2_inode *ip, unsigned long index, 
			if (!buffer_uptodate(bh))

				goto unlock_out;

		}

		if (gfs2_is_jdata(ip))

			gfs2_trans_add_data(ip->i_gl, bh);

		else

			gfs2_ordered_add_inode(ip);



		/* If we need to write to the next block as well */

		if (to_write > (bsize - boff)) {

fs/gfs2/trans.c

+8 −19
Original line number Diff line number Diff line
@@ -143,11 +143,7 @@ static struct gfs2_bufdata *gfs2_alloc_bufdata(struct gfs2_glock *gl, 
 * @gl: The inode glock associated with the buffer

 * @bh: The buffer to add

 *

 * This is used in two distinct cases:

 * i) In ordered write mode

 *    We put the data buffer on a list so that we can ensure that it's

 *    synced to disk at the right time

 * ii) In journaled data mode

 * This is used in journaled data mode.

 * We need to journal the data block in the same way as metadata in

 * the functions above. The difference is that here we have a tag

 * which is two __be64's being the block number (as per meta data)
@@ -160,15 +156,8 @@ void gfs2_trans_add_data(struct gfs2_glock *gl, struct buffer_head *bh) 
{

	struct gfs2_trans *tr = current->journal_info;

	struct gfs2_sbd *sdp = gl->gl_name.ln_sbd;

	struct address_space *mapping = bh->b_page->mapping;

	struct gfs2_inode *ip = GFS2_I(mapping->host);

	struct gfs2_bufdata *bd;



	if (!gfs2_is_jdata(ip)) {

		gfs2_ordered_add_inode(ip);

		return;

	}



	lock_buffer(bh);

	if (buffer_pinned(bh)) {

		set_bit(TR_TOUCHED, &tr->tr_flags);