All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Woodhouse <dwmw2@infradead.org>
To: joakim.tjernlund@transmode.se
Cc: linux-mtd@lists.infradead.org
Subject: Re: OOPS at mount
Date: Thu, 26 Apr 2007 09:29:55 +0100	[thread overview]
Message-ID: <1177576195.2755.286.camel@pmac.infradead.org> (raw)
In-Reply-To: <1177575617.28488.17.camel@gentoo-jocke.transmode.se>

On Thu, 2007-04-26 at 10:20 +0200, Joakim Tjernlund wrote:
> No this is a matter of installing a number of new execuables/libs in
> the FS and moving a few symlinks. Then a reboot.
> I suspect that the it is only one of these printout that actually
> makes the system crash, the other ones has problaby been there for a
> while. Does that make sense?

Yes. Only one of those was a complaint with "but the old size was 0",
which will have led to a NULL frag_last() since there were no frags.

> No, none acutally.

Hmmm.

> > 
> > This is with JFFS2 from 2.6.20, right? Not a bug in the read_inode code
> > I just committed a couple of days ago?
> 
> Plain 2.6.20 with my optimized scan you just commited.

By 'optimised scan' you just mean the fix to make it not crash, right?

Not the _real_ optimisation I committed before that, which rewrites the
entire read_inode() code path?

> > 
> > > Wonder how the lab manged to get that many corrupted nodes?
> > > One thing that is rather new in our system is that we trigger GC by
> > > sending HUP to the GC thread from a script in user space at startup
> > > and then every 24 hours.
> > 
> > That shouldn't make any difference. 
> 
> I know, but thats the only thing that I can think of that is somewhat
> unique to our system.

Hm. I'm trying to think how it could trigger the problem -- even if we
didn't block SIGHUP when we were in the GC routines, I still don't see
how we could lose old nodes of a file without writing out new ones.

You can reproduce this at will by mounting a _clean_ filesystem, then
"installing a number of new executables and moving a few symlinks" and
then rebooting?

Can you do that all with CONFIG_JFFS2_FS_DEBUG=1 and log it? I'll then
see where _all_ the nodes for these problematics files are on the first
boot, I'll see what happens when you make changes, and I'll see what we
find on the second boot.

If you kill the GC thread (or hack the kernel not to start it) that'll
make the dump a bit less noisy.

-- 
dwmw2

  reply	other threads:[~2007-04-26  8:29 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-25 15:09 OOPS at mount Joakim Tjernlund
2007-04-25 15:23 ` David Woodhouse
2007-04-25 15:40   ` Joakim Tjernlund
2007-04-25 15:44     ` David Woodhouse
2007-04-25 15:56       ` Joakim Tjernlund
2007-04-25 16:07         ` David Woodhouse
2007-04-25 17:39           ` Joakim Tjernlund
2007-04-26  6:32             ` David Woodhouse
2007-04-26  8:20               ` Joakim Tjernlund
2007-04-26  8:29                 ` David Woodhouse [this message]
2007-04-26  9:04                   ` Joakim Tjernlund
2007-04-26  9:21                     ` David Woodhouse
2007-04-26 11:26                     ` Joakim Tjernlund
2007-04-26 11:27                       ` David Woodhouse
2007-04-26 11:35                         ` Joakim Tjernlund
2007-04-26 11:41                           ` David Woodhouse
2007-04-26 12:49                             ` Joakim Tjernlund
2007-04-26 12:49                               ` David Woodhouse
2007-04-26 11:45             ` Joakim Tjernlund
2007-04-26 11:57               ` David Woodhouse
2007-04-26 12:31               ` David Woodhouse
2007-04-26 12:43                 ` Joakim Tjernlund
2007-04-26 12:49                   ` David Woodhouse
2007-04-26 13:00                     ` Joakim Tjernlund
2007-04-26 13:08                       ` David Woodhouse
2007-04-26 13:13                         ` Joakim Tjernlund
2007-04-26 13:15                           ` David Woodhouse
2007-04-26 13:31                             ` David Woodhouse
2007-04-26 13:39                             ` Joakim Tjernlund
2007-04-26 13:44                               ` David Woodhouse
2007-04-26 13:52                                 ` Joakim Tjernlund
2007-04-26 13:56                                   ` David Woodhouse
2007-04-26 14:37                                     ` Joakim Tjernlund
2007-04-26 19:39                                       ` David Woodhouse
2007-04-26 20:21                                         ` Joakim Tjernlund
2007-04-27  9:48                                           ` David Woodhouse
2007-04-25 15:46     ` David Woodhouse
2013-05-30 11:17 oops " Papp Tamas
2013-05-30 12:32 ` Josef Bacik
2013-05-30 12:55   ` Stefan Behrens
2013-05-30 14:03     ` Chris Mason
2013-05-30 14:59       ` Stefan Behrens
2013-05-30 16:37         ` Chris Mason
2013-06-03 11:56     ` Papp Tamas
2013-06-03 12:13       ` Hugo Mills
2013-05-31 14:55   ` Papp Tamas
2013-05-30 20:08 ` Stefan Behrens

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=1177576195.2755.286.camel@pmac.infradead.org \
    --to=dwmw2@infradead.org \
    --cc=joakim.tjernlund@transmode.se \
    --cc=linux-mtd@lists.infradead.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 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.