All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tao Ma <tao.ma@oracle.com>
To: ocfs2-devel@oss.oracle.com
Subject: [Ocfs2-devel] Fix for online resize with metaecc
Date: Thu, 01 Apr 2010 10:02:05 +0800	[thread overview]
Message-ID: <4BB3FE9D.709@oracle.com> (raw)
In-Reply-To: <20100401013021.GG28680@mail.oracle.com>



Joel Becker wrote:
> Mark figured out what happened with bug 1214 - online resize writes out
> the superblocks without calculating metaecc.  Here's a patch to fix it.
> 
> Tao, Tiger, Tristan... Can one of you test this quickly?
Tested-by: Tao Ma <tao.ma@oracle.com>
> 
> Joel
> 
>>From d917c07dd45b16827a207e90a145ae248416c96c Mon Sep 17 00:00:00 2001
> From: Joel Becker <joel.becker@oracle.com>
> Date: Wed, 31 Mar 2010 18:25:44 -0700
> Subject: [PATCH] ocfs2: Compute metaecc for superblocks during online resize.
> 
> Online resize writes out the new superblock and its backups directly.
> The metaecc data wasn't being recomputed.  Let's do that directly.
> 
> Signed-off-by: Joel Becker <joel.becker@oracle.com>
> Cc: stable at kernel.org
> ---
>  fs/ocfs2/buffer_head_io.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/fs/ocfs2/buffer_head_io.c b/fs/ocfs2/buffer_head_io.c
> index 21c808f..b18c6d6 100644
> --- a/fs/ocfs2/buffer_head_io.c
> +++ b/fs/ocfs2/buffer_head_io.c
> @@ -407,6 +407,7 @@ int ocfs2_write_super_or_backup(struct ocfs2_super *osb,
>  				struct buffer_head *bh)
>  {
>  	int ret = 0;
> +	struct ocfs2_dinode *di = (struct ocfs2_dinode *)bh->b_data;
>  
>  	mlog_entry_void();
>  
> @@ -426,6 +427,7 @@ int ocfs2_write_super_or_backup(struct ocfs2_super *osb,
>  
>  	get_bh(bh); /* for end_buffer_write_sync() */
>  	bh->b_end_io = end_buffer_write_sync;
> +	ocfs2_compute_meta_ecc(osb->sb, bh->b_data, &di->i_check);
>  	submit_bh(WRITE, bh);
>  
>  	wait_on_buffer(bh);

      parent reply	other threads:[~2010-04-01  2:02 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-01  1:30 [Ocfs2-devel] Fix for online resize with metaecc Joel Becker
2010-04-01  1:34 ` Mark Fasheh
2010-04-01  2:02 ` Tao Ma [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4BB3FE9D.709@oracle.com \
    --to=tao.ma@oracle.com \
    --cc=ocfs2-devel@oss.oracle.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.