From: Dave Chinner <david@fromorbit.com>
To: Nuno Subtil <subtil@gmail.com>
Cc: xfs-oss <xfs@oss.sgi.com>
Subject: Re: XFS umount issue
Date: Wed, 25 May 2011 09:39:43 +1000 [thread overview]
Message-ID: <20110524233943.GI32466@dastard> (raw)
In-Reply-To: <BANLkTikj_ZY9g3mSmKAAv=qRaSvNQN=B3A@mail.gmail.com>
On Tue, May 24, 2011 at 03:18:11AM -0700, Nuno Subtil wrote:
> On Tue, May 24, 2011 at 00:54, Dave Chinner <david@fromorbit.com> wrote:
>
> ...
>
> >> > Ok, so there's nothing here that actually says it's an unmount
> >> > error. More likely it is a vmap problem in log recovery resulting in
> >> > aliasing or some other stale data appearing in the buffer pages.
> >> >
> >> > Can you add a 'xfs_logprint -t <device>' after the umount? You
> >> > should always see something like this telling you the log is clean:
> >>
> >> Well, I just ran into this again even without using the script:
> >>
> >> root@howl:/# umount /dev/md5
> >> root@howl:/# xfs_logprint -t /dev/md5
> >> xfs_logprint:
> >> data device: 0x905
> >> log device: 0x905 daddr: 488382880 length: 476936
> >>
> >> log tail: 731 head: 859 state: <DIRTY>
> >>
> >>
> >> LOG REC AT LSN cycle 1 block 731 (0x1, 0x2db)
> >>
> >> LOG REC AT LSN cycle 1 block 795 (0x1, 0x31b)
> >
> > Was there any other output? If there were valid transactions between
> > the head and tail of the log xfs_logprint should have decoded them.
>
> There was no more output here.
That doesn't seem quite right. Does it always look like sthis, even
if you do a sync before unmount?
> >> I see nothing in dmesg at umount time. Attempting to mount the device
> >> at this point, I got:
> >>
> >> [ 764.516319] XFS (md5): Mounting Filesystem
> >> [ 764.601082] XFS (md5): Starting recovery (logdev: internal)
> >> [ 764.626294] XFS (md5): xlog_recover_process_data: bad clientid 0x0
> >
> > Yup, that's got bad information in a transaction header.
> >
> >> [ 764.632559] XFS (md5): log mount/recovery failed: error 5
> >> [ 764.638151] XFS (md5): log mount failed
> >>
> >> Based on your description, this would be an unmount problem rather
> >> than a vmap problem?
> >
> > Not clear yet. I forgot to mention that you need to do
> >
> > # echo 3 > /proc/sys/vm/drop_caches
> >
> > before you run xfs_logprint, otherwise it will see stale cached
> > pages and give erroneous results..
>
> I added that before each xfs_logprint and ran the script again. Still
> the same results:
>
> ...
> + mount /store
> + cd /store
> + tar xf test.tar
> + sync
> + umount /store
> + echo 3
> + xfs_logprint -t /dev/sda1
> xfs_logprint:
> data device: 0x801
> log device: 0x801 daddr: 488384032 length: 476936
>
> log tail: 2048 head: 2176 state: <DIRTY>
>
>
> LOG REC AT LSN cycle 1 block 2048 (0x1, 0x800)
>
> LOG REC AT LSN cycle 1 block 2112 (0x1, 0x840)
> + mount /store
> mount: /dev/sda1: can't read superblock
>
> Same messages in dmesg at this point.
>
> > You might want to find out if your platform needs to (and does)
> > implement these functions:
> >
> > flush_kernel_dcache_page()
> > flush_kernel_vmap_range()
> > void invalidate_kernel_vmap_range()
> >
> > as these are what XFS relies on platforms to implement correctly to
> > avoid cache aliasing issues on CPUs with virtually indexed caches.
>
> Is this what /proc/sys/vm/drop_caches relies on as well?
No, drop_caches frees the page cache and slab caches so future reads
need to be looked up from disk.
> flush_kernel_dcache_page is empty, the others are not but are
> conditionalized on the type of cache that is present. I wonder if that
> is somehow not being detected properly. Wouldn't that cause other
> areas of the system to misbehave as well?
vmap is not widely used throughout the kernel, and as a result
people porting linux to a new arch/CPU type often don't realise
there's anything to implement there because their system seems to be
working. That is, of course, until someone tries to use XFS.....
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next prev parent reply other threads:[~2011-05-24 23:39 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-23 21:39 XFS umount issue Nuno Subtil
2011-05-24 0:02 ` Dave Chinner
2011-05-24 6:29 ` Nuno Subtil
2011-05-24 7:54 ` Dave Chinner
2011-05-24 10:18 ` Nuno Subtil
2011-05-24 23:39 ` Dave Chinner [this message]
2011-05-25 8:14 ` Nuno Subtil
2011-05-24 13:33 ` Paul Anderson
2011-05-24 19:10 ` Nuno Subtil
2011-05-25 0:29 ` Dave Chinner
2011-05-25 8:08 ` Nuno Subtil
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=20110524233943.GI32466@dastard \
--to=david@fromorbit.com \
--cc=subtil@gmail.com \
--cc=xfs@oss.sgi.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox