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.
next prev 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