Linux NILFS development
 help / color / mirror / Atom feed
From: Yuri Chislov <yuri-EJC0RcYdacHR7s880joybQ@public.gmane.org>
To: users-JrjvKiOkagjYtjvyW6yDsg@public.gmane.org
Subject: Re: NILFS errors.
Date: Sat, 3 Oct 2009 12:21:33 +0200	[thread overview]
Message-ID: <200910031221.34016.yuri@itinteg.net> (raw)
In-Reply-To: <20091003.131155.120264262.ryusuke-sG5X7nlA6pw@public.gmane.org>

On Saturday 03 October 2009 06:11:55 Ryusuke Konishi wrote:
> Well, I guess the corruption didn't come from data loss on the md
> layer.  OTOH, there is a possibility that md behavior has affected
> nilfs.
> 
> Could you confirm if the following patch makes a difference?
> 
> This patch doesn't recover corrupted file system, so you need a new
> file system.  But, this can prevent the directory corruption if it
> came from bio allocation errors on write path.
> 
> The patch was already merged at 2.6.32-rc1 but not yet backported
> to 2.6.31.y and 2.6.30.y.
> 
> If the patch is confirmed to have effect on your problem, I will send
> it to -stable trees.
> 
> Thanks,
> Ryusuke Konishi
> 
> diff --git a/fs/nilfs2/segbuf.c b/fs/nilfs2/segbuf.c
> index 9e3fe17..e6d9e37 100644
> --- a/fs/nilfs2/segbuf.c
> +++ b/fs/nilfs2/segbuf.c
> @@ -316,10 +316,10 @@ static struct bio *nilfs_alloc_seg_bio(struct
>  super_block *sb, sector_t start, {
>         struct bio *bio;
>  
> -       bio = bio_alloc(GFP_NOWAIT, nr_vecs);
> +       bio = bio_alloc(GFP_NOIO, nr_vecs);
>         if (bio == NULL) {
>                 while (!bio && (nr_vecs >>= 1))
> -                       bio = bio_alloc(GFP_NOWAIT, nr_vecs);
> +                       bio = bio_alloc(GFP_NOIO, nr_vecs);
>         }
>         if (likely(bio)) {
>                 bio->bi_bdev = sb->s_bdev;
> 

Thank you for patch. I have install it just now and reformat /dev/md4.
Is it planned some fsck tool? I can help with tests if necessary.

Yuri. 

  parent reply	other threads:[~2009-10-03 10:21 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-10-02 10:46 NILFS errors Yuri Chislov
     [not found] ` <200910021246.19909.yuri-EJC0RcYdacHR7s880joybQ@public.gmane.org>
2009-10-02 11:55   ` Ryusuke Konishi
     [not found]     ` <20091002.205508.118939509.ryusuke-sG5X7nlA6pw@public.gmane.org>
2009-10-02 16:57       ` Yuri Chislov
     [not found]         ` <200910021857.28569.yuri-EJC0RcYdacHR7s880joybQ@public.gmane.org>
2009-10-03  4:11           ` Ryusuke Konishi
     [not found]             ` <20091003.131155.120264262.ryusuke-sG5X7nlA6pw@public.gmane.org>
2009-10-03 10:21               ` Yuri Chislov [this message]
     [not found]                 ` <200910031221.34016.yuri-EJC0RcYdacHR7s880joybQ@public.gmane.org>
2009-10-03 11:13                   ` Ryusuke Konishi
     [not found]                     ` <20091003.201330.10734740.ryusuke-sG5X7nlA6pw@public.gmane.org>
2009-10-03 11:48                       ` Yuri Chislov

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=200910031221.34016.yuri@itinteg.net \
    --to=yuri-ejc0rcydachr7s880joybq@public.gmane.org \
    --cc=users-JrjvKiOkagjYtjvyW6yDsg@public.gmane.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox