All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Ornati <ornati@fastwebnet.it>
To: Jens Axboe <axboe@suse.de>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	ck@vds.kolivas.org
Subject: Re: 2.6.17-ck1: fcache problem...
Date: Tue, 27 Jun 2006 11:21:05 +0200	[thread overview]
Message-ID: <20060627112105.0b15bfa1@localhost> (raw)
In-Reply-To: <20060625174358.GA21513@suse.de>

On Sun, 25 Jun 2006 19:43:59 +0200
Jens Axboe <axboe@suse.de> wrote:

> > > Hmm, and you are sure that the fs is properly umounted on reboot? Or is
> > > it just remounted ro? It looks like fcache_close_dev() isn't being
> > > called, so the cache serial doesn't match what we expect from the fs,
> > > hence fcache bails out since it could indicate that the fs has been
> > > changed without fcache being attached.
> > 
> > Ahh... it is the root fs and it's just remounted read-only by the
> > standard Gentoo scripts ;)
> > 
> > I don't think that unmounting it is trivial (you need to chroot to a
> > virtual FS or something...). Does any distro do it?
> 
> ro should be enough, something odd must be going on. I'll add it to the
> list of things to test tomorrow.

Since "fcache_close_dev()" is called by "ext3_put_super()" I have added
this stupid printk:

--- fs/ext3/super.c.orig        2006-06-27 10:47:15.000000000 +0200
+++ fs/ext3/super.c     2006-06-27 10:50:36.000000000 +0200
@@ -422,6 +422,8 @@ static void ext3_put_super (struct super

        has_fcache = test_opt(sb, FCACHE);

+       printk("!!! ext3_put_super !!!   has_fcache=%d\n", has_fcache);
+
        ext3_xattr_put_super(sb);
        journal_destroy(sbi->s_journal);
        if (!(sb->s_flags & MS_RDONLY)) {


It triggers on unmount but it doesn't on remount "ro".

So the problem is that "fcache_close_dev()" have zero chances to run ;)

-- 
	Paolo Ornati
	Linux 2.6.17-ck1 on x86_64

  reply	other threads:[~2006-06-27  9:21 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-25  7:35 2.6.17-ck1: fcache problem Paolo Ornati
2006-06-25 10:28 ` Jens Axboe
2006-06-25 13:23   ` Paolo Ornati
2006-06-25 17:43     ` Jens Axboe
2006-06-27  9:21       ` Paolo Ornati [this message]
2006-06-27  9:54         ` Jens Axboe
2006-06-27 10:24           ` Paolo Ornati
2006-06-27 13:04             ` Paolo Ornati
2006-06-27 13:10               ` Jens Axboe
2006-06-27 13:30                 ` Paolo Ornati
2006-06-27 13:35                   ` Jens Axboe
2006-06-27 14:20                     ` Paolo Ornati

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=20060627112105.0b15bfa1@localhost \
    --to=ornati@fastwebnet.it \
    --cc=axboe@suse.de \
    --cc=ck@vds.kolivas.org \
    --cc=linux-kernel@vger.kernel.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.